Skip to content

gpui: Fix a bug with Japanese romaji typing in input example #28507

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

Merged

Conversation

SergeevPavel
Copy link
Contributor

Steps to reproduce:

  • On macOS, run input example
  • type aaa|bbb place caret on the place marked with |
  • switch to japanese romaji
  • press ko
  • press left arrow
image

You will get aaa duplicated with every arrow press.

According to reference implementation we need to unmark text when we get empty line in setMarkedText

Copy link

cla-bot bot commented Apr 10, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @SergeevPavel on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@SergeevPavel SergeevPavel changed the title fix a bug with japanese romaji typing input example: fix a bug with japanese romaji typing Apr 10, 2025
@maxdeviant maxdeviant changed the title input example: fix a bug with japanese romaji typing gpui: Fix a bug with Japanese romaji typing in input example Apr 10, 2025
@SergeevPavel
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Apr 14, 2025
Copy link

cla-bot bot commented Apr 14, 2025

The cla-bot has been summoned, and re-checked this pull request!

@zed-industries-bot
Copy link

Warnings
⚠️

This PR is missing release notes.

Please add a "Release Notes" section that describes the change:

Release Notes:

- Added/Fixed/Improved ...

If your change is not user-facing, you can use "N/A" for the entry:

Release Notes:

- N/A

Generated by 🚫 dangerJS against f0a8f0c

Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore left a comment

Choose a reason for hiding this comment

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

Thanks!

@SomeoneToIgnore SomeoneToIgnore self-assigned this May 5, 2025
@SomeoneToIgnore SomeoneToIgnore merged commit 6e28400 into zed-industries:main May 5, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants