Closed Bug 900392 Opened 6 years ago Closed 5 years ago

Define new key name "SingleCandidate" for GTK and Qt

Categories

(Core :: DOM: Events, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: masayuki, Assigned: masayuki)

References

()

Details

(Whiteboard: [u= c= p=0])

Attachments

(1 file)

"SingleCandidate" is now defined for GTK and Qt.
Whiteboard: [u= c= p=0]
Status: NEW → ASSIGNED
I'm not sure what is this key, though. This key value was defined for GTK and Qt.

https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3Events-key.html#key-SingleCandidate
Attachment #8445188 - Flags: review?(romaxa)
Attachment #8445188 - Flags: review?(karlt)
Attachment #8445188 - Flags: review?(bugs)
This was added for evaluation in https://dvcs.w3.org/hg/dom3events/rev/9aa8afffed04 but I don't know why.

There exists an X11 keysym assignment but none of the XKeyboardConfig
layouts use it.
(In reply to Karl Tomlinson (needinfo?:karlt) from comment #2)
> This was added for evaluation in
> https://dvcs.w3.org/hg/dom3events/rev/9aa8afffed04 but I don't know why.

I requested the all keys which are defined for IME candidate list navigation:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=21116

> There exists an X11 keysym assignment but none of the XKeyboardConfig
> layouts use it.

Oh... Um, but I think it doesn't have problem for mapping it with the keysym.
Looks like it used to converting from Hangul to Hanja character.
http://www-01.ibm.com/support/knowledgecenter/ssw_aix_71/com.ibm.aix.nlsgdrf/korean_input_method.htm

It might be used by legacy keyboard layout in Korea.
Comment on attachment 8445188 [details] [diff] [review]
Support "SingleCandidate" key value on GTK and Qt

I just recently saw similar CR for zoomIn/Out, would it make sense to map all similar keys in one patch?
Attachment #8445188 - Flags: review?(romaxa) → review+
(In reply to Oleg Romashin (:romaxa) from comment #5)
> Comment on attachment 8445188 [details] [diff] [review]
> Support "SingleCandidate" key value on GTK and Qt
> 
> I just recently saw similar CR for zoomIn/Out, would it make sense to map
> all similar keys in one patch?

I separated the bugs and patches are for easier to discuss each case but not too messy.
Comment on attachment 8445188 [details] [diff] [review]
Support "SingleCandidate" key value on GTK and Qt

In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #4)
> Looks like it used to converting from Hangul to Hanja character.
> http://www-01.ibm.com/support/knowledgecenter/ssw_aix_71/com.ibm.aix.nlsgdrf/
> korean_input_method.htm
> 
> It might be used by legacy keyboard layout in Korea.

OK.  Thanks.
Attachment #8445188 - Flags: review?(karlt) → review+
Attachment #8445188 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/a4299c28c5cb
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.