-
Notifications
You must be signed in to change notification settings - Fork 183
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
Use messagease symbol layout for messagease number board #695
Conversation
app/src/main/java/com/dessalines/thumbkey/keyboards/ENMessageEaseSymbols.kt
Outdated
Show resolved
Hide resolved
Better match the [messagease layout](https://www.exideas.com/ME/iosfaq/numbers.png) Small variation from the real ME in that blank spaces have been filled with some of the extra math symbols that ME has on swipe-return keys, which we can't have on TK
I'll get this one merged before working on |
See previous commit for image evidence
a75c780
to
ce8c62a
Compare
rebased to main and addressed review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to run ./gradlew formatKotlin
to pass lint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are there changes to this file, it was already added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mentioned in my comment above, it didn't match the actual ME layout, + the extra symbols my version wanted didn't exist, so those have been added.
You can compare the image I linked if you care about the fidelity to messagease.
Otherwise if you'd prefer existing layouts remain set in stone, I'll create a variant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah okay. We can always make corrections later anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx!
Replace the thumb-key numbers with messagease numbers (difference is in how symbols are positioned.) Better matches the messagease layout
Small variation from the real ME in that blank spaces have been filled with some of the extra math symbols that ME has on swipe-return keys, which we can't have on TK