Skip to content

keyboardBegin.adoc: add Portuguese keyboard layout #913

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

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

edgar-bonet
Copy link
Contributor

Since commit cbee538 (pull request #77, merged on 2023-02-20), the Keyboard library supports the Portuguese keyboard layout. This pull request adds the layout to the documentation of Keyboard.begin().

This is a draft pull request. Since the latest release of the Keyboard library (1.0.4) does not include the Portuguese layout, this layout is only available on master. I will remove the “draft” status once the layout is available in a released version. Note that the commit message mentions release 1.0.5, which is the expected version number of the next release. If the version number turns out to be different (1.1.0 maybe?), I will amend the commit message.

@karlsoderby
Copy link
Collaborator

Hi @edgar-bonet , and apologies for the incredibly late response here. This looks great, and the 1.0.5 version was just released and your commit message is intact.

I saw in the Keyboard library that Hungarian support was added as well (arduino-libraries/Keyboard#84), so I made a suggestion that you also include this.

@edgar-bonet edgar-bonet marked this pull request as ready for review November 13, 2023 19:28
@edgar-bonet
Copy link
Contributor Author

Hi @karlsoderby, and thanks for the heads-up! I committed your suggestion, 👍 but then re-sorted the layouts to keep them in alphabetical order. I also removed the draft status, as it was meant to last only until release 1.0.5.

Should I squash the three commits together?

@karlsoderby
Copy link
Collaborator

Ah thank you, I did not consider alphabetical order. Yes please do so, and we can merge this PR! Thanks a lot!

Version 1.0.5 of the Keyboard library added the Portuguese and the
Hungarian keyboard layouts. Update the documentation of Keyboard.begin()
accordingly.

Co-authored-by: Karl Söderby <[email protected]>
@edgar-bonet
Copy link
Contributor Author

@karlsoderby: OK, done. I squashed the three commits together while rebasing on top of master.

@karlsoderby karlsoderby merged commit 0f26037 into arduino:master Nov 15, 2023
@karlsoderby
Copy link
Collaborator

Fantastic. Thank you @edgar-bonet !

@edgar-bonet edgar-bonet deleted the keyboard-pt branch November 15, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants