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

wayland: text input protocol uses UTF-8 encoding #415

Merged
merged 2 commits into from
May 8, 2024

Conversation

surban
Copy link
Contributor

@surban surban commented May 8, 2024

The Wayland text input protocol sends text in
UTF-8 encoding, but the Flutter TextInputModel
expects UTF-16.

This adds the missing conversion from UTF-8 to
UTF-16.

Fixes #414.

The Wayland text input protocol sends text in
UTF-8 encoding, but the Flutter TextInputModel
expects UTF-16.

This adds the missing conversion from UTF-8 to
UTF-16.

Signed-off-by: Sebastian Urban <surban@surban.net>
@HidenoriMatsubayashi
Copy link
Contributor

Thank you for sending this PR.

Could you please update AUTHORS too?

Also, it's unclear whether we can contribute this software to flutter/engine, but do you agree to delegate all rights related to this PR to Sony?

@HidenoriMatsubayashi HidenoriMatsubayashi added bug Something isn't working wayland Topics of Wayland on-screen keyboard Topics for on-screen keyboard labels May 8, 2024
@surban
Copy link
Contributor Author

surban commented May 8, 2024

Also, it's unclear whether we can contribute this software to flutter/engine, but do you agree to delegate all rights related to this PR to Sony?

Yes, I agree.

Copy link
Contributor

@HidenoriMatsubayashi HidenoriMatsubayashi left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you, Sebastian!

@HidenoriMatsubayashi HidenoriMatsubayashi merged commit f296db8 into sony:master May 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working on-screen keyboard Topics for on-screen keyboard wayland Topics of Wayland
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App crashes when non-ASCII characters are sent from input method
2 participants