[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 126545] it shows an extra divider for the band "A"
From:       Shane King <kde () dontletsstart ! com>
Date:       2006-05-01 4:11:53
Message-ID: 20060501041153.22915.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=126545         
kde dontletsstart com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From kde dontletsstart com  2006-05-01 06:11 -------
SVN commit 536042 by shakes:

Fixed divider grouping for single letter artists and accented characters.

BUG: 126545


 M  +7 -0      collectionbrowser.cpp  


--- trunk/extragear/multimedia/amarok/src/collectionbrowser.cpp #536041:536042
 @ -2760,6 +2760,13  @
         if ( b == i18n("Various Artists") )
             return 1;
 
+    // Need to make single letter artist names sort lower than acented divider items
+    // (e.g. The artist "A" should sort below the divider "Ä") so the divider colapsing
+    // code works correctly. Making the artist a two letter word makes localeAwareCompare
+    // give the result we want. See BR 126545.
+    if ( a.length() == 1 && dynamic_cast<DividerItem*>(i) )
+        a += a;
+
     // No special case, then fall on default
     return QString::localeAwareCompare( a.lower(), b.lower() );
 }
[prev in list] [next in list] [prev in thread] [next in thread]