Closed Bug 289300 Opened 19 years ago Closed 18 years ago

XUL menus appear compacted on Firefox/Mac

Categories

(Firefox :: Shell Integration, defect)

PowerPC
macOS
defect
Not set
normal

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).
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
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: kevin → stefanh
Status: NEW → ASSIGNED
Attachment #223091 - Flags: review?
Attachment #223091 - Flags: review? → review?(kevin)
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.
Fixed by the patch in bug 342515.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Depends on: 342515
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: