New issue
Advanced search Search tips

Issue 626744 link

Starred by 6 users
Status: Fixed
Owner:
schenney@chromium.org
Closed: Dec 13
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

SVG: currentColor change not updated on hover for children of groups with filters

Project Member Reported by pascalschmitt@google.com, Jul 8 2016 
UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.106 Safari/537.36

Example URL:
http://codepen.io/hanger/pen/bZoEER?editors=1100

Steps to reproduce the problem:
Apply an SVG filter to a group (here: identity), use currentColor on an element inside that the group.
When changing the currentColor on hover, the element doesn't change color. i.e. the rectangle stays black.

rect {
  fill: currentColor;
}

.filtered {
  filter: url(#myfilter);
}

.hovering:hover {
  color: green;
}

<svg>
  <defs>
    <filter id="myfilter">
      <feMerge>
         <feMergeNode in="SourceGraphic" />
      </feMerge>
    </filter>
  </defs>

  <g class="hovering filtered">
    <rect width="100" height="100" />
  </g>
</svg>

The codepen shows similar cases that don't have the problem too.

What is the expected behavior?
currentColor change should cause children to be redrawn correctly, i.e. rectangle becomes green on hover.

What went wrong?
The children should have been redrawn.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? Yes 

Chrome version: 51.0.2704.106  Channel: stable
OS Version: 
Flash Version: Shockwave Flash 22.0 r0

Comment 1 by dtapu...@chromium.org, Jul 8 2016

Components: -Blink Blink>SVG
Project Member

Comment 2 by sheriffbot@chromium.org, Jul 9 2016

Labels: Hotlist-Google

Comment 3 by f...@opera.com, Jul 11 2016

Labels: -OS-Linux
Status: Available (was: Unconfirmed)
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 12 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, Jul 12 2017

Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)

Comment 6 by schenney@chromium.org, Jul 13 2017

Labels: PaintTeamTriaged-20170713 BugSource-User
Owner: schenney@chromium.org
Status: Assigned (was: Available)
Still broken. Invalidation issue, it seems.

Comment 7 by jan.boes...@incors.com, Dec 13 

This seems to be fixed (testing with 71.0.3578.98)

Comment 8 by f...@opera.com, Dec 13

Status: Fixed (was: Assigned)
Thanks!

A bisection shows that it was probably fixed by 7004d7e19279e2d31f92fb295e06a8b987700f18.
Project Member

Comment 10 by bugdroid1@chromium.org, Dec 14 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a5fce755fd7c748af92be3995602f5ee1c9dc969

commit a5fce755fd7c748af92be3995602f5ee1c9dc969
Author: Fredrik Söderquist <fs@opera.com>
Date: Fri Dec 14 01:15:17 2018

Add reftest-wait to css/filter-effects/svg-sourcegraphic-currentcolor-dynamic-001.html

Attempt to stabilize in Gecko.

Bug:  626744 
Change-Id: Idfa06176948d1ffaf6ba8e65263bf5818ffd321b
Reviewed-on: https://chromium-review.googlesource.com/c/1375875
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Stephen Chenney <schenney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616543}
[modify] https://crrev.com/a5fce755fd7c748af92be3995602f5ee1c9dc969/third_party/blink/web_tests/external/wpt/css/filter-effects/svg-sourcegraphic-currentcolor-dynamic-001.html
►

Sign in to add a comment