Skip to content
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.

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

Workaround for egui having wrong scale in firefox #5466

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

lucasmerlin
Copy link
Collaborator

@lucasmerlin lucasmerlin commented Dec 12, 2024

  • I have followed the instructions in the PR template

@lucasmerlin lucasmerlin added web Related to running Egui on the web eframe Relates to epi and eframe labels Dec 12, 2024
Copy link

Preview available at https://egui-pr-preview.github.io/pr/5466-lucasdevice-pixel-ratio-workaround
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin lucasmerlin force-pushed the lucas/device-pixel-ratio-workaround branch from 3941bd2 to bb0d600 Compare December 12, 2024 17:26
@emilk emilk requested a review from jprochazk December 16, 2024 08:20
@lucasmerlin lucasmerlin merged commit 3bdb19e into master Dec 16, 2024
46 checks passed
@lucasmerlin lucasmerlin deleted the lucas/device-pixel-ratio-workaround branch December 16, 2024 10:38
@emilk
Copy link
Owner

emilk commented Dec 16, 2024

https://www.egui.rs/#rendering is now failing on chromium

image

Text is blurry, lines are not pixel-aligned

@jprochazk
Copy link
Collaborator

jprochazk commented Dec 16, 2024

It looks fine on Linux Chromium:
image

@emilk
Copy link
Owner

emilk commented Dec 16, 2024

Even when you zoom? ctrl+ etc

also… that image does not look very crisp, but maybe that's just the screenshot tool?

emilk added a commit that referenced this pull request Dec 16, 2024
emilk added a commit that referenced this pull request Dec 16, 2024
* Reverts #5466

Restores pixel-perfect rendering on chromium, Mac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eframe Relates to epi and eframe exclude from changelog web Related to running Egui on the web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants