Skip to content

Enable anti-aliasing for dirty indicator rendering#3371

Open
HeikoKlare wants to merge 1 commit into
eclipse-platform:masterfrom
HeikoKlare:dirtyindicator-antialias
Open

Enable anti-aliasing for dirty indicator rendering#3371
HeikoKlare wants to merge 1 commit into
eclipse-platform:masterfrom
HeikoKlare:dirtyindicator-antialias

Conversation

@HeikoKlare

Copy link
Copy Markdown
Contributor

The dirty indicator for editor tabs does currently not use anti-aliasing which makes it look quite jagged. This change enables anti-aliasing for the rendering of the dirty indicator.

Left is current state, right is with this change:
image

The dirty indicator for editor tabs does currently not use anti-aliasing
which makes it look quite jagged. This change enables anti-aliasing for
the rendering of the dirty indicator.
@merks

merks commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

It's great to see such attention to fine detail. 👀 Kudos!

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Test Results

  182 files  +    1    182 suites  +1   25m 16s ⏱️ +9s
4 730 tests ±    0  4 707 ✅ ±    0   23 💤 ± 0  0 ❌ ±0 
6 854 runs  +1 056  6 691 ✅ +1 024  163 💤 +32  0 ❌ ±0 

Results for commit 72bba9e. ± Comparison against base commit bb8949f.

@vogella

vogella commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

The dirty indicator for editor tabs does currently not use anti-aliasing which makes it look quite jagged. This change enables anti-aliasing for the rendering of the dirty indicator.

Left is current state, right is with this change: image

Anything specific to configure to see this ugly indicator? Looks fine for me under Linux and Windows.

@Phillipus

Copy link
Copy Markdown
Contributor

Anything specific to configure to see this ugly indicator? Looks fine for me under Linux and Windows.

I can see the jaggies on Windows at 200% display scale. A screenshot and zooming in reveals all.

@vogella

vogella commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Change LGTM, thanks @HeikoKlare

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.

4 participants