Skip to content

dbeaver/pro#10322 use svg as indicator to fix off center issue - #4601

Open
devnaumov wants to merge 1 commit into
develfrom
dbeaver/pro#10322-radio-off-center
Open

dbeaver/pro#10322 use svg as indicator to fix off center issue#4601
devnaumov wants to merge 1 commit into
develfrom
dbeaver/pro#10322-radio-off-center

Conversation

@devnaumov

@devnaumov devnaumov commented Aug 25, 2026

Copy link
Copy Markdown
Member

closes 10322

The issue occurred at page zoom levels below 100% because the outer ring and inner dot were rendered as separate CSS elements and rounded to physical pixels independently. At fractional zoom levels, this caused the dot to appear off-center. A radial-gradient fixed the alignment but reduced sharpness at the default zoom level.

The indicator now uses a single SVG where the ring and dot share the same viewBox and center. This keeps the indicator aligned at fractional zoom levels, preserves sharp rendering at 100%, and supports the component’s existing sizes.

@devnaumov
devnaumov requested a review from SychevAndrey August 25, 2026 13:42
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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.

1 participant