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
- Branch
- zoom-to-fit-shortcut (branched from frameworks)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 972 Build 985: arc lint + arc unit
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.)