Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various fixes #19

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Various fixes #19

wants to merge 8 commits into from

Conversation

Snapperfish
Copy link

Could not get save/load to local storage to work.

Caps lock key on keyboard works now. It is assumed to be off initially. Press once to change to caps lock on keyboard mode. Press again to change to caps lock off (lowercase). Virtual keyboard will follow caps lock key. Click caps lock on virtual keyboard to change to virtual keyboard mode.

If caps lock on keyboard and virtual keyboard get out of sync (eg if initial assumption is incorrect) typing a character will fix the virtual keyboard display.

Tested on Mac in Firefox.

@whscullin whscullin self-requested a review March 15, 2020 23:22
@whscullin
Copy link
Owner

Sorry I didn't notice this when it came in, I get so many github notifications from work that unless I'm assigned or a reviewer I can miss stuff. Looks like some good stuff, checking it out now.

keyboard.optionKey(true);
} else {
keyboard.commandKey(true);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This configuration was selected to allow keyboards with only Alt keys (in this case a Chromebook) to be able to use the apple keys and match the layout. I'm not sure this change matches the correct behavior, since now neither right side option or command activates closed apple.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change was so my Mac keyboard would do both open- and closed- apple on option and command keys. Apologies if it breaks other keyboards/OSs.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The way it currently works is both command/open-apple keys generate open-apple, and left option/alt generates open-apple and right alt generates closed-apple, which I think is the standard Mac keyboard. The problem is probably more of a doc issue. Someday I will write some docs.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. My previous comment was wrong. The change was because the left option key on the Mac keyboard was triggering the right (closed-apple) key on the virtual keyboard, and the right option key on the Mac keyboard was triggering the left (open-apple) key on the virtual keyboard. With the change above, the side of the Mac keyboard key matched the virtual keyboard. I also have my Mac option and command keys swapped, so for me it was the command keys in each case. However, it is not a big issue. Let me know if you want any more help with this.

@whscullin whscullin mentioned this pull request Mar 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants