https://bugs.kde.org/show_bug.cgi?id=390639

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/bre
                   |                            |eze/c32eb41292752fd9a82fa25
                   |                            |8fb93f538f712cb91

--- Comment #10 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit c32eb41292752fd9a82fa258fb93f538f712cb91 by David Edmundson.
Committed on 21/02/2018 at 13:35.
Pushed by davidedmundson into branch 'Plasma/5.12'.

Render application icon at paintdevice DPR

Summary:
QIcon::pixmap doesn't know anything about the paint device currently
being used so uses qApp->devicePixelRatio.
This works in systemsettings, but not in kwin and we get a standard
resolution pixmap returned.

We can't manually multiply by painter->device->dpr because then system
settings would get scaled twice.

QIcon::paint solves that in a way that won't break should kwin's
qApp->dpr change. Also it's IMHO cleaner as it does the alignment
internally.

Test Plan:
System settings look the same
Icon in kwin deco is crisper than before

Reviewers: #plasma, mart

Reviewed By: #plasma, mart

Subscribers: plasma-devel

Tags: #plasma

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

M  +2    -2    kdecoration/breezebutton.cpp

https://commits.kde.org/breeze/c32eb41292752fd9a82fa258fb93f538f712cb91

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to