Closed
Bug 347533
Opened 19 years ago
Closed 19 years ago
[Mac] 3px vertical gap between native-theme tabs and horizontal bar after Pinstripe visual refresh
Categories
(Firefox :: General, defect, P2)
Tracking
()
RESOLVED
FIXED
Firefox 2 beta2
People
(Reporter: mark, Assigned: asaf)
References
Details
(Keywords: fixed1.8.1, regression, Whiteboard: [Fx2 theme change])
Attachments
(2 files)
35.31 KB,
image/png
|
Details | |
650 bytes,
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Following the visual refresh, there is a 3px vertical gap between tabs drawn in the native appearance and the horizontal bar beneath the tabs from which the tabs are supposed to appear to protrude. This is visible in the tab strips in Tools:Page Info, Firefox:Preferences:Advanced, and Firefox:Preferences:Advanced:Encryption:View Certificates.
Reporter | ||
Comment 1•19 years ago
|
||
Comment 2•19 years ago
|
||
Tabs are also aligned to the left when they should be centered.
Reporter | ||
Comment 3•19 years ago
|
||
Comment 2 regressed earlier on the road between 1.5 to 2.0 - I noticed when it happened, but forgot to file a bug. Maybe it should be filed separately (unless it's already been filed)?
Assignee | ||
Comment 4•19 years ago
|
||
Please do file a bug and cc me (I bet this is a regression from ben's closebuttons patch).
Flags: blocking-firefox2?
![]() |
||
Updated•19 years ago
|
Assignee: nobody → mglenn
Flags: blocking-firefox2? → blocking-firefox2+
Whiteboard: [Fx2 theme change]
Reporter | ||
Comment 5•19 years ago
|
||
Filed bug 347797 for comments 2-5.
![]() |
||
Updated•19 years ago
|
Assignee: mglenn → jgoldman
![]() |
||
Updated•19 years ago
|
Assignee: jgoldman → mconnor
![]() |
||
Updated•19 years ago
|
Assignee: mconnor → dietrich
Assignee | ||
Comment 6•19 years ago
|
||
I don't know why was this change done. Note it doens't affect tabbrowser (browser.css overrides both rules)
Assignee | ||
Updated•19 years ago
|
Keywords: regression
Assignee | ||
Updated•19 years ago
|
Priority: -- → P2
Whiteboard: [Fx2 theme change] → [Fx2 theme change][needs review mconnor]
Target Milestone: --- → Firefox 2 beta2
![]() |
||
Comment 7•19 years ago
|
||
Comment on attachment 233345 [details] [diff] [review]
restore old margin and padding for 'normal' tabs
r+a=me (1.8 branch only at this point)
Attachment #233345 -
Flags: review?(mconnor)
Attachment #233345 -
Flags: review+
Attachment #233345 -
Flags: approval1.8.1+
Assignee | ||
Comment 8•19 years ago
|
||
Checking in toolkit/themes/pinstripe/global/tabbox.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/global/tabbox.css,v <-- tabbox.css
new revision: 1.2.26.3; previous revision: 1.2.26.2
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [Fx2 theme change][needs review mconnor] → [Fx2 theme change]
You need to log in
before you can comment on or make changes to this bug.
Description
•