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 97860 - ctl+key combos in main text field are broken
ctl+key combos in main text field are broken
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
2.0.x
Other other
: High major
: ---
Assigned To: gtk-bugs
gtk-bugs
: 90938 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-11-06 16:35 UTC by Luis Villa
Modified: 2011-02-04 16:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
here is a patch (1.10 KB, patch)
2003-02-04 00:12 UTC, Matthias Clasen
none Details | Review

Description Luis Villa 2002-11-06 16:35:04 UTC
This might be a gtk2.1 problem, but I only see it in gedit ATM- ctl+alt (to
escape the text field) and ctl+pgup/ctl+pgdown to change tabs both seem broken.
Comment 1 Paolo Maggi 2002-11-22 18:40:26 UTC
I think GtkTextView is eating ctl+pgup/ctl+pgdown.

In fact, ctrl+pgup moves the cursor at the beginning of the current line
and ctrl+pgdown moves the cursor at the end of the current line.

So, this is gtk+ bug.
Comment 2 Calum Benson 2002-11-28 16:54:09 UTC
Yes, people have been complaining about this sort of thing in a few
places recently.  

I think the only real fix might be to have GtkNotebook use keys other
than Ctrl+PgUp/PgDn for selecting next/previous tab, but it's kind of
a cross-platform standard unfortunately :/
Comment 3 Luis Villa 2002-12-05 16:32:23 UTC
Marking this high, as it's a user-visible problem, but not sure what
the best solution is. Can we change the textview bindings?
Comment 4 Luis Villa 2002-12-05 22:54:11 UTC
Making all 'high' textview bugs 2.2.0 as per havoc's request.
Comment 5 Havoc Pennington 2002-12-06 01:10:51 UTC
Isn't ctl+pgup/ctl+pgdown standard for both text widgets and notebooks?

I'm not sure there's a reasonable solution here. Not sure we should 
keep the bug open forever unless there's some kind of proposed 
course of action.
Comment 6 Calum Benson 2002-12-06 17:35:02 UTC
If we change it for textview we'd also have to change it for treeview,
scrolledwindow etc. which also use it for scroll left/right.  And
probably also for spinboxes which use Ctrl+PgUp/PgDn too, albeit for
something different.

I wouldn't be completely against choosing different keys for switching
between notebook tabs, but off the top of my head I can't think of
anything obvious, and we'd probably get a lot of complaints about
breakage.
Comment 7 Havoc Pennington 2002-12-06 18:17:57 UTC
It is also possible to have multiple keybindings for the same action, 
say an alternate notebook keybinding.
Comment 8 Paolo Maggi 2002-12-06 18:44:12 UTC
What about adding Ctrl+Alt+PgUp/Down as alternate notebook keybinding?
AFAIK, it is not a keybinding for GtkTextView.





        
Comment 9 Calum Benson 2002-12-13 18:36:14 UTC
You're right, no other control uses Ctrl+Alt+PgUp/PgDn for anything
AFAIK.
Comment 10 Matthias Clasen 2002-12-17 13:07:34 UTC
I just discovered one precedent for Ctl+Alt+PgUp for notebook tab switching: Opera 
(seen in Opera 6.0/Win)
Comment 11 Matthias Clasen 2003-02-04 00:12:22 UTC
Created attachment 14075 [details] [review]
here is a patch
Comment 12 Havoc Pennington 2003-02-20 19:04:26 UTC
The patch is to GtkNotebook so moving off TextView component so 
Owen will see it.
Comment 13 Andrew Sobala 2003-03-13 20:50:54 UTC
Killing GNOMEVER* keywords to stop cluttering up searches. The bug's
really up to the Gtk team's triaging now.
Comment 14 Owen Taylor 2003-03-28 19:45:11 UTC
Thoughts:

 - It won't solve everyone's problems since Control+Alt+Pg*
   are common window manager bindings.

   But the window manager will win so it doesn't make things
   any worse.

 - We don't already have any bindings on Control | Alt
   currently so this is sort of breaking new ground.

Looks OK to commit. Actually, it seems to have been 
comitted accidentally to HEAD, so just needs to go into
gtk-2-2 and a ChangeLog entry for HEAD.
 
Oh, also, 
 
 GDK_CONTROL_MASK|GDK_MOD1_MASK,

Woudl generally get spaces around the '|'.

 
Comment 15 Matthias Clasen 2003-03-28 23:45:30 UTC
Committed to both branches.
Comment 16 Matthias Clasen 2003-04-01 14:58:23 UTC
*** Bug 90938 has been marked as a duplicate of this bug. ***