New issue
Advanced search Search tips

Issue 603519 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 460698
Owner: ----
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

DEL element is not repainted with new color on :hover

Reported by luiz...@gmail.com, Apr 14 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36 OPR/36.0.2130.46

Example URL:
https://jsfiddle.net/v6pdz536/2/

Steps to reproduce the problem:
1. Create a DEL tag with a SPAN inside
2. Add a CSS to set "DEL" color to x
3. Add a CSS to set "DEL:hover" color to y
4. Add a CSS to set "DEL:hover SPAN" color to x
5. Hover DEL element, notice there is no color change on the strike-through
4. Select the DEL element content and then its color change (sometimes partially depending on the selected piece)

What is the expected behavior?
The DEL strike-through should change to the color Y when hovered.

What went wrong?
The color didn't change on hover, it changes only if its content is selected

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: 49.0.2623.110  Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 20.0 r0

For the issue happens the colors for "DEL" and "DEL:hover SPAN" must match (even if by inheritance or from user agent style sheet). if they are different the issue does not occur.

In the first moment I thought SPAN could not be a valid child of DEL but I checked the the html specs and it looks like it is valid.

I also face this issue in Opera (36.0.2130.46), but IE11 and Firefox works as expected.
 

Comment 1 by keishi@chromium.org, Apr 15 2016

Components: -Blink Blink>CSS
Status: Available (was: Unconfirmed)
Color changes when setting hover state through devtools or when selecting the text so it seems to be a style invalidation issue.
Labels: Hotlist-Interop Needs-Bisect
Status: Untriaged (was: Available)
This issue was not triaged correctly. Unclear if this is a regression.

New testcase for bisect: https://jsfiddle.net/ou1a3gxu/

Comment 3 by r...@opera.com, Aug 10 2016

Labels: -Pri-2 Pri-3
Mergedinto: 460698
Status: Duplicate (was: Untriaged)
I doubt we have a regression here, but some points:

1. If you set the font-size to e.g. 30px on the span, you see that we incorreclty paint two different strike-throughs (compare with Gecko).

2. This is not limited to :hover. Setting a class on <del> to do the change has the same effect

3. We propagate text decorations during style recalc and not layout, which is wrong (see issue 553174)

4. I've debugged, and we incorrectly get currentColor for text decoration color on the synthesized AppliedTextDecorations for the span. It needs to become red when propagated from the <del>, even though it's currentColor on <del> since currentColor on the span is still black.

5. Oh, and it's the same issue as 460698 and related to 462142

Sign in to add a comment