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