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

Reply via email to