Closed
Bug 128987
Opened 23 years ago
Closed 23 years ago
nsIDOMXULLabeledControlEl.idl: "accessKey" should be "accesskey"
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
VERIFIED
WONTFIX
People
(Reporter: dean_tessman, Assigned: dean_tessman)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
aaronlev
:
review+
|
Details | Diff | Splinter Review |
Checked with Aaron, who checked in the original version. This prevents using
object.accesskey = <blah> to set an accesskey, which is the obvious try.
SetAttribute("accesskey") works, and has been used elsewhere such as
ComposerCommands.js.
|
Assignee | |
Comment 1•23 years ago
|
||
|
|
Assignee | |
Comment 2•23 years ago
|
||
Adding keywords. I did a grep through all the code and couldn't find .accessKey
used anywhere. The only thing this change would affect would be my patch for
bug 78153.
|
|
Assignee | |
Updated•23 years ago
|
|
||
Comment 3•23 years ago
|
||
Comment on attachment 72548 [details] [diff] [review]
this makes things consistent
r=aaronl
Attachment #72548 -
Flags: review+
|
|
Assignee | |
Comment 4•23 years ago
|
||
Blake, can you sr=?
|
||
Comment 5•23 years ago
|
||
It's "accessKey" in HTML DOM. Does XUL DOM need to be different?
|
|
Assignee | |
Comment 6•23 years ago
|
||
Oh, it is too:
http://www.w3.org/TR/2001/WD-DOM-Level-2-HTML-20011210/html.html#ID-59914154
Probably better to leave it as accessKey, then?
|
|
Assignee | |
Comment 7•23 years ago
|
||
hewitt, jag: any comments?
|
||
Comment 8•23 years ago
|
||
I'd say WONTFIX :-)
I guess the rule is: attribute names in markup are all lowercase, in DOM they're
interCaps.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
|
|
Assignee | |
Comment 9•23 years ago
|
||
Works for me, verifying. Nice catch, Jesse.
Status: RESOLVED → VERIFIED
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•