XUL menus appear compacted on Firefox/Mac

RESOLVED FIXED

Status

()

RESOLVED FIXED
14 years ago
12 years ago

People

(Reporter: jhpedemonte, Assigned: stefanh)

Tracking

Trunk
PowerPC
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

14 years ago
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
(Assignee)

Comment 2

12 years ago
Created attachment 223091 [details] [diff] [review]
Add some native styling to xul menubars/menus

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?
(Assignee)

Updated

12 years ago
Attachment #223091 - Flags: review? → review?(kevin)
(Assignee)

Comment 3

12 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

12 years ago
Fixed by the patch in bug 342515.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Depends on: 342515
Resolution: --- → FIXED
(Assignee)

Updated

12 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.