Hi,

Its nice to see such improvement :)

Just a few things to add - It looks like entire result gets compared to
entire data from track. It's hardly the best way to it, since the longer
correct part is, the more similar results looks like. From the screen shots
its obviously not - while title and artist remains the same, album is
entirely different - so its not that close match.

That's why, I think it is better to match field to field and use average to
sort results down.

This means title gets compared to title, album to album etc. Doing so would
draw user attention to the field that has difference.
To indicate match level/ration discrete rather than continuous colors might
be used.
* None (or default background color) - If the is no difference
* Green - Difference is quite minor (like misspells, or fixing capitals)
* Yellow - There is some similarity e.g. one is a substring of another ("top
dance" and "Top Dance vol 2.0")
* Red - Almost nothing is in common (line in screenshots album filed)

To prevent color noise, only {border-bottom: 3px COLOR solid; font-weight:
bold } (no bold for "none" color) an be used instead of entire background.
Such way even color blind will know where to drag attention (bold)

Thanks,

Lukas



On 2 November 2010 17:32, Sergey Ivanov <123k...@gmail.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100127/
>   Review request for Amarok.
> By Sergey Ivanov.
>
> *Updated 2010-11-02 15:32:21.079819*
> Changes
>
> Change gradients for results ( (orange) FFAA00->55FF00 (green) for 
> MusicBrainz; (cyan)00AAFF->00FF55(green) for MusicDNS and green (00FF00) if 
> track found by both engines).
> Tooltips with match ratio.
>
>   Description
>
> Rebuild MBT dialog. Now search result presented by a Tree, 
> trackToSearch->SearchResults. Search results sorted by similarity ratio ( 
> more similar is on top ), and has different colors corresponding to that. 
> MusicDNS search results marked with a cyan color and stay on bottom. There is 
> no Start search button any more, both searches run simultaneously.
>
> Will be Implemented soon:
> 1) If one TrackID founded by both searches, this result automatically moves 
> on top.
> 2) MB icon linked to track page on musicbrainz.org. //need to find a place 
> for It
> 3..n Needs your advices. :)
>
>   Diffs (updated)
>
>    - src/CMakeLists.txt (2abeca2)
>    - src/dialogs/MusicBrainzTagger.h (e33c9f6)
>    - src/dialogs/MusicBrainzTagger.cpp (cdc3ec5)
>    - src/dialogs/MusicBrainzTagger.ui (3f14236)
>    - src/dialogs/TagDialog.cpp (e98acf5)
>    - src/musicbrainz/MusicBrainzFinder.h (49c8f6b)
>    - src/musicbrainz/MusicBrainzFinder.cpp (41ddbaa)
>    - src/musicbrainz/MusicBrainzMeta.h (57ed363)
>    - src/musicbrainz/MusicBrainzTagsModel.h (a93929a)
>    - src/musicbrainz/MusicBrainzTagsModel.cpp (1bf7389)
>    - src/musicbrainz/MusicBrainzTrackListModel.h (6eaa648)
>    - src/musicbrainz/MusicBrainzTrackListModel.cpp (fa1fc10)
>    - src/musicbrainz/MusicDNSFinder.h (2cd00d1)
>    - src/musicbrainz/MusicDNSFinder.cpp (9c31f38)
>
> View Diff <http://git.reviewboard.kde.org/r/100127/diff/>
> Screenshots (updated)
>  <http://git.reviewboard.kde.org/r/100127/s/12/>
> <http://git.reviewboard.kde.org/r/100127/s/13/>
>
> _______________________________________________
> Amarok-devel mailing list
> Amarok-devel@kde.org
> https://mail.kde.org/mailman/listinfo/amarok-devel
>
>
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to