Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Support Thai keyboard. #3544

Merged
merged 3 commits into from
Jun 29, 2020
Merged

Support Thai keyboard. #3544

merged 3 commits into from
Jun 29, 2020

Conversation

daoshengmu
Copy link
Contributor

@daoshengmu daoshengmu commented Jun 24, 2020

Fixes #3487.

I notice Android and iOS have different implementation when dealing vowels. Android can't display that little circle under a vowel like 'ึ`.

Besides, Android app will stack the coming vowels even though we can't see the stacked vowels in Android app (I believe they are cut-off), they indeed exist if we backspace them. We also can produce it when using Android Chrome and type a character with multiple vowels in the search bar.

So, I am looking for a way to stop showing the vowels that are out of vertical boundary. It might be an attribute in our EditText. @keianhzo do you have idea about this?

@daoshengmu daoshengmu requested a review from keianhzo June 24, 2020 20:34
@daoshengmu daoshengmu force-pushed the thaiKeyboard branch 2 times, most recently from bf77167 to 92f4c1c Compare June 24, 2020 21:28
@daoshengmu daoshengmu self-assigned this Jun 24, 2020
Copy link
Contributor

@keianhzo keianhzo left a comment

Choose a reason for hiding this comment

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

LGTM

@keianhzo
Copy link
Contributor

@daoshengmu waiting for the URL bar padding fix. Once it's added I'll merge.

@bluemarvin
Copy link
Contributor

@daoshengmu waiting for the URL bar padding fix. Once it's added I'll merge.

Is this a separate issue? I don't see it in the v11 milestone.

@daoshengmu
Copy link
Contributor Author

@daoshengmu waiting for the URL bar padding fix. Once it's added I'll merge.

Is this a separate issue? I don't see it in the v11 milestone.

Same here. I have fixed it in this PR.

@keianhzo keianhzo merged commit 93be5bc into master Jun 29, 2020
@keianhzo keianhzo deleted the thaiKeyboard branch June 29, 2020 16:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Thai keyboard support
3 participants