Skip to content

FontSmoothing fixes #22455

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.

Sign up for GitHub

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

Jump to bottom
Open
ickshonpe wants to merge 2 commits into
base: main
Choose a base branch
from
Open

FontSmoothing fixes #22455

ickshonpe wants to merge 2 commits into from

Conversation

@ickshonpe
Copy link
Contributor

@ickshonpe ickshonpe commented Jan 9, 2026 •
edited
Loading

Objective

  1. The font_smoothing field of TextFont is ignored in main currently. Instead it always uses FontSmoothing::Antialiased.
  2. The FontSmoothing enum is missing from the bevy_text prelude.
  3. The FontSmoothing bug wasn't caught by the screenshot CI.

Solution

  1. Store the antialiasing mode in ComputedTextBlock, for use in TextPipeline::update_buffer.
  2. Add FontSmoothing to bevy_text::prelude.
  3. Add a font smoothing example to testbed_ui's Text scene.

Testing

Check testbed_ui's text scene.

@ickshonpe
1. Bug fix: TextFont's font_smoothing field was being ignored, `F…
1016b67 
…ontSmoothing::Antialiased` was always used. To fix this, store the antialiasing mode in `ComputedTextBlock`, for use in `TextPipeline::update_buffer`.

2. Added the `FontSmoothing` enum to `bevy_text::prelude`.

3. Added a font smoothing example to `testbed_ui`'s `Text` scene.
@ickshonpe ickshonpe added C-Bug An unexpected or incorrect behavior A-Text Rendering and layout for characters D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered labels Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kfc35 kfc35 kfc35 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-Text Rendering and layout for characters C-Bug An unexpected or incorrect behavior D-Straightforward Simple bug fixes and API improvements, docs, test and examples M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ickshonpe @kfc35