1124608 - Support D3E EventModifierInit

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

dictionary SharedKeyboardAndMouseEventInit : UIEventInit {
    boolean ctrlKey = false;
    boolean shiftKey = false;
    boolean altKey = false;
    boolean metaKey = false;
    boolean keyModifierStateAltGraph = false;
    boolean keyModifierStateCapsLock = false;
    boolean keyModifierStateFn = false;
    boolean keyModifierStateFnLock = false;
    boolean keyModifierStateHyper = false;
    boolean keyModifierStateNumLock = false;
    boolean keyModifierStateOS = false;
    boolean keyModifierStateScrollLock = false;
    boolean keyModifierStateSuper = false;
    boolean keyModifierStateSymbol = false;
    boolean keyModifierStateSymbolLock = false;
};

This is necessary for supporting to initialize all modifier states at creating a KeyboardEvent or MouseEvent.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

and MouseEventInit and KeyboardEventInit extends this one?

Please call the dictionary something else.
Like EventModifierInit : UIEventInit

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Olli Pettay [:smaug] from comment #1)
> and MouseEventInit and KeyboardEventInit extends this one?

Yes, D3E spec defines so.

> Please call the dictionary something else.
> Like EventModifierInit : UIEventInit

Okay, thank you.

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8554108 [details] [diff] [review]
Patch


>+// Called as SharedKeyboardAndMouseEventInit in D3E spec.
Please get the dictionary fixed there too.
What if some other event type wants to use the dictionary? Then it won't 
be only for KeyboardEvent or MouseEvent


>+// NOTE: Gecko doesn't support commented out modifiers yet.
>+dictionary EventModifierInit : UIEventInit
>+{
>+  boolean ctrlKey = false;
>+  boolean shiftKey = false;
>+  boolean altKey = false;
>+  boolean metaKey = false;
>+  boolean keyModifierStateAltGraph = false;
>+  boolean keyModifierStateCapsLock = false;
>+  boolean keyModifierStateFn = false;
>+  // boolean keyModifierStateFnLock = false;
>+  // boolean keyModifierStateHyper = false;
>+  boolean keyModifierStateNumLock = false;
>+  boolean keyModifierStateOS = false;
>+  boolean keyModifierStateScrollLock = false;
>+  // boolean keyModifierStateSuper = false;
>+  // boolean keyModifierStateSymbol = false;
>+  boolean keyModifierStateSymbolLock = false;
>+};

Why the new names are so super long.
This feels a bit developer hostile.

Sorry that I didn't realize this earlier.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Hmm, so, do you think that these issues should be fixed in the spec side?

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

yes. I think we should figure out better API and get that spec'ed and implement that.

It is not clear to me why the properties in the dictionary need to be so long and repetitive.

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Filed:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=27902
https://www.w3.org/Bugs/Public/show_bug.cgi?id=27903

Comment 8

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

renamed:

dictionary EventModifierInit : UIEventInit {
    boolean ctrlKey = false;
    boolean shiftKey = false;
    boolean altKey = false;
    boolean metaKey = false;
    boolean modifierAltGraph = false;
    boolean modifierCapsLock = false;
    boolean modifierFn = false;
    boolean modifierFnLock = false;
    boolean modifierHyper = false;
    boolean modifierNumLock = false;
    boolean modifierOS = false;
    boolean modifierScrollLock = false;
    boolean modifierSuper = false;
    boolean modifierSymbol = false;
    boolean modifierSymbolLock = false;
};

https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3-Events.html#idl-def-EventModifierInit

Comment 9

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch

Comment 10

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1443401a43fb

teoli:

Looks like there are no entry for Event*Init dictionary in MDN. For example, https://developer.mozilla.org/en/docs/Web/API/Event/Event documents it as just an argument of Event constructor. However, the series of Event*Init has independent hierarchy. E.g.,

interface: Event -> UIEvent -> MouseEvent
dictionary: EventInit -> UIEventInit -> EventModifierInit -> MouseEventInit

So, I think that MDN should have /docs/Web/API/Event/(Init|Dictionary)/ or something.

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/1443401a43fb