<table><tr><td style="">mart added a subscriber: davidedmundson.<br />mart added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11308">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11308#226767" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11308#226767</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a>  You're essentially describing transforming the lock screen into a screensaver, with the default effect being an animated slideshow. I would favor this. Users like screensavers simply for the visual flair, and I've heard a lot of user feedback that they'd like something approximating a screensaver. However, that's a substantially different patch from this one, and we might run into memory pressure issues if we're not able to resolve <a href="https://bugs.kde.org/show_bug.cgi?id=368838" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=368838</a> beforehand.</p></div>
</blockquote>

<p>ok for pushing this, but with the idea to change to a slideshow in the future (to me with the default wallpaper looks less quality refined than now, but is just a personal opinion)</p>

<p>(btw that leak i think should be solved, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> also has a rewrite of the wallpaper code which should make it take less memory, as right now the first few times it changes memory gooes up not for an actual leak but more for the management not being correct)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R133 KScreenLocker</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11308">https://phabricator.kde.org/D11308</a></div></div><br /><div><strong>To: </strong>ngraham, Plasma, VDG, graesslin, abetts<br /><strong>Cc: </strong>davidedmundson, zzag, Pitel, progwolff, abetts, hein, mart, graesslin, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol<br /></div>