-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Description
Details for the issue
What did you do?
I saw this shortcut in the UI for the "Open tab" button in "Execute SQL":
sqlitebrowser/src/MainWindow.ui
Line 1694 in 41fd313
<string>Ctrl+T</string> |
What did you expect to see?
So I tried to use it, but it never opens a new tab.
What did you see instead?
Inside QScintilla it is used for transposing lines. Outside the editor, it is eclipsed by this other action (which seem to work when the focus is in most widgets):
sqlitebrowser/src/MainWindow.ui
Line 1648 in 41fd313
<string>Ctrl+T</string> |
But sometimes (don't know why) when the focus is in the docks it does nothing and this warning is printed:
QAction::eventFilter: Ambiguous shortcut overload: Ctrl+T
I suppose we either make it to work inside the Execute SQL or remove it from that action (it was undocumented so probably nobody is used to that, although it might be more useful than its current usages).
I saw this while adding native key shortcut names to tooltips programmatically for #721.
Useful extra information
DB4S v3.11.99 (Dec 5 2018) on Ubuntu 18.04.1 LTS (linux/4.15.0-39-generic) [x86_64]
using SQLite Version 3.15.2
and Qt 5.9.5