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

Add Armenian keyboard layouts #1654

Merged
merged 7 commits into from
Mar 24, 2022
Merged

Conversation

pjtsearch
Copy link
Contributor

@pjtsearch pjtsearch commented Mar 18, 2022

Thank you so much for your work on this project.

I've added the Armenian Dram currency set, layouts for Eastern and Western Armenian, Armenian popup mappings, and Eastern and Western Armenian subtype presets.

Closes #492

@patrickgold
Copy link
Member

Thanks for your PR, really appreciate it!

I've looked at the changes, and have found some issues. Will add a review comment to each of them so it is easier to reference.

Copy link
Member

@patrickgold patrickgold left a comment

Choose a reason for hiding this comment

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

These are the issues I found. If fixed, I will test out the layouts locally and if everything checks out merge them into master.

pjtsearch and others added 3 commits March 24, 2022 15:06
Co-authored-by: Patrick Goldinger <patrick@patrickgold.dev>
Co-authored-by: Patrick Goldinger <patrick@patrickgold.dev>
Co-authored-by: Patrick Goldinger <patrick@patrickgold.dev>
Copy link
Member

@patrickgold patrickgold left a comment

Choose a reason for hiding this comment

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

Thanks a lot for fixing the mentioned issues!

I just rechecked and tested your layouts and from a non-Armenian speaking person's view everything looks super clean in the UI!

@patrickgold
Copy link
Member

Will merge your changes now in, will be available in 0.3.15-beta01 tomorrow (Friday, may be today in your timezone already).

@patrickgold patrickgold merged commit 091d435 into florisboard:master Mar 24, 2022
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.

Add Armenian language support
2 participants