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)

2.0 Branch
PowerPC
macOS
defect

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)

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.
Blocks: NewTheme
Tabs are also aligned to the left when they should be centered.
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)?
Please do file a bug and cc me (I bet this is a regression from ben's closebuttons patch).
Flags: blocking-firefox2?
Assignee: nobody → mglenn
Flags: blocking-firefox2? → blocking-firefox2+
Whiteboard: [Fx2 theme change]
Filed bug 347797 for comments 2-5.
Assignee: mglenn → jgoldman
Assignee: jgoldman → mconnor
Assignee: mconnor → dietrich
I don't know why was this change done. Note it doens't affect tabbrowser (browser.css overrides both rules)
Assignee: dietrich → bugs.mano
Status: NEW → ASSIGNED
Attachment #233345 - Flags: review?(mconnor)
Keywords: regression
Priority: -- → P2
Whiteboard: [Fx2 theme change] → [Fx2 theme change][needs review mconnor]
Target Milestone: --- → Firefox 2 beta2
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+
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.

Attachment

General

Creator:
Created:
Updated:
Size: