-
Notifications
You must be signed in to change notification settings - Fork 123
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
Remove bilinear filtering above certain zoom percentage #356
Comments
Great suggestion. |
Now I'm hyped for 6.0 :D |
4.0 is basically feature-frozen right now so I don't think I'll be able to do that. I am trying to set up releases to move quicker in the future by having a smaller scope and automating my build process. |
Sounds great Ben, no hassle 😄 |
Definitely interested in this. Would be excellent for pixel art! Either that or a toggle for certain image resolution -> screen zoom ratios. |
Would accept a PR for this, shouldn't be super difficult to do besides understanding where to put it IIRC |
@jurplel I came up with something: First I simplified the scaling settings, I find it more understandable this way and don't see the need for independent checkboxes for bilinear + expensive scaling (i.e. not sure why someone would want expensive scaling enabled without bilinear also enabled as a fallback when the image goes above the window size). Then I added the zoom percent threshold to disable smooth scaling. No manual toggle, it's strictly based on zoom level. Would that work for you? Just the general concept, wording aside since you could easily re-label things. |
Isn't using zoom percentage the wrong way to specify a threshold? Such a threshold will behave differently for different image resolutions, which I don't think is desirable behavior. I think, as I believe @Skullfurious suggests, the threshold should be a specific ratio between image pixels and physical screen pixels. e.g. "I want smooth scaling to stop once each image pixel takes up 16 screen pixels (a 4:1 ratio in each dimension)". Or perhaps expressed in terms of rendered DPI. This is not my field of expertise. This is my intuition speaking. |
Depends on what you consider to be 100% zoom. If 100% means zoom to fit window, it would be wrong. I am considering 100% to mean 1 image pixel = 1 screen pixel (mostly, it gets more complicated with high DPI scaling, and there is a "Zoom level is relative to screen pixels" setting to control what 100% means in that context in the |
ok, if that's the definition of 100% zoom, then the setting makes sense. But I think some systems take into account the DPI settings of both the image and monitor, such that 100% means display an inch (as defined by the image) as an inch on screen. There's also HDPI screen scaling to take into account. I should assume by your definition the 1:1 pixel ratio of 100% zoom is in terms of physical not virtual pixels, yes? |
On Windows/Linux, assuming the On macOS, there kind of isn't a concept of physical pixels due to the way it handles high DPI scaling. |
The issue with My thoughts are:
I haven't had the time to dig through it, unfortunately (and I don't see any point in the near future where I will have the time). |
Having the filtering disabled above 200%-300% (could be a field in which the user can enter any percentage they want) would make viewing pixel art much better, without having to disable and re-enable it manually in the settings.
Also, a hotkey to enable/disable the filtering could be a temporary alternative.
A setting that snaps zooming to increments of 100% above said percentage would be awesome too, to avoid mangled pixels.
The text was updated successfully, but these errors were encountered: