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 107211 - GOK can't match more than 2 modifiers
GOK can't match more than 2 modifiers
Status: RESOLVED FIXED
Product: gok
Classification: Deprecated
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: David Bolter
David Bolter
Depends on: 108137
Blocks: 122112
 
 
Reported: 2003-02-27 18:36 UTC by bill.haneman
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.3/2.4


Attachments
patch to gok-key.c,checks label sizes whenever labels are updated. (10.12 KB, patch)
2003-03-11 01:39 UTC, bill.haneman
none Details | Review

Description bill.haneman 2003-02-27 18:36:43 UTC
something seems to be broken in GOK's parsing code for modifiers; though
more than one modifier can be matched using strtok:

i.e. "shift+mod2" or "shift+capslock", when three modifiers are strung
together the algorithms fails, i.e

"shift+capslock+mod2" doesn't match.

Of course long-term this whole approach probably needs rethinking since we
don't really want to do this much string comparison and tokenizing just to
figure out what labels to match... but in the short term it'd be nice to
fix this since it _is_ breaking GOK's ability properly display keypad keys for

Shift+CapsLock+NumLock.
Comment 1 bill.haneman 2003-03-11 01:39:33 UTC
Created attachment 14913 [details] [review]
patch to gok-key.c,checks label sizes whenever labels are updated.
Comment 2 bill.haneman 2003-03-11 01:40:18 UTC
oops. please ignore the above patch, it belongs to another bug!
Comment 3 Calum Benson 2003-08-07 16:16:30 UTC
Apologies for spam... marking as GNOMEVER2.3 so it appears on the official GNOME
bug list :)
Comment 4 bill.haneman 2003-09-10 12:06:33 UTC
adding dependency on 108137; we should fix the modifier code in
generally as part of the (much needed!) modifier-code refactor.
Comment 5 bill.haneman 2003-09-12 13:14:33 UTC
reducing severity for now, but this bug will block effective use with
some international keyboards.
Comment 6 bill.haneman 2003-09-12 13:40:12 UTC
internationalization implications since many non-En keyboards use
multiple modifiers to produce a wider character set.
Comment 7 bill.haneman 2003-10-30 10:50:40 UTC
A fix for this is in the gok_i18n branch.  We can close this bug when
we merge.
Comment 8 bill.haneman 2003-11-04 13:34:46 UTC
merge completed.

Need to file RFE regarding keyboard 'groups' however, since they don't
turn up in the standard X modifier bits.  We need to be able to detect
the current 'group' state in order to deal with really complex XKB
keymaps.