----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100630/#review3110 -----------------------------------------------------------
Hi all, I spend a lot of time making the dynamic playlist easier to understand without loosing (much) functionality. The changes since the last review are: 1. added a SearchBias where you can use a normal search query to filter your songs. Just like in the filter. 2. Use a normal tree view without much fancy widgets in it. This makes the playlist much easier to understand. See screenshot: http://i.imgur.com/uVbzf.jpg 3. preventing duplicates is no a global switch. No additional bias or setting needed 4. The "not" bias is not longer there. Instead we have a "invert" flag for the only bias where it makes sense. The SearchBias has this functionality already. 5. Drag and drop should work. So you can re-order a bias or a playlist by simply dragging it. 6. There are four default playlists from very simple to moderately complex showing a novice user what you can do. So, please have a look at the dynamicplaylist branch and give me your feedback. - Ralf On Feb. 11, 2011, 5:19 p.m., Ralf Engels wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100630/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2011, 5:19 p.m.) > > > Review request for Amarok. > > > Summary > ------- > > This fix (actually not the attached patch but the dynamicplaylist branch) > changes most of the dynamic playlist. > > Main parts are the UI which was seen as confusing by many users and produced > unexpected results. > The global bias is now a "part" bias which allows you to set a part of the > collection matching a criteria. > The custom bias is now part of the bias system. I converted all existing > custom biases to the new interface just to make sure that it's usable. > The fuzzy bias was completely removed. If someone really wants to have a bias > which results in a normal distribution of one numeric value then please feel > free to add it again. For now you can define this kind of bias by defining > something like this "(rating > 1) AND (rating < 4)" > > The branch needs to be merged back to mainline and might have conflicts. > > > This addresses bugs 175163, 175172, 177627, 188360, 188360, 228738, 230773, > 232673, 233859, 260001, 260003, 265191, and, more, probably, and some. > https://bugs.kde.org/show_bug.cgi?id=175163 > https://bugs.kde.org/show_bug.cgi?id=175172 > https://bugs.kde.org/show_bug.cgi?id=177627 > https://bugs.kde.org/show_bug.cgi?id=188360 > https://bugs.kde.org/show_bug.cgi?id=188360 > https://bugs.kde.org/show_bug.cgi?id=228738 > https://bugs.kde.org/show_bug.cgi?id=230773 > https://bugs.kde.org/show_bug.cgi?id=232673 > https://bugs.kde.org/show_bug.cgi?id=233859 > https://bugs.kde.org/show_bug.cgi?id=260001 > https://bugs.kde.org/show_bug.cgi?id=260003 > https://bugs.kde.org/show_bug.cgi?id=265191 > https://bugs.kde.org/show_bug.cgi?id=and > https://bugs.kde.org/show_bug.cgi?id=more > https://bugs.kde.org/show_bug.cgi?id=probably > https://bugs.kde.org/show_bug.cgi?id=some > > > Diffs > ----- > > ChangeLog 809d5e7 > > Diff: http://git.reviewboard.kde.org/r/100630/diff > > > Testing > ------- > > Used it for a couple of times and wrote new auto tests. > Tested against bug reports. > > > Thanks, > > Ralf > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
