I just finished a patch that implements tying grouping schemes to playlist layouts, and created a merge request at:<br><br><a href="http://www.gitorious.org/amarok/amarok/merge_requests/1415">http://www.gitorious.org/amarok/amarok/merge_requests/1415</a><br>

<br>If this is too much given the state of the freeze (it does add some extra strings), I'm happy to just let this sit until 2.2.1.<br><br>Thanks!<br>Tim<br><br><div class="gmail_quote">On Sun, Aug 30, 2009 at 9:11 AM, Téo Mrnjavac <span dir="ltr"><<a href="mailto:teo@getamarok.com">teo@getamarok.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">On Sun, Aug 30, 2009 at 4:28 PM, Tim Bocek<<a href="mailto:tim.bocek@gmail.com">tim.bocek@gmail.com</a>> wrote:<br>


> I'm willing to give it a try, though I might be a bit short on time in the<br>
> next few days and might not get it by string freeze, which IIRC is tomorrow<br>
> (if I gathered correctly from the database junior job thread, I can't find a<br>
> page on the InfoBase wiki that has this).<br>
><br>
</div>Yup, it's tomorrow.<br>
<div class="im">> Just to clarify: on the UI side, you're asking me to remove the "No<br>
> Grouping" checkbox and replace it with a "Group By" combo box that has:<br>
> Artist<br>
> Album<br>
> Composer<br>
> etc.<br>
> No Grouping<br>
><br>
</div>Exactly.<br>
<div class="im">> If so, some questions:  Does the menu at the bottom of the playlist window<br>
> that already has these options stay, or do we remove it in favor of the<br>
> combo box in the layout (I would vote to keep it, personally).<br>
</div>I had a pretty long discussion with other team members about this on<br>
IRC, more than once actually, and we figured that it's best to<br>
completely tie grouping into the layouts configuration for various<br>
reasons, mostly to keep grouping consistent and to avoid cluttering<br>
the interface.<br>
So the grouping menu at the bottom of the playlist would be replaced<br>
by the combobox in the playlist layout configuration dialog, and the<br>
current grouping category would be saved in the playlist layout xml.<br>
Each layout would incorporate a selected grouping category. We just<br>
don't see many users switching grouping schemes on the fly without<br>
switching layouts too, this is because the vast majority of grouping<br>
schemes look just plain wrong with some layouts, and vice versa. So<br>
the "No Grouping" layout would come with "No Grouping" as grouping<br>
category, the "Default" layout would come with "Album" as grouping<br>
category, and other grouping categories would be selectable if one<br>
chooses to create an alternative layout.<br>
<div class="im"><br>
><br>
> No worries about the sorting slip, I know what you meant ;)<br>
><br>
> Thanks<br>
> -Tim<br>
><br>
</div>Cheers<br>
<font color="#888888">Téo<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Amarok mailing list<br>
<a href="mailto:Amarok@kde.org">Amarok@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/amarok" target="_blank">https://mail.kde.org/mailman/listinfo/amarok</a><br>
</div></div></blockquote></div><br>