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

Add actions to the attribute table view so they get triggered by shortcut #38810

Merged
merged 1 commit into from
Sep 16, 2020

Conversation

troopa81
Copy link
Contributor

Description

Fixes #37503 and #37401

According to Qt::WidgetWithChildrenShortcut documentation

The shortcut is active when its parent widget, or any of its children has focus. Children which are top-level widgets, except pop-ups, are not affected by this shortcut context.

But in this case the parent is the menubar, not the dialog. So we need to add the actions to the QgsDualView in order to make the shortcut work when the QgsDualView has the focus.

@troopa81 troopa81 requested a review from nirvn September 16, 2020 14:54
@github-actions github-actions bot added this to the 3.16.0 milestone Sep 16, 2020
@troopa81
Copy link
Contributor Author

Only backport to 3.10 if #36955 is backported

@troopa81 troopa81 added Needs Backporting Bug Either a bug report, or a bug fix. Let's hope for the latter! labels Sep 16, 2020
@nyalldawson
Copy link
Collaborator

#36955 won't be backported -- it's too risky and not important enough for inclusion in LTR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

the keybord shortcut CTRL+C copy only the first cell, not the whole line
2 participants