New issue
Advanced search Search tips
Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 13
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment
link

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

Reported by pascalschmitt@google.com, Jul 8 2016 Project Member

Issue description

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

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

Project Member
Labels: Hotlist-Google

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

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

Comment 4 by sheriffbot@chromium.org, Jul 12 2017

Project Member
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.

Comment 10 by bugdroid1@chromium.org, Dec 14

Project Member
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