Closed Bug 32685 Opened 25 years ago Closed 24 years ago

using keyboard, can still pop out submenus for disabled menu items

Categories

(Core :: XUL, defect, P3)

Product:
Core
Component:
XUL
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: dean_tessman, Assigned: mikepinkerton)

References

(
URL
)

Details

(Whiteboard: [nsbeta3-])

Attachments

(1 file)

Here's the fix. It adds a MenuIsDisabled() method to nsIMenuFrame and nsMenuFrame.
(deleted), patch
Details | Diff | Splinter Review
Use the keyboard to navigate to the View > Use Stylesheet menu item, which is disabled and has a submenu associated with it. Press the right arrow. Actual Results: The associated submenu opens. Expected Behavior: Based on Windows menu behavior, the next item on the menu bar, Search in this example, should be opened. OS: NT4 sp6a Build: 2000032108
Aww, I was hoping no one would discover this. ;)
Target Milestone: ---
You know me and menus!
confirmed on Debian Linux/PowerPC, changing to XP.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows NT → All
Hardware: PC → All
I've got a fix for this at home. It would be attached by now if my stupid %@#$ internet connection hadn't gone down last night.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
Mass moving M17 bugs to M18
Target Milestone: M17 → M18
Adding 'patch' keyword. Should have done this a while back.
Keywords: patch
mass-moving all bugs to m21 that are not dofood+, or nsbeta2+
Target Milestone: M18 → M21
*spam*: transferring current XP Menu bugs over to jrgm, the new component owner. feel free to add me to the cc list (unless am the Reporter) of any of these, if you have any questions/etc.
QA Contact: sairuh → jrgm
nsbeta3-, but not futuring since we could still take the patch. ->pinkerton
Assignee: hyatt → pinkerton
Status: ASSIGNED → NEW
Whiteboard: [nsbeta3-]
*** Bug 43740 has been marked as a duplicate of this bug. ***
r=blake
a=waterson
checked in
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Hey, I didn't make it as a contributor in two of the three files - what a rip! (Just jokin')
Adding keyword to bugs with nsbeta3 triage value in status whiteboard so tracking queries will not be misled
Keywords: nsbeta3
verified fixed on WinNT using build 2000081404. I couldn't find a disabled menu with a submenu anymore in Mozilla, so I had to go into navigatorOverlay.xul and change: <menu value="&useStyleSheetMenu.label;" disabled="false"> to: <menu value="&useStyleSheetMenu.label;" disabled="true">
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: