New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 788187 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 785233
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

visibility not applied for children during transition

Reported by gess2...@gmail.com, Nov 23 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce the problem:
I have popups and try to animate open/close with css. Some steps during animation:

1. style='transition-property: none;';
2. class list manitpulation
3. reflow (popup.offsetWidth)
4. style=''
5. apply class for animation

animating classes set opacity visibility and transform

What is the expected behavior?
Children should inherit visibility from popup node.

What went wrong?
In some cases some children not inherit visibility form popup node.
Strange values in debugger for computed property.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 62.0.3202.94  Channel: stable
OS Version: 10.0
Flash Version:
 
issue.png
22.7 KB View Download
rules.png
74.3 KB View Download
regular.png
5.9 KB View Download
with-issue.png
2.5 KB View Download

Comment 1 by gess2...@gmail.com, Nov 23 2017

visibility not applied for children during(and after) transition

Comment 2 by meade@chromium.org, Nov 24 2017

Cc: flackr@chromium.org
Labels: Needs-Feedback
Owner: meade@chromium.org
Could you please attach or link your code? It's a little hard to tell what I'm looking at if I can't interact with it. Thanks!

Comment 3 by gess2...@gmail.com, Nov 24 2017

Hi, it took some time but finally I reproduced issue in small example (attached). It actually reversed key of issue we have in app - in example #inner must be hidden but it doesn't.
Not sure such functionality covered by spec, but in other browsers it works as I expected (well at list text is hidden :)).

index.html
986 bytes View Download
Project Member

Comment 4 by sheriffbot@chromium.org, Nov 24 2017

Cc: meade@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "meade@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Needs-Bisect
Able to repro on chrome 62.0.3202.94 on linux but not on 64.0.3269.3 on linux. Confirmed this is an issue with the computed style, possibly invalidation.

Requesting reverse bisect to see when this was fixed

Comment 6 by meade@chromium.org, Nov 27 2017

Status: Untriaged (was: Unconfirmed)
Cc: vamshi.k...@techmahindra.com
Labels: -Type-Bug -Pri-2 -Needs-Bisect hasbisect-per-revision Triaged-ET M-62 Needs-Triage-M62 OS-Linux OS-Mac Pri-1 Type-Bug-Regression
Owner: treib@chromium.org
Status: Assigned (was: Untriaged)
"Able to reproduce the issue on reported version 62.0.3202.94 using Mac 10.12.6, Ubuntu 14.04 and Windows 10. As the issue is not seen on the latest beta 63.0.3239.59 and  canary 64.0.3278.0 hence providing reverse bisect.

Reverse Bisect Info:
================
Good build: 63.0.3226.0  Revision(504841)
Bad build:   63.0.3225.0  Revision(504540)

CHANGELOG URL:
You are probably looking for a change made after 504560 (known good), but no later than 504561 (first known bad).

https://chromium.googlesource.com/chromium/src/+log/a9ba55c8571ab0d2624c83d413ccc7215a185f51..9f0c480da9cb188c82e16e6c6751875a60a114cf

suspect: https://chromium-review.googlesource.com/684186

Suspecting same from changelog.

@treib: Could you please confirm if this change is related to changes of  Rune Lillesveen, if not could you please help in assigning to correct owner

Thanks!"

Comment 8 by treib@chromium.org, Nov 27 2017

Cc: nainar@chromium.org
Owner: futhark@chromium.org
Mergedinto: 785233
Status: Duplicate (was: Assigned)

Sign in to add a comment