Skip to content

Fix last character disappearance if emoji are in comment#5587

Open
shiny-comic wants to merge 1 commit intoiv-org:masterfrom
shiny-comic:fix-comment-with-emoji
Open

Fix last character disappearance if emoji are in comment#5587
shiny-comic wants to merge 1 commit intoiv-org:masterfrom
shiny-comic:fix-comment-with-emoji

Conversation

@shiny-comic
Copy link
Contributor

This closes #5503

This comment was marked as off-topic.

Copy link
Member

@Fijxu Fijxu left a comment

Choose a reason for hiding this comment

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

Opps

@Fijxu Fijxu requested a review from unixfox February 23, 2026 21:44
@Fijxu Fijxu added the ready label Feb 24, 2026
@shiny-comic shiny-comic force-pushed the fix-comment-with-emoji branch 2 times, most recently from a96de72 to 490a0b7 Compare February 26, 2026 22:03
Previous code use UTF-8 to count characters however Emojis are UTF-16 units.
This difference leads to misalignment of index offsets.
@shiny-comic shiny-comic force-pushed the fix-comment-with-emoji branch from 490a0b7 to 9209b21 Compare February 26, 2026 22:07
Copy link
Member

@SamantazFox SamantazFox left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] The last character of comments disappear if emoji are included.

4 participants