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

Cursor selection no longer appears over whitespace #14713

Closed
Lord-McSweeney opened this issue Jan 12, 2024 · 1 comment · Fixed by #18437
Closed

Cursor selection no longer appears over whitespace #14713

Lord-McSweeney opened this issue Jan 12, 2024 · 1 comment · Fixed by #18437
Labels
bug Something isn't working regression Something was working before, but is now broken text Issues relating to text rendering/input

Comments

@Lord-McSweeney
Copy link
Collaborator

Describe the bug

Whitespace in the selection is never rendered. For example, if the text "Hello world" is selected, "Hello" and "world" will have an opaque black rectangle rendered under the text, but the space will not have it. I suspect this is due to the device font changes (not #14690), but I'm not sure.

Expected behavior

The selection is rendered on whitespace.

Content Location

Any basic reproduction works

Affected platform

Desktop app

Operating system

Ubuntu 20.04

Browser

Irrelevant

Additional information

No response

@Lord-McSweeney Lord-McSweeney added bug Something isn't working text Issues relating to text rendering/input regression Something was working before, but is now broken labels Jan 12, 2024
@kjarosh
Copy link
Member

kjarosh commented Jan 12, 2024

I ran bisect over this, which reports 4db87a9 as the culprit (which appears to be from #14535). Also it does not always happen, I guess for some fonts only (perhaps the default ones?).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression Something was working before, but is now broken text Issues relating to text rendering/input
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants