Closed Bug 29109 Opened 25 years ago Closed 24 years ago

Context menu items that have submenus have smaller vertical size then other menu items.

Categories

(Core :: XUL, defect, P3)

x86
All
defect

Tracking

()

VERIFIED WORKSFORME
Future

People

(Reporter: sspitzer, Assigned: mikepinkerton)

References

Details

Jason (jce2@po.cwru.edu) pointed out that in the context menu for the thread pane, the copy and move items are too close together. (Take a look, they do get layed out closer than other items.) his fix was to add an addition menu seperator diff messenger.xul messenger.xul.old 197d196 < <menuseparator id="threadPaneContext-sep-move"/> but that goes against the spec. http://gooey/client/5.0/specs/mail/messenger/MailMenus.html#Context (sorry jason, that is an internal url, but basically adding a separator between copy and move is not in the spec.)
it looks like the rdf generated menu items gets drawn differently then static menu items, so this might affect other menus.
1) Why don't you post the spec somewhere where we can see it? (other then for commercial items, obviously) 2) The distance problem only seems to occur when two items with submenus are close together. I moved the "Move to" item to the top of the context menu (which I wanted to do anyways) and the distance between that and the item next to it is normal.
Jason, we have posted some specs, but there's effort attached to sanitizing them, and sometimes there's higher priority work to do (like right before a beta :-) The separator thing is interesting, but I don't think that's the UI we want. The real bug sounds like an XPMenus issue, so I'm reassigning to trudelle.
Assignee: phil → trudelle
recharacterized the bug...I've played around a bit with the chrome, and I've discovered that ANY menu item that leads to a submenu has a smaller vertical size then menu items without a sub-menu.
Summary: in the context menu for the thread pane, the copy and move items are too close together → Context menu items that have submenus have smaller vertical size then other menu items.
reassigning to pinkerton for triage. I'm not sure whether they are too close together, or other items too far apart, but I don't think they should be outdented.
Assignee: trudelle → pinkerton
m18.
Status: NEW → ASSIGNED
Target Milestone: M18
QA Contact: lchiang → nbaca
Mass moving M18 bugs to M19
Target Milestone: M18 → M19
*** Bug 31133 has been marked as a duplicate of this bug. ***
mass-moving all bugs to m21 that are not +/-
Target Milestone: M19 → M21
can we ship with this? yes.
Target Milestone: M21 → Future
I think it would be ok.
Component: Mail Window Front End → XP Toolkit/Widgets: Menus
Product: MailNews → Browser
this doesn't appear to be a problem anymore, with either classic or modern.
Severity: normal → minor
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
Verified Worksforme.
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: nbaca → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.