Closed
Bug 108170
Opened 22 years ago
Closed 19 years ago
Decimal separator ',' (comma) on german keyboard (on keypad/numpad) shows up as '|' (pipe)
Categories
(Core Graveyard :: GFX: Gtk, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: c, Assigned: blizzard)
References
Details
Attachments
(2 files)
540 bytes,
patch
|
caillon
:
review+
bryner
:
superreview+
dbaron
:
approval1.8b+
|
Details | Diff | Splinter Review |
444 bytes,
patch
|
blizzard
:
review+
bryner
:
superreview+
caillon
:
approval1.8b+
|
Details | Diff | Splinter Review |
My system: SuSE 7.1 / KDE, german keyboard, fresh (english) Mozilla CVS build. If I press the decimal separator in the numeric block of the keyboard (this is supposed to be a comma on german keyboards) I get a vertical bar ('|') instead (everywhere I can type text in Mozilla). This is because GDK_KP_Separator is translated into '|' in nsGtkEventHandler.cpp, but should be ','.
Reporter | ||
Comment 1•22 years ago
|
||
Reporter | ||
Updated•22 years ago
|
Updated•22 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1
Comment 2•22 years ago
|
||
1.1alpha is gone.. c@c07.de (Andreas M. "Clarence" Schneider) : You must request a review via mail. (if the patch isn't out of date) Are you still there ?
Target Milestone: mozilla1.1alpha → ---
Comment 4•22 years ago
|
||
Is this still an issue, and if so, is the patch the appropriate fix. (Sorry for the fact that this has not moved anywhere).
Comment 5•20 years ago
|
||
*** Bug 205533 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Summary: Decimal separator ',' on german keyboard shows up as '|' → Decimal separator ',' on german keyboard (on keypad) shows up as '|'
Comment 6•20 years ago
|
||
*** Bug 218283 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Assignee: jag → blizzard
Component: XP Toolkit/Widgets → GFX: Gtk
QA Contact: jrgmorrison → ian
Assignee | ||
Comment 7•20 years ago
|
||
Is the seperator on english keyboards a slash?
Comment 8•20 years ago
|
||
According to gtk2 source, 0xFFAC (KP_Separator) is supposed to produce ',', while KP_Decimal is supposed to produce a '.' KP_Separator is not present on English keyboard. But it is in some european countries, instead of KP_Decimal, such as finnish (see bug http://bugs.mandrakelinux.com/query.php?bug=376 ) The problem is present in both gtk1 and gtk2 version of mozilla (in source/widget/src/gtk2/nsGtkKeyUtils.cpp for gtk2)
Comment 9•20 years ago
|
||
*** Bug 193560 has been marked as a duplicate of this bug. ***
Comment 10•20 years ago
|
||
*** Bug 159733 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Summary: Decimal separator ',' on german keyboard (on keypad) shows up as '|' → Decimal separator ',' (comma) on german keyboard (on keypad/numpad) shows up as '|' (pipe)
Comment 11•20 years ago
|
||
The problem still exists in current builds. This patch fixes the GTK2 source tree in the same way, as the GTK1 patch does.
Comment 12•20 years ago
|
||
Comment on attachment 160938 [details] [diff] [review] Patch against GTK2 source tree It would be nice, if the old GTK1 patch could also get r/sr.
Attachment #160938 -
Flags: superreview?(bryner)
Attachment #160938 -
Flags: review?(blizzard)
Assignee | ||
Updated•19 years ago
|
Attachment #160938 -
Flags: review?(blizzard) → review+
Comment 13•19 years ago
|
||
Bug is still not resolved. It's annoying if you have to type a lot of numbers... My system: Fedora core 1; linux 2.4.22-1.2115.nptlsmp; GUI: Gnome-german; Mozilla 1.7-german, Keyboard: german
Comment 14•19 years ago
|
||
This bug is still present, confirming with both Firefox 1.0 (Debian package 1.0-4) and Seamonkey 1.7.3 (Debian package 1.7.3-5). Is anything slated to fix this? It really *is* annoying, especially for online banking.
Comment 15•19 years ago
|
||
C'mon dudes! This bug was opened _2001_ and a patch available immediatly and it's still not incorporated properly and still present in 2005! I can confirm that the same problem is present on swedish keyboards too. Feels awkward to DL the source and recompile to fix a _oneline_ bug. Good work on the rest nonetheless.. ;-)
Comment 16•19 years ago
|
||
hmm... the patch even has r+ already, what happened to sr+? Is Bryner around to do superreviewing or should we ask s.o. else?
Flags: blocking1.8a6?
Flags: blocking-aviary1.1?
Updated•19 years ago
|
Attachment #56238 -
Flags: superreview?(bryner)
Attachment #56238 -
Flags: review?(blizzard)
Updated•19 years ago
|
Attachment #56238 -
Attachment description: patch → gtk1 patch
Comment 17•19 years ago
|
||
too late for non-critical fixes. blocking-
Flags: blocking1.8a6? → blocking1.8a6-
Updated•19 years ago
|
Flags: blocking1.8b?
Flags: blocking1.8a6?
Flags: blocking1.8a6-
Comment 18•19 years ago
|
||
oops, somehow the 1.8a6- got overridden...
Flags: blocking1.8a6? → blocking1.8a6-
Comment 19•19 years ago
|
||
Confirming on Danish keyboard, with Ubuntu packages. And with official Firefox official binaries.
Updated•19 years ago
|
Flags: blocking1.8b?
Flags: blocking1.8b-
Flags: blocking1.8a6-
Flags: blocking-aviary1.1?
Flags: blocking-aviary1.1-
Comment 20•19 years ago
|
||
May I ask _why_ blocking-aviary1.1 was denied? Firefox 1.1 is not scheduled until June and there are patches, of which one already has r+ - so why not implement it?
Updated•19 years ago
|
Attachment #160938 -
Flags: superreview?(bryner) → superreview+
Updated•19 years ago
|
Attachment #56238 -
Flags: superreview?(bryner) → superreview+
Comment 21•19 years ago
|
||
Comment on attachment 160938 [details] [diff] [review] Patch against GTK2 source tree low risk patch - just changes the character that's generated for GDK_KP_Separator
Attachment #160938 -
Flags: approval1.8b?
Updated•19 years ago
|
Attachment #56238 -
Flags: review?(blizzard) → review+
Comment 22•19 years ago
|
||
Comment on attachment 160938 [details] [diff] [review] Patch against GTK2 source tree a=caillon for 1.8beta
Attachment #160938 -
Flags: approval1.8b? → approval1.8b+
Comment 23•19 years ago
|
||
Comment on attachment 56238 [details] [diff] [review] gtk1 patch would be nice to have this too in 1.8b - the same patch for gtk2 is already approved.
Attachment #56238 -
Flags: approval1.8b?
Updated•19 years ago
|
Attachment #56238 -
Flags: approval1.8b? → approval1.8b+
Comment 24•19 years ago
|
||
gtk1: Checking in widget/src/gtk/nsGtkEventHandler.cpp; /cvsroot/mozilla/widget/src/gtk/nsGtkEventHandler.cpp,v <-- nsGtkEventHandler.cpp new revision: 1.189; previous revision: 1.188 done gtk2: Checking in widget/src/gtk2/nsGtkKeyUtils.cpp; /cvsroot/mozilla/widget/src/gtk2/nsGtkKeyUtils.cpp,v <-- nsGtkKeyUtils.cpp new revision: 1.12; previous revision: 1.11 done marking FIXED.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•