Skip to content

Always centered radio marks#24749

Open
ickshonpe wants to merge 1 commit into
bevyengine:mainfrom
ickshonpe:always-centered-radio-mark
Open

Always centered radio marks#24749
ickshonpe wants to merge 1 commit into
bevyengine:mainfrom
ickshonpe:always-centered-radio-mark

Conversation

@ickshonpe

Copy link
Copy Markdown
Contributor

Objective

At certain scale factors due to coordinate rounding radio button marks aren't centered:

offcenterradio

Solution

Disable layout rounding for radio button marks so they are always centered. To mitigate underdraw problems the size of mark node is increased by 4 pixels and then given a transparent border so the filled circle area remains the same diameter.

radio_b

…tered. To mitigate underdraw problems the size of mark node is increased by 4 pixels and then given a transparent border so the filled circle area remains the same diameter.
@ickshonpe ickshonpe added C-Bug An unexpected or incorrect behavior A-UI Graphical user interfaces, styles, layouts, and widgets D-Straightforward Simple bug fixes and API improvements, docs, test and examples labels Jun 25, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in UI Jun 25, 2026
@ickshonpe ickshonpe added the S-Needs-Review Needs reviewer attention (from anyone!) to move forward label Jun 25, 2026
@kfc35 kfc35 added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 26, 2026
@ickshonpe ickshonpe added this to the 0.19.1 milestone Jun 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Needs SME Triage

Development

Successfully merging this pull request may close these issues.

4 participants