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

Portuguese azerty keyboard #2604

Merged
merged 4 commits into from Mar 21, 2024
Merged

Portuguese azerty keyboard #2604

merged 4 commits into from Mar 21, 2024

Conversation

mauriu
Copy link
Contributor

@mauriu mauriu commented Mar 17, 2024

This is a pull request for a Portuguese keyboard based on azerty layout. It was used before qwerty.
Its the same as current layout, with the following keys swapped:
a <-> q z <-> w ç <-> m
pt_azerty
image

Portuguese keyboard based on azerty layout.
@keyman-server
Copy link
Collaborator

Thank you for your pull request. The Keyman keyboard review team have been notified of your pull request and will review it and build it shortly.

@LornaSIL LornaSIL added this to In progress in Keyboards Mar 18, 2024
@LornaSIL
Copy link
Contributor

@mauriu thanks for the keyboard submission. It's in fairly good shape. A few issues:

  • The copyright is SIL. I you prefer, you can be the copyright holder. You would need to change that in all the locations.
  • The keyboard_info file needs all the "related" entries removed. This keyboard will not deprecate any of our existing keyboards.
  • The .kps file needs the pt language code added to the Details tab

@keyman-server
Copy link
Collaborator

Thank you for your pull request. The Keyman keyboard review team have been notified of your pull request and will review it and build it shortly.

Delete extra comma.
@keyman-server
Copy link
Collaborator

Thank you for your pull request. The Keyman keyboard review team have been notified of your pull request and will review it and build it shortly.

Adjust language setting.
Copy link
Contributor

@LornaSIL LornaSIL left a comment

Choose a reason for hiding this comment

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

LGTM with changes made.

@LornaSIL LornaSIL merged commit e1791fa into keymanapp:master Mar 21, 2024
2 checks passed
Keyboards automation moved this from In progress to Done Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Keyboards
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants