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

Add windows compatibility #17

Merged
merged 1 commit into from
Nov 8, 2021
Merged

Add windows compatibility #17

merged 1 commit into from
Nov 8, 2021

Conversation

zwang20
Copy link
Member

@zwang20 zwang20 commented Nov 8, 2021

No description provided.

@Edward-Ji
Copy link
Contributor

Usually, Mac uses the command and Windows uses the control modifier key (but not at the same time). Can we distinguish the two platforms and create two separate keymaps?

@zwang20
Copy link
Member Author

zwang20 commented Nov 8, 2021

Usually, Mac uses the command and Windows uses the control modifier key (but not at the same time). Can we distinguish the two platforms and create two separate keymaps?

ok

@zwang20
Copy link
Member Author

zwang20 commented Nov 8, 2021

we should make this user configurable

@Edward-Ji
Copy link
Contributor

Edward-Ji commented Nov 8, 2021

we should make this user configurable

But the default could be command on Mac and control on Windows as per convention. If we were to make it configurable, we as well build a complete shortcut settings tab. I just noticed that you created #20.

@Edward-Ji Edward-Ji merged commit 16ea872 into Quark-Games:master Nov 8, 2021
@Edward-Ji
Copy link
Contributor

Although we could have updated the README to reflect this before merging, I feel like the README need a complete rework anyway.

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