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

Feature request: High DPI compatibility #21

Open
ZeyuSun opened this issue Oct 19, 2020 · 6 comments
Open

Feature request: High DPI compatibility #21

ZeyuSun opened this issue Oct 19, 2020 · 6 comments
Assignees
Labels
🐞 bug Something isn't working 🦌 enhancement Improve existing feature 🏭 3.x.x To implement in v3.0.0 and later

Comments

@ZeyuSun
Copy link

ZeyuSun commented Oct 19, 2020

The icons and fonts are too small on a high DPI screen. This post suggests the reason might be the program somehow thinks it's high DPI aware. By overriding the high DPI scaling behavior, I get normal-sized icons and fonts, but they become blurry as shown in the screenshot below (click the picture to open it in a new tab to see the difference).

image

I think it would be nice to make the app compatible with high DPI display.

@Beelink Beelink added the 🦌 enhancement Improve existing feature label Oct 20, 2020
@Beelink
Copy link
Member

Beelink commented Oct 20, 2020

Not sure if it's possible because the WinForms library has bad scaling support. But I will try to implement it.

@Beelink Beelink self-assigned this Oct 27, 2020
Beelink added a commit that referenced this issue Nov 1, 2020
Fixed:
- Plugin manager: Resizable columns
- EscToExit option is working in fullscreen mode
- Wrong restoring behavior of maximized window

Changes:
- Better support for Windows display scaling (#21)
@chisli
Copy link

chisli commented Nov 10, 2020

And when one change the DPI scaling behavior then the screenshot does not work anymore as it supposed to be, it doesn't capture the whole screen.

2020-11-10 12_05_21-Übersetzer

@Beelink Beelink added 🏭 3.x.x To implement in v3.0.0 and later 🐞 bug Something isn't working labels Nov 10, 2020
@Beelink
Copy link
Member

Beelink commented Feb 17, 2021

And when one change the DPI scaling behavior then the screenshot does not work anymore as it supposed to be, it doesn't capture the whole screen.

This issue seems to be fixed. Can you install the latest version of the app and check it?
Here's a link to the latest version: https://github.com/ModuleArt/quick-picture-viewer/raw/master/inno-setup/beta/v3.0.3-beta4.exe

P.S.
The screenshot feature moved to this menu:

image

@szywise
Copy link

szywise commented Feb 17, 2021

I am using a high DPI monitor with 200% scaling so things don't look too small. This is what I see
image

Everything looks sharp and clean. But the menu icons are too small. I wonder if there is a way to make the icons also scale as the fonts do.

@chisli
Copy link

chisli commented Feb 18, 2021

The problem is not really solved. I'm using a Surface Book 2 with a high resolution screen. So that I can use the program correctly, I have to change the resolution (DPI settings). With these settings the screenshot take only a part of the screen.
QPV_0
QPV_1

QPV_0.png => original settings, icons are too small
QPV_1.png => DPI setting for normal use but problems with screenshot

@Beelink
Copy link
Member

Beelink commented Feb 18, 2021

Thank you for this info. I will try to check everything again.

Beelink added a commit that referenced this issue Mar 22, 2021
- Better HiDPI support (#21)
- Fix: Blurry icons in context menus
@Beelink Beelink pinned this issue Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working 🦌 enhancement Improve existing feature 🏭 3.x.x To implement in v3.0.0 and later
Projects
None yet
Development

No branches or pull requests

4 participants