Skip to content
Commit 4a82adba authored by Nate Graham's avatar Nate Graham
Browse files

Fix Ctrl+F shortcut conflict

Summary:
The base app was always setting Ctrl+F as a search shortcut, even when the toolbar upon
which the search field lives was not visible. This caused the dreaded "ambiguous shortcut"
dialog to appear. The patch fixes the issue by setting the shortcut only when the toolbar
contains a search field, and unsetting it when the toolbar is hidden.

BUG: 405030
FIXED-IN: 5.18

Test Plan:
Hit {key Ctrl F} in both sidebar and icons view. In both cases, it focuses the search
field.

Reviewers: #plasma, GB_2, apol

Reviewed By: apol

Subscribers: apol, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D26305
parent a1695cb9
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment