After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 554506 - combining diacritics broken, became deadkeys
combining diacritics broken, became deadkeys
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Input Methods
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Hidetoshi Tajima
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2008-09-30 23:00 UTC by Simos Xenitellis
Modified: 2008-10-02 22:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch that identifies better if a key is a dead key. (522 bytes, patch)
2008-09-30 23:02 UTC, Simos Xenitellis
accepted-commit_now Details | Review
Updated version with longer comment. (1.19 KB, patch)
2008-10-01 10:46 UTC, Simos Xenitellis
accepted-commit_now Details | Review

Description Simos Xenitellis 2008-09-30 23:00:06 UTC
With the new updates in gtkimcontextsimple.c and the check_algorithmically(), there has been a regression. 

Specifically, it is not possible to type several verbatim combining diacritic marks, because they are converted and treated as dead keys.

For example, typing 'a', then the verbatim character 0x301 (acute). GTK+ now  converts them to dead key sequences.

The source of the problem is that the new keyboard layouts use 0x1000000 + 'codepoint' to signify a verbatim character. Combining diacritic Acute is 0x1000301. And the current "IS_DEAD_KEY()" macro evaluates all values over 0x1000000 as dead keys.
Comment 1 Simos Xenitellis 2008-09-30 23:02:35 UTC
Created attachment 119692 [details] [review]
Patch that identifies better if a key is a dead key.

This issue was first reported at FDO,
http://bugs.freedesktop.org/show_bug.cgi?id=17468
Comment 2 Matthias Clasen 2008-10-01 02:09:53 UTC
*** Bug 554192 has been marked as a duplicate of this bug. ***
Comment 3 Matthias Clasen 2008-10-01 02:11:37 UTC
Looks good.
Might be good to add some of the above explanation as a comment.
Comment 4 Simos Xenitellis 2008-10-01 10:46:06 UTC
Created attachment 119718 [details] [review]
Updated version with longer comment.
Comment 5 Simos Xenitellis 2008-10-01 10:54:32 UTC
2008-10-01  Simos Xenitellis  <simos@gnome.org>

        Bug 554506 – combining diacritics broken, became deadkeys

        * gtk/gtkimcontextsimple.c: added check if keysym is greater
        than 0x1000000, in this case it is not a dead key.

Comment 6 Simos Xenitellis 2008-10-02 22:36:33 UTC
Also committed to branch 'gtk-2-14',

2008-10-02  Simos Xenitellis  <simos@gnome.org>

        Bug 554506 – combining diacritics broken, became deadkeys

        * gtk/gtkimcontextsimple.c: added check if keysym is greater
        than 0x1000000, in this case it is not a dead key.