Changed the new marking shortcuts from control + key to alt + shift + key to avoid conflicting with common browser shortcuts (such as control + c for copy).
Also added shortcuts for redo and discuss, and made it so the shortcut help modal gets the application name from DoubtfireConstants instead of being fixed as Formatif.
Type of change
[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
How Has This Been Tested?
Tested all shortcuts on firefox and chrome.
Testing Checklist:
[x] Tested in latest Chrome
[ ] Tested in latest Safari
[x] Tested in latest Firefox
Checklist:
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have requested a review from @macite and @jakerenzella on the Pull Request
Description
Changed the new marking shortcuts from
control + key
toalt + shift + key
to avoid conflicting with common browser shortcuts (such ascontrol + c
for copy).Also added shortcuts for redo and discuss, and made it so the shortcut help modal gets the application name from
DoubtfireConstants
instead of being fixed as Formatif.Type of change
How Has This Been Tested?
Tested all shortcuts on firefox and chrome.
Testing Checklist:
Checklist: