Philipp Hörist pushed to branch master at gajim / gajim


Commits:
305731ca by lovetox at 2020-11-23T14:37:51+01:00
ChatControlBase: Fix scroll hotkey infinite recursion

Handling of SHIFT + PAGE_UP/PAGE_DOWN was missing in the
ConversationTextview key press handler

- - - - -


1 changed file:

- gajim/chat_control_base.py


Changes:

=====================================
gajim/chat_control_base.py
=====================================
@@ -269,6 +269,10 @@ def __init__(self, parent_win, widget_name, contact, acct,
         CommandTools.__init__(self)
 
     def _on_conv_textview_key_press_event(self, textview, event):
+        if event.get_state() & Gdk.ModifierType.SHIFT_MASK:
+            if event.keyval in (Gdk.KEY_Page_Down, Gdk.KEY_Page_Up):
+                return Gdk.EVENT_PROPAGATE
+
         if event.keyval in COPY_MODIFIER_KEYS:
             # Don’t route modifier keys for copy action to the Message Input
             # otherwise pressing CTRL/META + c (the next event after that)
@@ -844,20 +848,20 @@ def _on_message_textview_key_press_event(self, textview, 
event):
         if self._type.is_groupchat:
             if event.keyval not in (Gdk.KEY_ISO_Left_Tab, Gdk.KEY_Tab):
                 self.last_key_tabs = False
+
         if event.get_state() & Gdk.ModifierType.SHIFT_MASK:
-            # CTRL + SHIFT + TAB
             if event.get_state() & Gdk.ModifierType.CONTROL_MASK and \
                             event.keyval == Gdk.KEY_ISO_Left_Tab:
                 self.parent_win.move_to_next_unread_tab(False)
                 return True
-            # SHIFT + PAGE_[UP|DOWN]: send to conv_textview
-            if event.keyval == Gdk.KEY_Page_Down or \
-                            event.keyval == Gdk.KEY_Page_Up:
+
+            if event.keyval in (Gdk.KEY_Page_Down, Gdk.KEY_Page_Up):
                 self.conv_textview.tv.event(event)
                 self._on_scroll(None, event.keyval)
                 return True
+
         if event.get_state() & Gdk.ModifierType.CONTROL_MASK:
-            if event.keyval == Gdk.KEY_Tab:  # CTRL + TAB
+            if event.keyval == Gdk.KEY_Tab:
                 self.parent_win.move_to_next_unread_tab(True)
                 return True
 



View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/305731ca69310f0a85b66cdfa71997f2828657ac

-- 
View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/305731ca69310f0a85b66cdfa71997f2828657ac
You're receiving this email because of your account on dev.gajim.org.


_______________________________________________
Commits mailing list
[email protected]
https://lists.gajim.org/cgi-bin/listinfo/commits

Reply via email to