<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/120880/">https://git.reviewboard.kde.org/r/120880/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 29th, 2014, 4:06 p.m. CET, <b>Thomas Pfeiffer</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">That means that other desktop enviornments cannot really use KWin without installing at least one l&f package anymore, right? Could that be a problem, or is KWin no fun without any L&Fs installed, anyway, even without that change?</p></pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I already prepared a blog post and addressed the implication. Verbatim quote from the draft:</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The change has also some implications for users of non Plasma desktop environments wanting to use KWin as their window manager. By moving the switchers out, KWin removes some of the Plasma dependencies. All switchers provided by KWin are using Plasma components, the default switcher is part of the design concept for Plasma 5 following the same idea as other similar components. Thus KWin had a direct dependency on Plasma with the window switchers. This is now kind of solved by not offering any switcher at all.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">My suggestion for desktop environment projects wanting to use KWin is to provide their own default Look'n'Feel package with a Window and Desktop switcher specific for their environment. On the other hand I don't see a problem with providing a simple fallback theme in KWin. And of course there are still the Desktop Effects for switching between Windows (CoverSwitch and FlipSwitch) installed by default and just need to be enabled.</p>
</blockquote></pre>
<br />










<p>- Martin</p>


<br />
<p>On October 29th, 2014, 8:50 a.m. CET, Martin Gräßlin wrote:</p>









<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for kwin and Plasma.</div>
<div>By Martin Gräßlin.</div>


<p style="color: grey;"><i>Updated Oct. 29, 2014, 8:50 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdeplasma-addons
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It was decided that the default window switcher goes into the
look and feel package and all other window switchers go into
kdeplasma-addons. This means KWin does no longer offer any window
switchers.</pre>
  </td>
 </tr>
</table>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>windowswitchers/small_icons/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/small_icons/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/text/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/text/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/thumbnails/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/thumbnails/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/present_windows/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/present_windows/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/informative/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/compact/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/informative/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/big_icons/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/compact/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/big_icons/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>CMakeLists.txt <span style="color: grey">(09180ea05aa4dad12a1788c72abc082707cf08af)</span></li>

 <li>windowswitchers/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>

 <li>windowswitchers/IconTabBox.qml <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/120880/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>








  </div>
 </body>
</html>