extragear/multimedia/amarok/src/playlist

Jeff Mitchell kde-dev at emailgoeshere.com
Sun Jul 27 22:19:11 CEST 2008


SVN commit 838427 by mitchell:

Adds colons after the artist in the playlist.  I think this definitely looks better for single tracks when the artist and album are on the same line, as it helps indicate that the multi-character 
space is separating the two elements.  Not sure about multi-line text in headers for more than one track.

CCMAIL:amarok-devel at kde.org


 M  +2 -2      PlaylistGraphicsItem.cpp  


--- trunk/extragear/multimedia/amarok/src/playlist/PlaylistGraphicsItem.cpp #838426:838427
@@ -289,7 +289,7 @@
             QString artist;
             if( track->artist() )
                 artist = track->artist()->name();
-            m_items->topLeftText->setEditableText( artist, spaceForTopLeft );
+            m_items->topLeftText->setEditableText( artist + ":", spaceForTopLeft );
             m_items->topLeftText->setPos( leftAlignX, MARGIN );
         }
 
@@ -337,7 +337,7 @@
             font.setBold( true );
             m_items->topLeftText->setFont( font );
 
-            artist = s_fm->elidedText ( artist, Qt::ElideRight, headTextWidth );
+            artist = s_fm->elidedText ( artist + ":", Qt::ElideRight, headTextWidth );
             int artistWidth = (int)s_fm->width( artist );
             offsetX = static_cast<int>( MARGIN + ALBUM_WIDTH + ( ( headTextWidth - artistWidth ) / 2) );
 


More information about the Amarok-devel mailing list