Closed
Bug 348565
Opened 18 years ago
Closed 18 years ago
"Icons with Text" feature misaligned
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
RESOLVED
FIXED
Firefox 2 beta2
People
(Reporter: pcmedic, Assigned: pamg.bugs)
References
Details
(Keywords: fixed1.8.1, regression, Whiteboard: [Fx2 theme change])
Attachments
(5 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b1) Gecko/20060813 BonEcho/2.0b1
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b1) Gecko/20060813 BonEcho/2.0b1
It appears it won't allow the space for "Icons and Text" feature. Just "Icons" or "Text" is ok.
Reproducible: Always
It appears it won't allow the space for "Icons and Text" feature. Just "Icons"
or "Text" is ok.
Comment 3•18 years ago
|
||
this regressed between the nighty builds of 0812 and 0813
Comment 4•18 years ago
|
||
*** This bug has been marked as a duplicate of 347412 ***
No longer blocks: NewTheme
Status: NEW → RESOLVED
Closed: 18 years ago
Flags: blocking-firefox2?
Resolution: --- → DUPLICATE
Comment 5•18 years ago
|
||
Hrm, not quite the same bug.
Updated•18 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
Whiteboard: [Fx2 theme change]
Target Milestone: --- → Firefox 2 beta2
Updated•18 years ago
|
Status: REOPENED → NEW
Comment 6•18 years ago
|
||
Similar problem on Linux. The Back/Forward icons are a bit misplaced and they behave strangely (clicking on the text in the button opens the submenu instead of going back/forward).
Assignee | ||
Updated•18 years ago
|
Assignee: nobody → pamg.bugs
Assignee | ||
Comment 7•18 years ago
|
||
The max-height removed here was added for bug 347463, but as far as I can tell it can be safely removed once the lock icon is reduced to 16x16 pixels. Mark, please speak up if this regresses 347463 in some way I haven't tested.
Attachment #234280 -
Flags: review?(mconnor)
Attachment #234280 -
Flags: approval1.8.1?
Assignee | ||
Comment 8•18 years ago
|
||
This icon fits in the urlbar properly, so no max-height is needed and the toolbar can stretch appropriately when both icons and text are shown.
The other UI artifact shown in the first screenshot was fixed in bug 347400. Mano said he's about to split the back- and forward-button polish from the second screenshot into another bug.
Comment 9•18 years ago
|
||
See bug 349079, we should land these in sync.
Comment 10•18 years ago
|
||
Comment on attachment 234280 [details] [diff] [review]
Fixes original issue (toolbar text on Mac)
Hmm, I didn't know max-height could constrain parent elements like that, either way thanks! r+a=me on this branch-only patch
Attachment #234280 -
Flags: review?(mconnor)
Attachment #234280 -
Flags: review+
Attachment #234280 -
Flags: approval1.8.1?
Attachment #234280 -
Flags: approval1.8.1+
Comment 11•18 years ago
|
||
Should I file another bug about the Linux issue?
Assignee | ||
Comment 12•18 years ago
|
||
Attaching this to help with other fixes.
Comment 13•18 years ago
|
||
(In reply to comment #11)
> Should I file another bug about the Linux issue?
>
If it's still an issue (please check with a build that has the fix for bug 348565).
Assignee | ||
Comment 14•18 years ago
|
||
Landing patch and new image on branch. Marking as fixed since the back- and forward-button problems are being addressed in bug 349079.
Status: NEW → RESOLVED
Closed: 18 years ago → 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Comment 15•18 years ago
|
||
The same issues exists on Windows builds when comparing the main toolbar icons with the address and searchbar icons when displaying text. The text for the main toolbar icons looks as if it's aligned along the top of the toolbar while the address bar and searchbar text is centered vertically along the toolbar. If you have small icons enabled then this doesn't appear to be as much of an issue as all text appears centered.
BUILD: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060818 BonEcho/2.0b2 ID:2006081802
~B
Assignee | ||
Comment 16•18 years ago
|
||
(In reply to comment #15)
> The same issues exists on Windows builds when comparing the main toolbar icons
> with the address and searchbar icons when displaying text.
Sorry, I don't understand what problem you're describing. I'm looking at the 2005081803 build, and nothing looks too strange. Since the address and searchbar icons don't show text at all in the default theme, I'm guessing you're using another theme. Could you post a screenshot, please?
Comment 17•18 years ago
|
||
Right click > Customize > Display: [ Icons and text |v]
Comment 18•18 years ago
|
||
Oops, sorry, my previous comment was a mistake.
You need to log in
before you can comment on or make changes to this bug.
Description
•