<br><br><div class="gmail_quote">2010/4/8 Marco Martin <span dir="ltr">&lt;<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Thursday 08 April 2010, Christophe Olinger wrote:<br>
</div><div class="im">&gt; &gt; I like the idea of the state machine and how it could help us correctly<br>
&gt; &gt; managing the workflow of the mediacenter. Unfortunately i feel something<br>
&gt; &gt; in the current implementation lacks of robustness (does this word<br>
&gt; &gt; exist?).<br>
&gt; &gt;<br>
&gt; &gt; As Aaron pointed out, we need to understend whether we are aproaching<br>
&gt; &gt; this the right way.<br>
&gt; &gt;<br>
&gt; &gt; The mistake with the MediaLayout in this patch well shows that we should<br>
&gt; &gt; probably re-design something. The need for states is due to the fact that<br>
&gt; &gt; we want PMC to know the current media in order to differently behave and<br>
&gt; &gt; interact with the user. This way we could give the best set of actions<br>
&gt; &gt; for each kind of media chosen by the user.<br>
&gt; &gt; The first clear issue was the lack for a way of changing the UI<br>
&gt; &gt; accordingly to the chosen media.<br>
&gt;<br>
&gt; Here is a list of the issues we need to tackle correctly and agree on where<br>
&gt; to do it<br>
&gt; Things that change on a state switch:<br>
&gt;<br>
&gt; * The applets (called UIComponents) get a new set of widgets. Some widgets<br>
&gt; are always the same. These I called MainSubComponents and are only added<br>
&gt; once at PMC initialization. Others change with each state, these are<br>
&gt; shown/hidden on each state switch.<br>
<br>
</div>those can be added by the applet itself and the outside world isn&#39;t required<br>
to eve know they exists<br>
<div class="im"><br>
&gt; -&gt;adding MainSubcomponents is currently handled by the MediaCenterState<br>
&gt; class, adding subcomponents is handled by the different state classes.<br>
<br>
</div>probably should all be done by the MediaCenterState class. all that subclasses<br>
should do is to give the list of what they want enabled<br></blockquote><div><br></div><div>I do agree with this.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
but again, should be committed then experiments can be made</blockquote><div><br></div><div>Committed some hour ago! :-)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
 </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><br>
Cheers,<br>
Marco Martin<br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>KDE e.V. member<br><br>