Changeset 11406

Timestamp:

Nov 14, 2010, 5:03:38 AM (12 years ago)

Author:

charles

Message:

(trunk qt) #3739 "filterbar should use locale info to group thousands' place (ex: 1,000 instead of 1000)" -- done.

Location:

trunk/qt

Files:

• filterbar.cc (modified) (7 diffs)
• filterbar.h (modified) (1 diff)

trunk/qt/filterbar.cc

@@ -184 +184 @@
 
         // draw the count
-        const int count = modelIndex.data( TorrentCountRole ).toInt();
-        if( count >= 0 ) {
-            const QString text = QString::number( count);
+        QString text = modelIndex.data(TorrentCountRole).toString();
+        if( !text.isEmpty( ) )
+        {
             const QPen pen = painter.pen( );
             painter.setPen( opt.palette.color( QPalette::Disabled, QPalette::Text ) );
@@ -196 +196 @@
 
         // draw the text
-        QString text = modelIndex.data( Qt::DisplayRole ).toString();
+        text = modelIndex.data( Qt::DisplayRole ).toString();
         text = painter.fontMetrics().elidedText ( text, Qt::ElideRight, rect.width() );
         s->drawItemText( &painter, rect, Qt::AlignLeft|Qt::AlignVCenter, opt.palette, true, text );
@@ -320 +320 @@
 
     // update the "All" row
-    myTrackerModel->setData( myTrackerModel->index(0,0), myTorrents.rowCount(), TorrentCountRole );
+    myTrackerModel->setData( myTrackerModel->index(0,0), getCountString(myTorrents.rowCount()), TorrentCountRole );
 
     // rows to update
@@ -327 +327 @@
         const QString name = readableHostName( host );
         QStandardItem * row = myTrackerModel->findItems(name).front();
-        row->setData( torrentsPerHost[host], TorrentCountRole );
+        row->setData( getCountString(torrentsPerHost[host]), TorrentCountRole );
         row->setData( favicons.findFromHost(host), Qt::DecorationRole );
     }
@@ -353 +353 @@
         // add the row
         QStandardItem * row = new QStandardItem( favicons.findFromHost( host ), readableHostName( host ) );
-        row->setData( torrentsPerHost[host], TorrentCountRole );
+        row->setData( getCountString(torrentsPerHost[host]), TorrentCountRole );
         row->setData( favicons.findFromHost(host), Qt::DecorationRole );
         row->setData( host, TrackerRole );
@@ -374 +374 @@
     QStandardItem * row = new QStandardItem( tr( "All" ) );
     row->setData( "", TrackerRole );
-    row->setData( myTorrents.rowCount(), TorrentCountRole );
+    row->setData( getCountString(myTorrents.rowCount()), TorrentCountRole );
     model->appendRow( row );
 
@@ -550 +550 @@
         QAbstractItemModel * model = myActivityCombo->model( );
         QModelIndexList indices = model->match( model->index(0,0), ActivityRole, m.mode(), -1 );
-        if( !indices.isEmpty( ) ) {
-            const int count = myFilter.count( m );
-            model->setData( indices.first(), count, TorrentCountRole );
-        }
+        if( !indices.isEmpty( ) )
+            model->setData( indices.first(), getCountString(myFilter.count(m)), TorrentCountRole );
     }
 
     refreshTrackers( );
 }
+
+QString
+FilterBar :: getCountString( int n ) const
+{
+    return n>0 ? QString("%L1").arg(n) : QString();
+}

trunk/qt/filterbar.h

@@ -72 +72 @@
         void recountSoon( );
         void refreshTrackers( );
+        QString getCountString( int n ) const;
 
     private: