I'm willing to give it a try, though I might be a bit short on time in the next few days and might not get it by string freeze, which IIRC is tomorrow (if I gathered correctly from the database junior job thread, I can't find a page on the InfoBase wiki that has this).<br>

<br>Just to clarify: on the UI side, you're asking me to remove the "No 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>If so, some questions:  Does the menu at the bottom of the playlist window that already has these options stay, or do we remove it in favor of the combo box in the layout (I would vote to keep it, personally).  If yes to that, if the user selects a playlist that is set up to group by, say, composer, and then selects in the widget in the playlist window to group by artist instead, does that change the playlist layout or just override the default?<br>

<br>No worries about the sorting slip, I know what you meant ;)<br><br>Thanks<br>-Tim<br><br><div class="gmail_quote">On Sun, Aug 30, 2009 at 3:19 AM, Nikolaj Hald Nielsen <span dir="ltr"><<a href="mailto:nhnfreespirit@gmail.com" target="_blank">nhnfreespirit@gmail.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;">So...<br>
<br>
I committed your patch and it works great. however, it started a bit<br>
of a discussion among the regular Amarok devs about whether grouping<br>
really should be integrated into the playlist layouts instead of<br>
something completely separate. So if you are up for it, I have an idea<br>
of how you can expand on the patch you already did simply the user<br>
experience even more, and likely deal with the remaining issues of<br>
your patch (the inline editing stuff) as a side effect.<br>
<br>
The idea is that instead of your new checkbox, we would actually have<br>
a drop down list for selecting the grouping mode to use for each<br>
layout. One of these options would be "no grouping" which would work<br>
much like your checkbox does now (disabling editing of header and body<br>
items). This makes sense, as a grouping mode is really mostly<br>
worthless unless there is a playlist layout that matches. (for<br>
instance, it makes little sense to use a layout that shows album info<br>
in the header if you are grouping by composer... )<br>
<br>
However, instead of having the rendering code use the single items<br>
layout config for both body and single tracks, you would integrate it<br>
with the sorting proxy so that if the sorting mode is set to "no<br>
sorting" it will simply return the "single" group status for each and<br>
every track, meaning that they will all get painted using the "single"<br>
layout config anyway. This would remove some of the code you have<br>
added to the delegate and make sure that you don't have to special<br>
case anything for the inline editor either.<br>
<br>
I think this would really tie everything together in a very nice way<br>
and simplify the code a bit as well. If you are interested in this,<br>
let me know. If not We will likely try to do it ourselves asap as we<br>
have a freeze coming up soon. You can go to irc channel #amarok on<br>
<a href="http://irc.freenode.net" target="_blank">irc.freenode.net</a> and poke either me (nhn) or Teo (Teo`) for help, or<br>
you can of course ask questions here.<br>
<div><div></div><div><br>
- Nikolaj<br>
_______________________________________________<br>
Amarok mailing list<br>
<a href="mailto:Amarok@kde.org" target="_blank">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>