diff options
| author | Matthias Clasen <mclasen@redhat.com> | 2020-04-05 19:52:25 -0400 |
|---|---|---|
| committer | Matthias Clasen <mclasen@redhat.com> | 2020-04-06 16:32:03 -0400 |
| commit | c297d45b8a19d84e0bc04ffad17df1382d7a9bf7 (patch) | |
| tree | 7c6059e55e44a27d9158b4395f022da1e0b3455c /gtk/gtkflowbox.c | |
| parent | dce8d251c618acbd38643a8ba4bd74110f178275 (diff) | |
| download | gtk+-c297d45b8a19d84e0bc04ffad17df1382d7a9bf7.tar.gz | |
gtk: Stop using modifier intents
Reviewing the existing settings, the only backend with
some differences in the modifier intent settings is OS X,
and we would rather have that implemented by interpreting
the existing modifiers in the appropriate way.
X11 Wayland Win32 OS X
primary ctrl ctrl ctrl mod2
mnemonic alt alt alt alt
context menu - - - ctrl
extend sel shift shift shift shift
modify sel ctrl ctrl ctrl mod2
no text alt|ctrl alt|ctrl alt|ctrl mod2|ctrl
shift group varies - - alt
GTK now uses the following modifiers:
primary ctrl
mnemonic alt
extend sel shift
modify sel ctrl
no text alt|ctrl
The context menu and shift group intents were not used
in GTK at all.
Update tests to no longer expect <Primary> to roundtrip
through the accelerator parsing and formatting code.
Diffstat (limited to 'gtk/gtkflowbox.c')
| -rw-r--r-- | gtk/gtkflowbox.c | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/gtk/gtkflowbox.c b/gtk/gtkflowbox.c index ded20adb33..73214fb90f 100644 --- a/gtk/gtkflowbox.c +++ b/gtk/gtkflowbox.c @@ -142,10 +142,10 @@ get_current_selection_modifiers (GtkWidget *widget, if (gtk_get_current_event_state (&state)) { - mask = gtk_widget_get_modifier_mask (widget, GDK_MODIFIER_INTENT_MODIFY_SELECTION); + mask = GDK_CONTROL_MASK; if ((state & mask) == mask) *modify = TRUE; - mask = gtk_widget_get_modifier_mask (widget, GDK_MODIFIER_INTENT_EXTEND_SELECTION); + mask = GDK_SHIFT_MASK; if ((state & mask) == mask) *extend = TRUE; } @@ -2992,19 +2992,9 @@ gtk_flow_box_add_move_binding (GtkWidgetClass *widget_class, GtkMovementStep step, gint count) { - GdkDisplay *display; GdkModifierType extend_mod_mask = GDK_SHIFT_MASK; GdkModifierType modify_mod_mask = GDK_CONTROL_MASK; - display = gdk_display_get_default (); - if (display != NULL) - { - extend_mod_mask = gdk_display_get_modifier_mask (display, - GDK_MODIFIER_INTENT_EXTEND_SELECTION); - modify_mod_mask = gdk_display_get_modifier_mask (display, - GDK_MODIFIER_INTENT_MODIFY_SELECTION); - } - gtk_widget_class_add_binding_signal (widget_class, keyval, modmask, "move-cursor", |