-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Set default pixel density to the display's density #950
Comments
I can work on this issue, please assign it to me. I will follow up with a proposed solution here as well since this is the first time for me to contribute to this repo. |
Just to make sure that I'm on the right track for solving this issue, I've tried setting the That said, whenever I try to set the pixel density to 1 using I know that the DemoNormal ScalingNormal.Scaling.Non-HiDPI.mode.mp4Custom scaling (Hi-DPI)High.Scaling.HiDPI.mode.mp4 |
Hi @yehiarasheed Sorry for not replying earlier, This change is best to be as minimal as possible, if you find other issues on the way, please create separate PR's for those or just document them through a separate issue. |
Hi @Stefterv, no worries! I now understand that you're aiming for a minimal enhancement—I initially thought Jim's comments reflected the entire team's vision. |
Most relevant area for this enhancement?
Core/Environment/Rendering
Feature to enhance
pixelDensity()
Desired enhancement
Currently, Processing defaults to
pixelDensity(1)
, requiring users to manually setpixelDensity(2)
orpixelDensity(displayDensity())
for high-DPI support. This proposal sets the default to the display's actual density, improving visual quality by default while allowing users to override it.Potential benefits
Sharper rendering on high-DPI displays.
Possible drawbacks
No response
Additional context
No response
Would you like to work on the issue?
This is a small change which would make a good first issue. I'm happy to leave it to a new contributor.
The text was updated successfully, but these errors were encountered: