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 653136 - Ctrl + Alt functionality in Caribou
Ctrl + Alt functionality in Caribou
Status: RESOLVED OBSOLETE
Product: caribou
Classification: Applications
Component: default
git master
Other Linux
: Normal normal
: ---
Assigned To: caribou-maint
caribou-maint
Depends on:
Blocks:
 
 
Reported: 2011-06-21 22:27 UTC by Nohemi
Modified: 2021-05-25 17:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix compilation issue (4.77 KB, patch)
2011-06-21 23:18 UTC, Nohemi
none Details | Review
Ctrl/Alt key (9.71 KB, patch)
2011-06-23 17:50 UTC, Nohemi
none Details | Review
Add Ctrl/Alt key functionality (13.72 KB, patch)
2011-06-30 17:43 UTC, Nohemi
none Details | Review
Modifier key rework. (13.76 KB, patch)
2011-07-13 23:34 UTC, Eitan Isaacson
none Details | Review

Description Nohemi 2011-06-21 22:27:42 UTC
I'm working on creating the patch for Ctrl/Alt/Shift functionality, as stated in the summary. So far, I have been having trouble with the signal passing, since the single key must alert the keyboard to store the "latch" until a non-mod key is pressed. Here's what I have so far.
Comment 1 Nohemi 2011-06-21 23:18:28 UTC
Created attachment 190401 [details] [review]
Fix compilation issue
Comment 2 Nohemi 2011-06-23 17:50:49 UTC
Created attachment 190523 [details] [review]
Ctrl/Alt key 

This is my hefty fix to the Ctrl/Alt key functionality. Eitan and Dan have told me to take a look again to see if I can use any existing methods to simplify the code. Will do that, but for now this works :)
Comment 3 Nohemi 2011-06-30 17:43:55 UTC
Created attachment 191044 [details] [review]
Add Ctrl/Alt key functionality

Patch that GNOME Onscreen keyboard is currently using for Ctrl/Alt functions.
Comment 4 Eitan Isaacson 2011-07-01 18:51:28 UTC
I need to look at this more, but two things that jump out are:
1. This should be rebased onto caribou master. We are now using an XML format that is more flexible.
2. The names "Alt", and "Control" should be changed to X key names like Control_L  and Alt_L for consistency with the rest of the format. Look at keysymdef.h.
Comment 5 Eitan Isaacson 2011-07-13 23:34:08 UTC
Created attachment 191928 [details] [review]
Modifier key rework.
Comment 6 Eitan Isaacson 2011-07-13 23:39:07 UTC
(In reply to comment #5)
> Created an attachment (id=191928) [details] [review]
> Modifier key rework.

Patch should be applied to your HEAD modifiers branch in github[1], it is here for illustrative purposes only, and ultimately will be squashed and cleaned up with Nohemi's other patches.

The main thing I did here was keep most of the modifier logic encapsulated in the KeyModel, and left it open for other interaction modes, for example long press for lock (as opposed to latch), and visual feedback for hardware modifier key presses (ie. if you press ctrl on your real keyboard, you should see the key become depressed in caribou). For this purpose I got rid of most of the "latch" language.

1. https://github.com/danwinship/caribou/tree/modifiers
Comment 7 André Klapper 2021-05-25 17:46:08 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/caribou/-/issues/

Thank you for your understanding and your help.