(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?
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.
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.
Fine with me, as long as its well-documented somewhere.
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.
Target Milestone: --- → Future
I filed this bug?? Geez, what was I on? Marking INVALID
Status: NEW → RESOLVED
Last Resolved: 16 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.