Closed Bug 350742 Opened 18 years ago Closed 18 years ago

Missing left (and right) margin for address and search bars

Categories

(Firefox :: Toolbars and Customization, defect, P1)

2.0 Branch
x86
Windows XP
defect

Tracking

()

RESOLVED FIXED
Firefox 2

People

(Reporter: zeniko, Assigned: Gavin)

References

Details

(Keywords: fixed1.8.1, Whiteboard: [Fx2 theme change])

Attachments

(3 files)

If you move either of them to the very left of a toolbar, they look quite squeezed - and when moved to the right of the menu bar, there's no margin between the right-most menu and the bar at all.

Not sure how this will be handled when the Go button can be removed separately, but when done through CSS, there's currently no margin to the right either.
Attached image screenshot
BTW: Spacing between the individual elements still looks slightly flaky in general.
Depends on: 347964
After the Winstripe rollup, in the default layout the Home button's bevel now touches the address bar which looks kind of weird (especially since there's margin above and below the address bar). Requesting blocking.
Flags: blocking-firefox2?
Yeah, now that we're using system bevels, this is strange. I'll find someone to take this. Marking theme change, even though I don't know if it is, but just to get it to show up in the right queries.
Flags: blocking-firefox2? → blocking-firefox2+
Whiteboard: [Fx2 theme change]
Target Milestone: --- → Firefox 2
*** Bug 347720 has been marked as a duplicate of this bug. ***
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Priority: -- → P1
Attached patch patchSplinter Review
Adds a left margin to the urlbar and search bar.
Attachment #238005 - Flags: review?(mconnor)
Whiteboard: [Fx2 theme change] → [patch-r?][Fx2 theme change]
Comment on attachment 238005 [details] [diff] [review]
patch

r+a=me, let's get this in ASAP.
Attachment #238005 - Flags: review?(mconnor)
Attachment #238005 - Flags: review+
Attachment #238005 - Flags: approval1.8.1+
mozilla/browser/themes/winstripe/browser/browser.css 	1.17.2.57
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [patch-r?][Fx2 theme change] → [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: