D29100: Icon: improve icon rendering on multi-screen multi-dpi setups

[Aleix Pol Gonzalez]

apol created this revision.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
apol requested review of this revision.

REVISION SUMMARY
  Prefer the usage of QIcon::pixmap(QWindow*...) overload that will take into account which window it's rendering to.
  Otherwise it was assuming we were rendering on the primary screen and it would end up being unpredictable.

TEST PLAN
  All looks correct, if I move emojier from a scaled screen to a non-scaled one, icons re-render properly.

REPOSITORY
  R169 Kirigami

BRANCH
  master

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

AFFECTED FILES
  src/icon.cpp

To: apol
Cc: plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200422/6e315e49/attachment.html>