Testcase: http://www.hixie.ch/tests/adhoc/css/fonts/variant/001.xml If you have font-variant: small-caps set with text-decoration: underline, the colour of the small caps (not the big caps) is drawn the colour of the underline, not the colour of the text (like the big caps).
I'm guessing one chunk of code is doing a SetColor on a rendering context and not undoing it, and another chunk of code is relying on the color not being changed.
yeah nsTextFrame::RenderString never sets the color on the rendering context (this is called from PaintTextSlowly) unliks PaintAsciiText, which does set the color.
Assignee: font → dbaron
Priority: P3 → P1
Target Milestone: Future → mozilla1.4alpha
OK, this needs a new testcase, since it's now quirks-only, since bug 1777 never triggers this ickiness. The issue is that RenderString calls DrawString and DrawTextDecorations in a loop one after the other. So the calls to SetColor before the call into RenderString are not enough; the color needs to be set before every DrawString in RenderString
Actually, not that simple. Those SetColor() calls _used_ to be there. Then anthonyd removed them as "redundant" and "breaking selection colors". Too bad there's no bug number cited and no steps to reproduce.... (see http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/layout/html/base/src&command=DIFF_FRAMESET&file=nsTextFrame.cpp&rev2=1.247&rev1=1.246)
*** Bug 6574 has been marked as a duplicate of this bug. ***
selection colors look fine to me
Comment on attachment 139752 [details] [diff] [review] patch Yeah, if selection colors are still good, r+sr=bzbarsky.
Target Milestone: mozilla1.4alpha → mozilla1.7alpha
Fix checked in to trunk, 2004-02-03 10:27 -0800.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Hmm.. In a current build if I select the text "HAS: PASS" in that testcase, the selection colors are off -- the "HAS:" is green while the "PASS" is white. The "HAS:" is white up until I select the "P". Similarly, if I select the "SE" in "PASSED", the "PAS" is painted in white (so it's white on white). Reopening; that patch is no good...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to comment #11) > Selection problem mentioned in comment 10 is bug 235101. Adding dependency. ...so this bug is still fixed.
Status: REOPENED → RESOLVED
Closed: 16 years ago → 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.