----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129299/ -----------------------------------------------------------
Review request for KDE Frameworks, David Faure and Elvis Angelaccio. Repository: kxmlgui Description ------- Add a warning at the createGui stage about ambiguous shortcuts being found in the same action collection. This is usually a developer issue, but the error message about ambiguity will only show up when someone tries to use the shortcut, so it is relatively easy to miss if you do not try all your actions via a shortcut. Also if the involved shortcut is one of the non primary shortcuts of edit_cut, just give it away, since it's usually Shift+Delete being fought over. Diffs ----- src/kxmlguiwindow.cpp 519fb26 Diff: https://git.reviewboard.kde.org/r/129299/diff/ Testing ------- gwenview now defaults to Shift+Delete being "Hard delete" and not "Cut", if you remove the if (action == editCutAction || existingShortcutAction == editCutAction) { part, you get warning about the actions involved Thanks, Albert Astals Cid
