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

Keyboard Multi Layout support #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Keyboard Multi Layout support #10

wants to merge 3 commits into from

Conversation

joelsernamoreno
Copy link
Contributor

Hi!

I added multi layout support

Now you can use en_us, es_es, it_it layouts

I will try to add more layouts when I have free time

To use this: read the Keyboard multi layout support section in README.md

@chegewara
Copy link
Owner

Thanks for this PR, but i would suggest to change code the way the users wont need to change library to use it.

@joelsernamoreno
Copy link
Contributor Author

I think this is a little difficult

If you have any idea how to do this, I can try this

@chegewara
Copy link
Owner

chegewara commented Jan 16, 2021

There is few ways. One is to include all keymap arrays and add function that will assign one when language/layout is selected with separate function. This way is not the best option, because will include all arrays and increase binary by 1-2kB, depending on layouts amount. But this is some start.

Another option may be to add extern KEYMAP[] in hidkeyboard.h and user will have to add #include layoutxxx.h in their code.

@joelsernamoreno
Copy link
Contributor Author

Hi

I have seen that you have added SD Mass Storage

I tried to test this but when I compile the example your library gives many errors.

Now I can't use the keyboard example either

I attach screenshots with the errors:

Selección_049

Selección_050

Selección_051

Selección_052

Selección_053

@chegewara
Copy link
Owner

@joelsernamoreno Please open new issue so we can track it.

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