ALT-O Keyboard shortcut fix. Issue: #888

[awareness481]

Description

Pull request to address the issue of keyboard shortcuts not working for Mac. I also discovered that the shortcuts weren't working for Windows as well.

Fixes #888

Type of Change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

The conditions for keyboard shortcuts for Windows & Mac were always falsy, unless CAPS lock was enabled. Tested locally on Windows, however please test additionally for Mac machines before merging.

Checklist: (Feel free to delete this section upon completion)

• I have performed a self-review of my own code
• My code follows the style guidelines of this project (I have run yarn format)
• New and existing unit tests pass locally with my changes (I have run yarn test)
• My changes generate no new warnings

[awareness481]

Seems like the conditional for Mac should use .toLowerCase() instead of .toUpperCase()?

[kale-stew]

I agree, @awareness481 - reviewing your PR now. Currently, the presenterMode tests are failing, might have something to do with that callout.

[kale-stew]

Aaaand you were right! Pushed up that change, we should be good to merge pending final tests. 👍 Thanks for the contribution!