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)
Tracking
()
RESOLVED
FIXED
Firefox 2
People
(Reporter: zeniko, Assigned: Gavin)
References
Details
(Keywords: fixed1.8.1, Whiteboard: [Fx2 theme change])
Attachments
(3 files)
21.45 KB,
image/png
|
Details | |
1.31 KB,
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
166.59 KB,
image/png
|
Details |
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.
Reporter | ||
Comment 1•18 years ago
|
||
BTW: Spacing between the individual elements still looks slightly flaky in general.
Reporter | ||
Comment 2•18 years ago
|
||
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?
Comment 3•18 years ago
|
||
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
Comment 4•18 years ago
|
||
*** Bug 347720 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•18 years ago
|
Assignee: nobody → gavin.sharp
Assignee | ||
Updated•18 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Assignee | ||
Comment 5•18 years ago
|
||
Adds a left margin to the urlbar and search bar.
Attachment #238005 -
Flags: review?(mconnor)
Assignee | ||
Comment 6•18 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Whiteboard: [Fx2 theme change] → [patch-r?][Fx2 theme change]
Comment 7•18 years ago
|
||
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+
Assignee | ||
Comment 8•18 years ago
|
||
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.
Description
•