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

Make keyboard layouts easier to find #123

Open
Sogolumbo opened this issue May 1, 2021 · 4 comments
Open

Make keyboard layouts easier to find #123

Sogolumbo opened this issue May 1, 2021 · 4 comments

Comments

@Sogolumbo
Copy link

Sogolumbo commented May 1, 2021

Edit:
Already implemented but slightly unintuitive to find, see comments below.

Problem

Currently a Querty layout is assumed to convert the input into notes. This leads to wrong notes or no notes being played on different keyboard layouts.

It would be nice if other language specific layouts would also work.

Possible solution

Detect the language and default to the most popular layout from that language to do the conversion.
Maybe it's also possible to get the layout directly from the browser, I don't know.

Workaround

Change the system keyboard layout to querty. This will not always make all keys work but it will fix the z/y in qwertz layouts.

To make the workaround more viable (if a proper solution won't be implemented):
Detect the language and inform the user about the keyboard layout if the most popular layout from their language is not querty.

@meszaros-lajos-gyorgy
Copy link
Collaborator

Hi!

There are already a few alternative keyboard layouts added to scale workshop that might solve the issue for you. There is a panel on the right hand side of the app labelled Isomorphic keyboard settings, where you can select a different layout via the Computer keyboard layout dropdown.

If the only issue with the current layout is that Z and Y is flipped, then Hungarian (QWERTZ) will solve exactly that.

@Sogolumbo
Copy link
Author

Thanks!
I didn't see that. That's exactly the kind of thing I was looking for.
I didn't search at that spot because I didn't think I wanted to do something "isomorphic" (although the term may be technically correct). I guess I didn't even fully process the meaning of that word and skipped the rest of the menu title. Maybe it would make sense to rename that menu to just "keyboard settings".
Also it says

Press QWERTY keys to play current tuning.

at the top. Maybe it would make sense to add some note there to inform people that querty is not the only option (because that's the conclusion I drew from this sentence. I thought if there's no possibility to change the keyboard layout where it's mentioned anyways then I don't have to look anywhere else.

@Sogolumbo Sogolumbo changed the title Add support for different keyboard layouts (language specific) Make keyboard layouts easier to find May 2, 2021
@Azorlogh
Copy link

Azorlogh commented May 2, 2021

The deeper issue here is that the keyboard handling code uses the KeyboardEvent.which property (which by the way is the obsolete, KeyboardEvent.key is the new name for it).
Instead, it should use the KeyboardEvent.code, which is independent of the layout.

@meszaros-lajos-gyorgy
Copy link
Collaborator

I guess it sounds logical to link here the other keyboard related ticket: #31

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

No branches or pull requests

3 participants