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

added german keyboard layout #46

Closed
wants to merge 1 commit into from
Closed

added german keyboard layout #46

wants to merge 1 commit into from

Conversation

Henry-Jones
Copy link

No description provided.

Copy link
Collaborator

@evaherrada evaherrada left a comment

Choose a reason for hiding this comment

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

I think this is a good idea, but I also think that if it were added this way, then you'd have to update both the US and German layout when you want to update the files which isn't ideal. Instead, I think it could be good to have a super class that has the methods for doing everything with the keycodes, and then a class for each layout so it can be imported easily. Let me know if you're up to doing this, otherwise, I'd be more than glad to work on it

@Henry-Jones
Copy link
Author

Hello, I guess this is far beyond my knowledge of Phyton and the micro Phyton in particular. I can follow the idea technology wise but my pull request was not more than a quick "hack" from a usually C# and Powershell programmer. So go ahead and build it in a Python fitting fashion. ;)

@evaherrada
Copy link
Collaborator

@Henry-Jones Sure thing. I was actually just about to get started on that.

@dhalbert
Copy link
Collaborator

One issue is that the current HID is a very tight fit as a frozen library on SAMD21 boards right now; any new code (refactored or not) is probably going to overflow flash. We might consider adding keyboard layouts in separate libraries. A common superclass might be able to fit.

@kattni
Copy link
Contributor

kattni commented Aug 19, 2020

I agree with Dan. I'm closing this PR for now as we cannot add another keyboard layout reasonably in this fashion. Dan created an issue and linked it to this PR. Hopefully someone picks it up - Dylan is away for a bit and will not be able to pick it up anytime soon.

@Henry-Jones If you're interested in looking into the suggested solution, please let us know. It sounds like you're not as comfortable in Python as other languages, so I understand if you're not interested.

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.

None yet

4 participants