<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Mar 29, 2009, at 11:32 AM, juanjuan wang wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><pre>Hi,Amarok team members<br>Please take your time to read my proposal for the gsoc 2009 for Multilevel playlist sorting and sorting GUI, give me some guide and advice.<br>Here is the proposal: <a href="http://socghop.appspot.com/document/show/user/gosustar/y1021_fww">http://socghop.appspot.com/document/show/user/gosustar/y1021_fww</a> <br></pre></blockquote><div>Should</div>"One is "single_sort" has some subitems that can directly performed sorting.These are the sorting categories which the user can't sort by simply clicking on the headers in the playlist view, such as "playcount","genre","score".The view can't call the sort() virtual function on the model to reorder the data in the model by them."</div><div><br></div><div>actually read</div><div>"One is "single_sort" has some subitems that can directly performed sorting.These are the sorting categories which the user CAN sort by simply clicking on the headers in the playlist view, such as "playcount","genre","score".The view CAN call the sort() virtual function on the model to reorder the data in the model by them."</div><div><br></div><div>?<br>Comments:</div><div>The "second way" is visually cluttered and far too complicated IMO. The playlist isn't guaranteed a lot of space so sticking this type of interface in is a poor design decision.</div><div><br></div><div>You list the "third part" as your implementation. This should be the first thing you do because if it isn't finished the rest of your proposal becomes useless. It seems it'll also be the most challenging part so best get it out of the way quickly. <br><blockquote type="cite"><pre>Best regards<br>Juanjuan Wang<br></pre> _______________________________________________<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">https://mail.kde.org/mailman/listinfo/amarok</a><br></blockquote></div><br></body></html>