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

Issue 618142 link

Starred by 13 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

transition-property on master element breaks fill property when declared on a use element

Reported by patrickk...@gmail.com, Jun 7 2016

Issue description

Chrome Version       : 53.0.2753.0
OS Version: OS X 10.11.5
URLs (if applicable) :
Other browsers tested:
  Add OK or FAIL after other browsers where you have tested this issue:
     Safari: PASS
  Firefox: PASS
     IE/dge: PASS

What steps will reproduce the problem?
1. go to https://jsbin.com/lamija/edit
2. observe the page
3.

What is the expected result?

The left column should be black, the right green

What happens instead of that?

both columns are black


Please provide any additional information below. Attach a screenshot if
possible.


UserAgentString: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2753.0 Safari/537.36



 
jsbin.lamija.11.html
890 bytes View Download
Originally reported by Lea Verou 
https://twitter.com/leaverou/status/740321621118853120
Components: Blink>SVG
Labels: OS-Android OS-Chrome OS-Linux OS-Windows
Status: Untriaged (was: Unconfirmed)
Confirmed in both Chrome 51.0.2704.63 and Chrome 50.0.2661.102.

Comment 3 by f...@opera.com, Jun 8 2016

Labels: -OS-Linux -OS-Android -OS-Windows -OS-Chrome -OS-Mac OS-All
Status: Available (was: Untriaged)
Summary: transition-property on master element breaks fill property when declared on a use element (was: transition-property on master element breaks fill prop when declared on a use element)
Project Member

Comment 4 by sheriffbot@chromium.org, Jun 8 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by f...@opera.com, Jun 8 2017

Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)
Labels: BugSource-User PaintTeamTriaged-20170608
Still happening.
 Issue 761022  has been merged into this issue.
 Issue 776339  has been merged into this issue.
Owner: schenney@chromium.org
Status: Assigned (was: Available)

Comment 10 by f...@opera.com, Nov 30 2017

 Issue 789763  has been merged into this issue.

Comment 11 by f...@opera.com, Nov 30 2017

Labels: -Pri-3 Pri-2

Comment 12 by f...@opera.com, Dec 1 2017

What happens here is: When resolving style for the <use>d <rect>, we pass the non-<use>d as the context element (because of the style-matching rules IIRC.) Since the non-<use>d element has already had its style resolved, the transition machinery thinks a transition should start for the 'fill' property and thus applies a transition to the style resolved for the <use>d <rect>, yielding the black color.
The (document) order of the elements thus matter, and if the <rect> and <use> change places in the testcase, the right colors will be used.

The solution would seem to be to pass the <use>d element rather than its corresponding element, and then fix up the flat tree structure in the StyleResolver (ElementResolveContext and possibly more.) Implementing issue 785246 would probably also address this.

Comment 13 by f...@opera.com, Jan 13 2018

 Issue 801815  has been merged into this issue.

Comment 14 by f...@opera.com, Jan 29 2018

 Issue 806616  has been merged into this issue.

Comment 15 by f...@opera.com, Feb 26 2018

Cc: sindhu.chelamcherla@chromium.org
 Issue 811042  has been merged into this issue.
Cc: schenney@chromium.org
Owner: ----
Status: Available (was: Assigned)
No idea why I owned this.
 Issue 877087  has been merged into this issue.
Cc: f...@opera.com
 Issue 876693  has been merged into this issue.

Sign in to add a comment