Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not work well with QTerminal #5

Closed
probonopd opened this issue Oct 16, 2021 · 3 comments
Closed

Does not work well with QTerminal #5

probonopd opened this issue Oct 16, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@probonopd
Copy link
Member

probonopd commented Oct 16, 2021

Looks like QTerminal really doesn't play well with QKeySequenceOverride in place.

Running it causes its config file to be rewritten automatically like this:

sudo env PAGER=cat pkg.real update && sudo env PAGER=cat pkg.real upgrade
(...)
[Shortcuts]
Add%20Tab=\x2318\x21e7T
Bottom%20Subterminal=\x2325\x2193
Clear%20Active%20Terminal=\x2318\x21e7X
Close%20Tab=\x2318\x21e7W
Collapse%20Subterminal=
Copy%20Selection=\x2318\x21e7\x43
Find=\x2318\x21e7\x46
Fullscreen=F11
Handle%20history=
Hide%20Window%20Borders=
Left%20Subterminal=\x2325\x2190
Move%20Tab%20Left=\x2325\x21e7\x2190|\x2318\x21e7Pg\x2191
Move%20Tab%20Right=\x2325\x21e7\x2192|\x2318\x21e7Pg\x2193
New%20Window=\x2318\x21e7N
Next%20Tab=\x2318Pg\x2193
Next%20Tab%20in%20History=\x2318\x21e7\x21e5
Paste%20Clipboard=\x2318\x21e7V
Paste%20Selection=\x21e7Ins
Preferences...=
Previous%20Tab=\x2318Pg\x2191
Previous%20Tab%20in%20History=\x2318\x21e5
Quit=\x2318Q
Rename%20Session=\x2325\x21e7S
Right%20Subterminal=\x2325\x2192
Show%20Tab%20Bar=
Split%20Terminal%20Horizontally=
Split%20Terminal%20Vertically=
Tab%201=
Tab%2010=
Tab%202=
Tab%203=
Tab%204=
Tab%205=
Tab%206=
Tab%207=
Tab%208=
Tab%209=
Toggle%20Bookmarks=\x2318\x21e7\x42
Toggle%20Menu=\x2318\x21e7M
Top%20Subterminal=\x2325\x2191
Zoom%20in=\x2318+
Zoom%20out=\x2318-
Zoom%20reset=\x2318\x30

Apparently QTerminal is making the assumption that what gets returened by QKeySequence::toString() is in a certain format.

@probonopd
Copy link
Member Author

Possibly we need to do more work to support the distinction between QKeySequence::NativeText and QKeySequence::PortableText?

@probonopd
Copy link
Member Author

Workaround: Close QTerminal, run cp /usr/share/skel/dot.config/qterminal.org/qterminal.ini ~/.config/qterminal.org/qterminal.ini && sudo chown root ~/.config/qterminal.org/qterminal.ini

@probonopd probonopd added the bug Something isn't working label Oct 24, 2021
@probonopd
Copy link
Member Author

The same happens if we run Qt Creator. It rewrites shortcut in a way that renders them unusable.

@@ -598,7 +596,7 @@
     <string>Go Back</string>
    </property>
    <property name="shortcut">
-    <string>Alt+Left</string>
+    <string>⌥Left</string>
    </property>
   </action>
   <action name="actionGoForward">
@@ -613,7 +611,7 @@
     <string>Go Forward</string>
    </property>
    <property name="shortcut">
-    <string>Alt+Right</string>
+    <string>⌥Right</string>
    </property>
    <property name="visible">
     <bool>true</bool>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant