Skip to content

Sharper icons on low-dpi screens (which is most of them)#10508

Merged
Wumpf merged 2 commits into
mainfrom
emilk/sharper-icons
Jul 3, 2025
Merged

Sharper icons on low-dpi screens (which is most of them)#10508
Wumpf merged 2 commits into
mainfrom
emilk/sharper-icons

Conversation

@emilk
Copy link
Copy Markdown
Member

@emilk emilk commented Jul 3, 2025

I went in and pixel-aligned vertical and horizontal lines in some of our most prominent icons.

The result is a lot sharper icons on low-DPI screens (which is most screens). The difference is pretty huge, especially in light mode.

Before/after

Screenshot 2025-07-03 at 16 42 00 Screenshot 2025-07-03 at 16 43 38

Screenshot 2025-07-03 at 16 42 07 Screenshot 2025-07-03 at 16 43 45

Screenshot 2025-07-03 at 16 42 17 Screenshot 2025-07-03 at 16 43 55

Screenshot 2025-07-03 at 16 50 57 Screenshot 2025-07-03 at 16 51 06

image image

@emilk emilk added 📺 re_viewer affects re_viewer itself do-not-merge Do not merge this PR labels Jul 3, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 3, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
073fdcd https://rerun.io/viewer/pr/10508 +nightly +main

Note: This comment is updated whenever you push a commit.

@emilk emilk marked this pull request as ready for review July 3, 2025 14:44
@emilk emilk force-pushed the emilk/sharper-icons branch from dc57d2f to 954b1a2 Compare July 3, 2025 14:44
Base automatically changed from emilk/viewport-improvements to main July 3, 2025 16:05
@emilk emilk removed the do-not-merge Do not merge this PR label Jul 3, 2025
@emilk emilk force-pushed the emilk/sharper-icons branch from 954b1a2 to 073fdcd Compare July 3, 2025 16:10
Copy link
Copy Markdown
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noice

@Wumpf Wumpf merged commit 608e79a into main Jul 3, 2025
40 checks passed
@Wumpf Wumpf deleted the emilk/sharper-icons branch July 3, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants