Closed
Bug 434348
Opened 17 years ago
Closed 17 years ago
Shadows of tab labels are rendered incorrectly due to simple rules in my userChrome.css
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3.5
People
(Reporter: coordt, Assigned: jruderman)
References
Details
Attachments
(3 files)
30.70 KB,
image/png
|
Details | |
1.19 KB,
text/css
|
Details | |
749 bytes,
patch
|
kevin
:
review+
mconnor
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9) Gecko/2008051202 Firefox/3.0
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9) Gecko/2008051202 Firefox/3.0
When a tab appears, the light gray shadow is rendered as if the font size is fractionally bigger and the shadow thus renders bigger than the label.
Reproducible: Always
Steps to Reproduce:
1.Create more than one tab.
Actual Results:
Rendering of shadow is bigger than that of label
Expected Results:
Render of shadow is the same size as label
No extensions are enabled.
Mac OS X 10.5.2, MacBook Pro
Reporter | ||
Comment 1•17 years ago
|
||
Comment 2•17 years ago
|
||
Just to be safe, try Firefox in safe-mode:
http://support.mozilla.com/en-US/kb/Safe+Mode
Report your findings after
Updated•17 years ago
|
Component: Tabbed Browser → Theme
QA Contact: tabbed.browser → theme
Assignee | ||
Comment 4•17 years ago
|
||
Do you guys (Corey and Dominique) have a userChrome.css file that might be altering the size of either the text label or its shadow? I assume that's what was happening to the user in bug 427384. Look for it in the chrome subdirectory of your Firefox profile. In a stock profile, there will be a userChrome-example.css and a userContent-example.css, but no corresponding files without "-example".
FWIW, safe mode does not disable userChrome.css and userContent.css (see bug 333808).
Assignee | ||
Comment 5•17 years ago
|
||
It seems a little sketchy to me that ".tab-text" has a font specified, but its
sibling, ".tab-text-shadow", does not:
http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/browser/themes/pinstripe/browser/browser.css&rev=1.156&mark=1638-1646#1637
(DOM Inspector FTW.)
Assignee | ||
Comment 6•17 years ago
|
||
As a workaround, you can *create* a userChrome.css file containing:
.tab-text-shadow { display: none }
But it would be great if we also figured out the cause of the bug; it might be something that affects a lot of users and can be fixed.
Comment 7•17 years ago
|
||
(In reply to comment #5)
> It seems a little sketchy to me that ".tab-text" has a font specified, but its
> sibling, ".tab-text-shadow", does not:
>
> http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/browser/themes/pinstripe/browser/browser.css&rev=1.156&mark=1638-1646#1637
Yeah, this could be it.
Flags: blocking1.9.0.1?
Updated•17 years ago
|
Version: unspecified → 3.0 Branch
Reporter | ||
Comment 8•17 years ago
|
||
I did have a userChrome.css file. There wasn't much to it, but removing it fixed the issue. I'll attach a copy of the css file.
Reporter | ||
Comment 9•17 years ago
|
||
Updated•17 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 10•17 years ago
|
||
I can reproduce the bug with Corey's userChrome.css. This patch fixes Firefox to work correctly in that case, and doesn't seem to break the normal case.
Assignee | ||
Updated•17 years ago
|
Summary: Shadows of tab labels are rendered incorrectly → Shadows of tab labels are rendered incorrectly due to simple rules in my userChrome.css
Updated•17 years ago
|
Attachment #324511 -
Flags: review?(kevin) → review+
Assignee | ||
Updated•17 years ago
|
Attachment #324511 -
Flags: superreview?(mconnor)
Updated•17 years ago
|
Attachment #324511 -
Flags: superreview?(mconnor) → superreview+
Comment 11•17 years ago
|
||
Nice catch Corey
Assignee | ||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1
Updated•17 years ago
|
Flags: wanted1.9.0.x+
Flags: blocking1.9.0.1?
Flags: blocking1.9.0.1-
Comment 12•17 years ago
|
||
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b1pre) Gecko/20081002 Minefield/3.1b1pre.
Updated•16 years ago
|
Target Milestone: Firefox 3.1 → Firefox 3.5
You need to log in
before you can comment on or make changes to this bug.
Description
•