342197 - Can't type '}' with a french keyboard

Status: VERIFIED FIXED

(Core :: Widget: Win32, defect)

Description

[Régis Caspar]

This works on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.4) Gecko/20060508 Firefox/1.5.0.4 ID:2006050817 and Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060215 Firefox/1.6a1 ID:2006021506 
but fails in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060620 Minefield/3.0a1 ID:2006062004 [cairo]

 -> Core / Widgets:Win32 because of the location of nsKeyboardLayout.cpp

Comment 1

[Dão Gottwald [:dao]]

It's '~' with German keyboard layout.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060620 Minefield/3.0a1 ID:2006062012 [cairo]

Comment 2

[Régis Caspar]

Attachment: tool

With this "tool" I have some strangeness in the output:

 -> j <- 
charCode = 106
keyCode = 0
ctrlKey = false
shiftKey = false
altKey = false
--------------------------------------
 -> @ <- 
charCode = 64
keyCode = 0
ctrlKey = false
shiftKey = false
altKey = false
--------------------------------------
 -> ] <- 
charCode = 93
keyCode = 0
ctrlKey = false
shiftKey = false
altKey = false
--------------------------------------
 -> + <- 
charCode = 43
keyCode = 0
ctrlKey = true
shiftKey = false
altKey = true
--------------------------------------

Note the difference between ']' and '+' (which should be '}'), the ctrlKey and altKey seem wrong for the last.
NB: On a french keyboard (i.e. azerty), '@', ']' and '}' are obtained by [Alt Gr] + [à], [)], [=], the three keys preceding the backspace one.

I found a picture of a (ugly) french keyboard (if someone need to see the key layout):
http://www.ibt.ca/images/AZERTY_1.jpg

Comment 3

[Frank Wein [:mcsmurf]]

Probably depends on Bug 339723.

Comment 4

[Olivier Vit (just a reporter)]

also fails on Mozilla/5.0 (Windows; U; Windows NT 5.0; fr-FR; rv:1.9a1) Gecko/20060618 SeaMonkey/1.5a

Comment 5

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

probably, this is fixed by the additional patch of bug 339723.
please check it.

Comment 6

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

-> FIXED

Comment 7

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

Oops, the tree is closed now, I backed-out the patch.

Comment 8

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

re-checked-in.

-> FIXED

Comment 9

[Régis Caspar]

Verified on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060717 Minefield/3.0a1 ID:2006071709 [cairo]

Thanks Masayuki :)

Comment 10

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

Thank you for your testing.

-> v.