Skip to content

Ambiguous shortcut: Ctrl+T #1647

@mgrojo

Description

@mgrojo

Details for the issue

What did you do?

I saw this shortcut in the UI for the "Open tab" button in "Execute SQL":

<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):

<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

Metadata

Metadata

Assignees

Labels

bugConfirmed bugs or reports that are very likely to be bugs.

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions