D6763: Fix app icon button in window decos with high DPI in previews.

Marco Martin noreply at phabricator.kde.org
Tue Jul 18 11:06:47 UTC 2017


mart accepted this revision.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> breezebutton.cpp:140
>              const QPixmap pixmap = decoration()->client().data()->icon().pixmap( m_iconSize );
> -            painter->drawPixmap(iconRect.center() - QPoint(pixmap.width()/2, pixmap.height()/2), pixmap);
> +            painter->drawPixmap(iconRect.center() - QPoint(pixmap.width()/2, pixmap.height()/2)/pixmap.devicePixelRatio(), pixmap);
>  

couldn't also need a check that the resulting ratio is at least 1 or more?

REPOSITORY
  R31 Breeze

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D6763

To: davidedmundson, #plasma, mart
Cc: mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170718/5bdfc367/attachment.html>


More information about the Plasma-devel mailing list