Closed Bug 586748 Opened 9 years ago Closed 8 years ago

text-decorations are drawn through text areas

Categories

(Core :: Layout, defect)

x86
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: jrmuizel, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 585684 has some of the details on this problem and an example at https://bugzilla.mozilla.org/attachment.cgi?id=465037
Blocks: 585684
Note that this behavior is purposeful.  See http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/forms/nsTextControlFrame.cpp&rev=3.151#1589 (there's equvalent code in forms.css now) and the blame on revision 3.106 of nsTextControlFrame.cpp.
Summary: text-decoration is are drawn through text areas → text-decorations are drawn through text areas
(In reply to comment #1)
> Note that this behavior is purposeful.  See
> http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/forms/nsTextControlFrame.cpp&rev=3.151#1589
> (there's equvalent code in forms.css now) and the blame on revision 3.106 of
> nsTextControlFrame.cpp.

Is this the behaviour we want? It doesn't really seem useful to have text-decoration drawn on a text area.

If it is what we want, than I suppose we should fix the reftest.
> Is this the behaviour we want? 

An excellent question.  I don't think it is, myself...
(In reply to comment #4)
Confirmed concerning https://bugzilla.mozilla.org/attachment.cgi?id=465037 against Tinderbox Win32 Build http://hg.mozilla.org/integration/mozilla-inbound/rev/088a2d2cdf33 compared to yesterday's Nightly.
Yep looks like it.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: FIXED → ---
Target Milestone: --- → mozilla8
Patch for bug 403524 makes the anonymous div inside textareas and inputs into  inline-blocks, thereby ignoring text-decorations defined outside the textarea itself.
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.