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

core: Render EditText's selection when filters are present #16422

Merged
merged 2 commits into from
May 24, 2024

Conversation

kjarosh
Copy link
Member

@kjarosh kjarosh commented May 22, 2024

This patch ensures that the selection in EditText is rendered even when filters are present (so that cache as bitmap is enabled). This is done by invalidating the cached bitmap on selection change.

Fixes #16374.

After this PR the caret is rendered but does not blink (due to the cached bitmap).

@kjarosh kjarosh force-pushed the edittext-select-filters branch 2 times, most recently from 763aaf6 to 8f44efb Compare May 22, 2024 22:10
@danielhjacobs danielhjacobs added text Issues relating to text rendering/input waiting-on-review Waiting on review from a Ruffle team member labels May 23, 2024
kjarosh added 2 commits May 24, 2024 23:40
This patch ensures that the selection in EditText is rendered even when
filters are present (so that cache as bitmap is enabled).
This is done by invalidating the cached bitmap on selection change.
This test verifies that selection is rendered when filters are present.
@Dinnerbone Dinnerbone force-pushed the edittext-select-filters branch from 8f44efb to 20ddfac Compare May 24, 2024 21:40
@Dinnerbone Dinnerbone enabled auto-merge (rebase) May 24, 2024 21:41
@Dinnerbone Dinnerbone merged commit 3aa7c07 into ruffle-rs:master May 24, 2024
17 checks passed
@danielhjacobs danielhjacobs removed the waiting-on-review Waiting on review from a Ruffle team member label May 24, 2024
@kjarosh kjarosh deleted the edittext-select-filters branch May 24, 2024 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
text Issues relating to text rendering/input
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

The Clockmaker: Broken Copy paste
3 participants