Bug 407726 - Button icons blurry on HiDPI
Summary: Button icons blurry on HiDPI
Status: RESOLVED FIXED
Alias: None
Product: yakuake
Classification: Applications
Component: general (show other bugs)
Version: 3.0.5
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: markuss
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-05-19 13:47 UTC by markuss
Modified: 2019-11-21 19:44 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 19.12.0


Attachments
Screenshot (48.33 KB, image/png)
2019-05-19 13:47 UTC, markuss
Details

Note You need to log in before you can comment on or make changes to this bug.
Description markuss 2019-05-19 13:47:14 UTC
Created attachment 120181 [details]
Screenshot

Both the title bar buttons as well as OK/Cancel buttons in preferences are blurry on HiDPI.

See attached screenshot.
Comment 1 markuss 2019-05-19 13:48:19 UTC
I hope assigning this bug to Nate is OK. He's done much HiDPI work as far as I can see.
Comment 2 Nate Graham 2019-05-19 14:21:38 UTC
Nope, I don't work on Yakuake, sorry. However seeing as you've taken an interest, why don't we assign it to you? :) You can easily try out the typical fix yourself; it's a simple one-line change and it's very easy. Add "app.setAttribute(Qt::AA_UseHighDpiPixmaps, true);" right below here https://cgit.kde.org/yakuake.git/tree/app/main.cpp#n33

Then re-compile Sakuake, run it, and see if that helps. This is all described here: https://community.kde.org/Get_Involved/development#Applications

Let me know if you need a hand with anything.
Comment 3 markuss 2019-05-19 21:52:28 UTC
(In reply to Nate Graham from comment #2)

> Let me know if you need a hand with anything.

Move Yakuake to GitLab, so I can have a nice web-based editor than to fight with Git for a single line. :p
Comment 4 Patrick Silva 2019-05-19 22:52:25 UTC

*** This bug has been marked as a duplicate of bug 397590 ***
Comment 5 markuss 2019-05-20 15:53:49 UTC
(In reply to Nate Graham from comment #2)

> Let me know if you need a hand with anything.

You could merge https://phabricator.kde.org/D19237 if nobody else does.
Comment 6 markuss 2019-06-15 19:50:01 UTC
Not a dupe. The other bug is about the title bar buttons (those have been fixed). This is about OK/Cancel buttons in preferences.
Comment 7 Nicolas 2019-10-21 06:22:55 UTC
The issue is still present. I've built locally and adding QGuiApplication::setAttribute(Qt::AA_UseHighDpiPixmaps, true);

just after main() helps indeed.
Comment 8 Nicolas 2019-10-29 08:25:14 UTC
I've opened a change request on phabricator: maybe someone is interested (and has the rights) to merge it: https://phabricator.kde.org/D25038
Comment 9 Patrick Silva 2019-10-29 12:24:50 UTC
(In reply to Nicolas from comment #8)
> I've opened a change request on phabricator: maybe someone is interested
> (and has the rights) to merge it: https://phabricator.kde.org/D25038

I has just applied your patch on my system. It fixes this issue.
Comment 10 Nate Graham 2019-11-21 19:43:57 UTC
Git commit 0134f68c906957be49fa8c14fa34ef335db1165b by Nate Graham, on behalf of Nicolas Höft.
Committed on 21/11/2019 at 19:42.
Pushed by ngraham into branch 'release/19.12'.

Enable high DPI pixmaps

Summary:
Enable High DPI pixmaps to make icons in dialogs looks nice
if High DPI is enabled.

Test Plan: Checked manually in high DPI environment.

Reviewers: hein, cfeck, ngraham

Reviewed By: cfeck, ngraham

Subscribers: ngraham

Differential Revision: https://phabricator.kde.org/D25038

M  +1    -0    app/main.cpp

https://commits.kde.org/yakuake/0134f68c906957be49fa8c14fa34ef335db1165b