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

Hungarian, Italian, Portuguese and Turkish layouts #143

Merged
merged 12 commits into from
Apr 5, 2024

Conversation

allthelazorz
Copy link
Contributor

@allthelazorz allthelazorz commented Apr 5, 2024

Closed the previous pull requests, because their ordering in Settings.kt would have required manual conflict resolution. This request could be automatically merged.

Layouts in request:

Hungarian

  • official MessagEase
  • by Dániel Tenke
  • by Máté Farkas
  • U always up

Italian
Portuguese

  • official MessagEase on Android
  • iOS

Turkish

Copy link
Owner

@nightkr nightkr left a comment

Choose a reason for hiding this comment

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

Thanks for this! Just a few nits but nothing major.

@@ -0,0 +1,115 @@
package se.nullable.flickboard.model.layouts
Copy link
Owner

Choose a reason for hiding this comment

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

Please add a comment to the file explaining what DT means.

@@ -0,0 +1,115 @@
package se.nullable.flickboard.model.layouts
Copy link
Owner

Choose a reason for hiding this comment

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

Same here for MF.

@@ -0,0 +1,115 @@
package se.nullable.flickboard.model.layouts
Copy link
Owner

Choose a reason for hiding this comment

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

We don't differentiate the others on platform, I think it's fair to call this one just PTMessagEase (and treat iOS as the special case worth calling out).

HungarianMF("Hungarian (MessagEase, by Máté Farkas)", HU_MF_MESSAGEASE),
HungarianUUp("Hungarian (MessagEase, U always up)", HU_UUP_MESSAGEASE),
Italian("Italian (MessagEase)", IT_MESSAGEASE),
PortugueseAnd("Portuguese (MessagEase, Android)", PT_AND_MESSAGEASE),
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
PortugueseAnd("Portuguese (MessagEase, Android)", PT_AND_MESSAGEASE),
Portuguese("Portuguese (MessagEase)", PT_AND_MESSAGEASE),

@allthelazorz
Copy link
Contributor Author

Thank you for the review! I have made the changes.

Copy link
Owner

@nightkr nightkr left a comment

Choose a reason for hiding this comment

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

Thanks

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

2 participants