Details
Details
- Open Cirkuit
- Check if no shortcut conflict warnings appear
- Load a file and check if zoom shortcuts work
Diff Detail
Diff Detail
- Repository
- R657 Cirkuit
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
The new shortcut is just a suggestion. If you wish to use another or even any better solution to a possible conflict, I can surely update this patch.
Comment Actions
Thanks, works great. Unless @asturmlechner or @kmorwinski find anything wrong with this until Monday, let's land this to the frameworks branch.
As for the choice of shortcut, the new one should actually be easier to type with only one hand.
(Disclaimer: I originally proposed the change, because instead of adding it to Gwenview it makes more sense to standardize it everywhere. Cirkuit was the only app with a conflict, so I asked Peter to help resolve it.)