[Webkit-unassigned] [Bug 24862] REGRESSION: Backspacing link from new line adds "text-decoration: none" to style

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 21 20:19:47 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=24862





------- Comment #7 from justin.garcia at apple.com  2009-04-21 20:19 PDT -------
> which calls removeInlineStyle, which sees text-decoration:none, which calls
> pushDownTextDecorationStyleAtBoundaries, which calls
> pushDownTextDecorationStyleAroundNode(start.node(), true) (start is the text
> node inside the anchor).  This code finds the anchor ancestor which does have
> text decoration, and calls extractAndNegateTextDecorationStyle, which negates
> it, setting text-decoration:none on the anchor.  Thus, we wind up with
> text-decoration:none on the anchor.

The bug is that when/after we push down text decoration, we should only be
removing it from the input range.  Seems like
pushDownTextDecorationStyleAtBoundaries and friends make the assumption that
the start position will be in a node that is necessarily part of that range,
because of work that applyInlineStyle did to split the text nodes that
contained the start and end.

Noticed that this happens with plain old underlined text too, which makes
sense.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list