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

UI is blurry on HiDPI screens on MacOS (retina) since 1.2.x #355

Closed
antwan opened this issue Feb 19, 2018 · 8 comments
Closed

UI is blurry on HiDPI screens on MacOS (retina) since 1.2.x #355

antwan opened this issue Feb 19, 2018 · 8 comments

Comments

@antwan
Copy link

antwan commented Feb 19, 2018

Steps to reproduce: Open the app, it's blurry.

Don't know if the dmg was built with a wrong version of QT, but it looks like a massive regression as soon as you open it on a hi-dpi machine (most of them are now).

@lukedirtwalker
Copy link
Contributor

Hi @Antwan86 can you please add a screenshot to see the problem? Did version 1.2.0 not have this problem?

@antwan
Copy link
Author

antwan commented Mar 19, 2018

Before (1.1.x)
1 1 6

After (1.2.x)
1 1 6

@lukedirtwalker
Copy link
Contributor

Hm 1.1.6 dmg file includes a Info.plist file and in 1.2.1 it is missing. So I'm pretty sure this causes the problem. From this page: http://doc.qt.io/qt-5/highdpi.html I would guess the missing part is the NSPrincipalClass.

I'm not sure where you would need to put the Info.plist file (no Mac user), but judging from 1.1.6 its content should be something like this:

<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
        <key>CFBundleDisplayName</key>
        <string>QtPass 1.2.1</string>
        <key>CFBundleExecutable</key>
        <string>QtPass</string>
        <key>CFBundleGetInfoString</key>
        <string>1.2.1</string>
        <key>CFBundleIconFile</key>
        <string>icon.icns</string>
        <key>CFBundleIdentifier</key>
        <string>$(PRODUCT_BUNDLE_IDENTIFIER)</string>
        <key>CFBundlePackageType</key>
        <string>APPL</string>
        <key>CFBundleShortVersionString</key>
        <string>1.2</string>
        <key>CFBundleSignature</key>
        <string>????</string>
        <key>NOTE</key>
        <string>QtPass is a multi-platform GUI for pass</string>
        <key>NSHumanReadableCopyright</key>
        <string>Copyright © 2014-2018 IJhack
This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.</string>
        <key>NSPrincipalClass</key>
        <string>NSApplication</string>
</dict>
</plist>

@cgrinds
Copy link

cgrinds commented Apr 1, 2018

Thanks, @lukedirtwalker. I can confirm that fixes the bluriness when added to /Applications/QtPass.app/Contents/Info.plist

@annejan
Copy link
Member

annejan commented Apr 29, 2018

This should be fixed in 1caf6d7

@antwan
Copy link
Author

antwan commented May 21, 2018

1 1 6

Icons still not fixed on 1.2.2 which includes 1caf6d7

@pheraph
Copy link

pheraph commented Nov 6, 2019

Icons are still blurry on macOS using v1.3.2. Please reopen the issue.

@annejan annejan reopened this Nov 6, 2019
@annejan
Copy link
Member

annejan commented May 30, 2020

Fixed in upcoming 1.3.3 release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants