commit 645e4c372fa2f8a2b95635e141ffccc59358f084
author Edric Milaret <edric.ladent-milaret@savoirfairelinux.com> Mon Jan 18 15:55:08 2016 -0500
committer Edric Milaret <edric.ladent-milaret@savoirfairelinux.com> Tue Jan 19 15:34:54 2016 -0400
tree 437c905a2d25550ce659f205e1296a74514d56c7
parent a13b48c35dc3f78887da7c84139fed1df9f301a9

ui: hide non-displayable character in smartlist

- Also move the video button to the right

Change-Id: Ib1f862ca923a6600457248d672f0b973e3834366
Tuleap: #148

smartlistdelegate.cpp

diff --git a/smartlistdelegate.cpp b/smartlistdelegate.cpp
index 8e742ab..755c261 100644
--- a/smartlistdelegate.cpp
+++ b/smartlistdelegate.cpp
@@ -41,9 +41,11 @@
 {
     QStyleOptionViewItem opt(option);
     painter->setRenderHint(QPainter::Antialiasing);
+
     opt.decorationSize = QSize(sizeImage_, sizeImage_);
     opt.decorationPosition = QStyleOptionViewItem::Left;
     opt.decorationAlignment = Qt::AlignCenter;
+
     if (opt.state & QStyle::State_HasFocus)
         opt.state ^= QStyle::State_HasFocus;
 
@@ -53,12 +55,15 @@
     QRect rect = opt.rect;
     rect.setLeft(0);
 
-    QRect rectTexts(16 + rect.left() + dx_+sizeImage_, rect.top(), rect.width(), rect.height()/2);
+    QRect rectTexts(16 + rect.left() + dx_ + sizeImage_,
+                    rect.top(),
+                    rect.width(),
+                    rect.height() / 2);
     QRect rectAvatar(16 + rect.left(), rect.top() + dy_, sizeImage_, sizeImage_);
-    drawDecoration(painter, opt, rectAvatar, QPixmap::fromImage(index.data(Qt::DecorationRole).value<QImage>()));
+    drawDecoration(painter, opt, rectAvatar,
+                   QPixmap::fromImage(index.data(Qt::DecorationRole).value<QImage>()));
 
     const int currentRow = index.row();
-
     if (currentRow == rowHighlighted_)
         emit rowSelected(opt.rect);
 
@@ -85,7 +90,16 @@
             painter->setPen(pen);
             font.setBold(true);
             painter->setFont(font);
-            painter->drawText(rectTexts, Qt::AlignBottom | Qt::AlignLeft, name.toString());
+            QFontMetrics fontMetrics(font);
+            QString nameStr = name.toString();
+            auto realRect = fontMetrics.boundingRect(rect, Qt::AlignBottom | Qt::AlignLeft, nameStr);
+            if (realRect.width() > (rect.width() - rectTexts.left() - 30)) {
+                /* 30 here is the size of the video button */
+                auto charToChop = (realRect.width() - (rect.width() - rectTexts.left())) / fontMetrics.averageCharWidth();
+                nameStr.chop(charToChop + 8);
+                nameStr.append(QStringLiteral("..."));
+            }
+            painter->drawText(rectTexts, Qt::AlignBottom | Qt::AlignLeft, nameStr);
         }
 
         QVariant state = index.data(static_cast<int>(Ring::Role::FormattedState));
@@ -126,6 +140,5 @@
 {
     QSize size = QItemDelegate::sizeHint(option, index);
     size.setHeight(cellHeight_);
-    size.setWidth(cellWidth_);
     return size;
 }