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

Emojis need to press backspace twice to remove #1168

Closed
6 of 7 tasks
pxeemo opened this issue Nov 30, 2024 · 4 comments · Fixed by #1169
Closed
6 of 7 tasks

Emojis need to press backspace twice to remove #1168

pxeemo opened this issue Nov 30, 2024 · 4 comments · Fixed by #1169
Labels
bug Something isn't working

Comments

@pxeemo
Copy link
Contributor

pxeemo commented Nov 30, 2024

Steps to reproduce

  1. Type a emoji
  2. Press the backspace key

Expected behavior

Emojis should remove completely when backspace is pressed

Actual behavior

You'll see emojis become like this when you press backspace key: ���

version of the program

4.0.6

Android version

Android 14

Device

POCO M3

Other details

This commit #1162 changes the backspace behavior to remove only one unicode character, since emojis are two-width unicode character they won't be removed as normal

Acknowledgements

  • I have written a short but informative title.
  • I have updated the app to the latest version.
  • I have searched the existing issues and this is a new one, NOT a duplicate or related to another open issue.
  • This is not a question or a discussion, in which case I should have gone to lemmy.ml/c/thumbkey
  • This is a single bug report, in case of multiple bugs I will open a separate issue for each one (they can always link to each other if related)
  • I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements. 🤡
  • I have filled out all of the requested information in this form.
@pxeemo pxeemo added the bug Something isn't working label Nov 30, 2024
@dessalines
Copy link
Owner

This was caused by #1162

@gitterrost4 All the other keyboards must have this issue then also, unless they have exceptions for certain characters.

@gitterrost4
Copy link
Contributor

gitterrost4 commented Nov 30, 2024 via email

@gitterrost4
Copy link
Contributor

I have it fixed. My bad. I didn't think about emoji. In later API-Versions this code could be simplified by using deleteSurroundingTextInCodePoints, but in the minimum required API, this is not yet present.

@KraXen72
Copy link
Contributor

KraXen72 commented Dec 1, 2024

experiencing the same issue. hopefully a fix can be pushed soon. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants