D23677: [Folder View] Load icon only for isPopup case

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Sep 2 13:34:33 BST 2019


broulik created this revision.
broulik added reviewers: Plasma, hein.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  There's no point in loading the icon when it cannot be seen.

TEST PLAN
  No longer have desktop containment query the icon name of the root item

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/package/contents/ui/main.qml

To: broulik, #plasma, hein
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190902/90eed5b6/attachment-0001.html>


More information about the Plasma-devel mailing list