Closed
Bug 289300
Opened 19 years ago
Closed 18 years ago
XUL menus appear compacted on Firefox/Mac
Categories
(Firefox :: Shell Integration, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jhpedemonte, Assigned: stefanh)
References
Details
Attachments
(1 obsolete file)
Offshoot from bug 210204. When displaying XUL menus on Firefox/Mac, the menu items appear one after the other, with no padding between them. The same XUL menu appears fine on Mozilla, and on Firefox with other themes. So the Pinstripe theme needs to be updated. I added the following code to mozilla/toolkit/themes/pinstripe/global/menu.css, and it seemed to fix the problem: +/* ::::: menu/menuitems in menubar ::::: */ + +menubar > menu { + border: 1px solid transparent; + padding: 2px 5px 2px 7px; + margin: 1px 0px 1px 0px; +} This seems to fix the padding issue. The menu doesn't fit in with the theme, though. The menu items are drawn using the system theme, and there is a noticeable difference between the XUL menu and the rest of the toolbar (i.e. load http://www.mozilla.org/docs/tutorials/sitenav/4.xul in a nightly from April 6 or later).
Comment 1•19 years ago
|
||
This is a firefox theme bug, not toolkit/widgets, Kevin would need to come up with nice css rules :)
Assignee: nobody → kevin
Component: XP Toolkit/Widgets: XUL → OS Integration
Product: Core → Firefox
QA Contact: bugs.mano
Assignee | ||
Comment 2•18 years ago
|
||
This will make xul menubars look more "native" (basically putting back stuff from classic). I also ported some stuff from the patch in bug 301105 in order to make the behaviour of the menubar be more "native".
Assignee | ||
Updated•18 years ago
|
Attachment #223091 -
Flags: review? → review?(kevin)
Assignee | ||
Comment 3•18 years ago
|
||
Another testcase here: https://bugzilla.mozilla.org/attachment.cgi?id=194850. Note that left/right ends (borders) of the menubar look a bit washed-out.
Assignee | ||
Comment 4•18 years ago
|
||
Fixed by the patch in bug 342515.
Assignee | ||
Updated•18 years ago
|
Attachment #223091 -
Attachment is obsolete: true
Attachment #223091 -
Flags: review?(kevin)
You need to log in
before you can comment on or make changes to this bug.
Description
•