Review Request: New dynamic bias

[Ralf Engels]

-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110504/6960c557/attachment.htm