----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100630/#review1599 -----------------------------------------------------------
Reminder to myself: We need a "if-else" Bias because currently an album-play is not possible. So: if(pick next song from current album) else (pick song with track number 1) To make the UI easier for non-experts: Add three example default playlists Add a "collection search bias" where you can add a search query from the collection search bar. Current plan to improve the overview with a complex bias: Use a tree view to display a textual representation of the bias. Only when clicked on should it display the full options in a popup or a dialog. - 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
