Closed
Bug 55934
Opened 24 years ago
Closed 22 years ago
modifiers on menu items are case-sensitive
Categories
(Core :: XUL, defect, P3)
Tracking
()
RESOLVED
INVALID
Future
People
(Reporter: deanis74, Assigned: hyatt)
References
()
Details
(Assigning to hyatt from CVS Blame) Maybe this is as designed, and if it is then mark this as invalid because it's hopefully documented somewhere. But as far as I can tell http://lxr.mozilla.org/seamonkey/source/layout/xul/base/src/nsMenuFrame.cpp#1413 requires that the modifiers be specified as "shift", "alt", etc., and not "Shift", "Alt", "SHIFT", "ALT", or anything else. Wouldn't it be easier for these to allow case-insensitivity?
Comment 1•24 years ago
|
||
"easier"?
Sorry, I meant easier from a XUL designer's perspective. As far as I can tell, the modifiers have to match case otherwise this code won't pick them up.
Comment 3•24 years ago
|
||
XUL is case sensitive. <menuitem/> is recognised in XUL, <MENUITEM/> isn't. I don't think there is any added value in making this attribute value case insensitive. Actually it looks like anywhere attributes have bound values we prefer all lowercase over mixed case, so I guess it's a style issue too. I suggest wontfix.
Comment 5•24 years ago
|
||
It's close to being documented :-] (although, this doesn't actually state that attribute values should be lower case). http://www.mozilla.org/xpfe/xptoolkit/xulintro.html 'A Word on Case and Namespaces, and Filetypes' "... tags and attributes will, as a rule, always be lower case as suggested in the XHTML Working Draft." cc: oeschger for doc issues.
I filed this bug?? Geez, what was I on? Marking INVALID
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → INVALID
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.
Description
•