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