<table><tr><td style="">graesslin 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#226803" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11308#226803</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>Thanks <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>. I think using an animated slideshow by default will be really popular, especially if we do some work to add more nice images by default and polish up the slideshow plugin's UI a bit. I'm glad to hear that the memory issue is or should be resolved soon.</p>

<p><a href="https://phabricator.kde.org/p/graesslin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@graesslin</a>, have we satisfactorily resolved your objections yet? It not, what more needs to be done here?</p></div>
</blockquote>

<p>Please be aware of two things:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Most distros only install one wallpaper by default. If we go this route we must reach out to distros before and check whether they are able to install more default wallpapers</li>
<li class="remarkup-list-item">If you talk about this please never, never,ever call it a screen saver. We don't have to save screens, we don't have crts anymore. We removed the screen saver support because of that and because of the energy saving. Call it animations or transitions, but not screen saver.</li>
</ul>

<p>Personally I dislike animations on the lock screen by default due to the environment impact. We burn CPU and GPU cycles to show animations nobody will see (if the screen is locked the reason is mostly that the user went away). In times of global warming we should consider the impact of enabling animations for maybe millions of systems.</p>

<p>My suggestion is to:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Decrease the default power saving time</li>
<li class="remarkup-list-item">Ensure those animations stop when screen is power saved</li>
<li class="remarkup-list-item">Increase the time between wallpaper transitions in e.g. Fibonacci time.</li>
</ul></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>