D13907: [SDDM Theme] Set sourceSize on background image
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Jul 6 07:55:35 BST 2018
broulik created this revision.
broulik added reviewers: Plasma, davidedmundson.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
I noticed that loading the background was a significant portion of sddm's startup time and found that we didn't set a `sourceSize` and just load a `3200x2000` image
TEST PLAN
Verified that the image isn't loaded multiple times due to changing sizes or property re-evaluation
Had to change anchors to manual width as anchors seem to set geometry separately in a later step where the image already started loading, with explicit width/height Image waits until they're set, so we load the image only once
Didn't really notice any impact on my machine but with 4K screen I would have loaded the full image anyway but might be noticeable on FHD or small laptops
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D13907
AFFECTED FILES
sddm-theme/Background.qml
To: broulik, #plasma, davidedmundson
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180706/50eaf813/attachment-0001.html>
More information about the Plasma-devel
mailing list