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

[AttributedText] - Fix incorrect character lookup by index (Resolves #2531) #2532

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

matthew-carroll
Copy link
Contributor

[AttributedText] - Fix incorrect character lookup by index (Resolves #2531)

We need to lookup text characters while accounting for placeholders. Previously were were looking up characters in the plain text string.

Copy link
Collaborator

@angelosilvestre angelosilvestre left a comment

Choose a reason for hiding this comment

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

LGTM

@matthew-carroll matthew-carroll merged commit 0f7044d into main Jan 31, 2025
16 checks passed
@matthew-carroll matthew-carroll deleted the 2531_attributed-text-fix-character-lookup branch January 31, 2025 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] - [AttributedText] - Character lookup logic is wrong
2 participants