Fix chars sometimes overflowing a tiny little bit
[quassel.git] / src / qtui / chatitem.cpp
index ff81259..dfb069e 100644 (file)
@@ -34,7 +34,7 @@ ChatItem::ChatItem(const QPersistentModelIndex &index_, QGraphicsItem *parent) :
 }
 
 ChatItem::~ChatItem() {
-
+  delete _layout;
 }
 
 QVariant ChatItem::data(int role) const {
@@ -48,8 +48,8 @@ QVariant ChatItem::data(int role) const {
 int ChatItem::setWidth(int w) {
   w -= 10;
   if(w == _boundingRect.width()) return _boundingRect.height();
-  int h = heightForWidth(w);
   _boundingRect.setWidth(w);
+  int h = heightForWidth(w);
   _boundingRect.setHeight(h);
   if(haveLayout()) updateLayout();
   return h;
@@ -59,55 +59,72 @@ int ChatItem::heightForWidth(int width) {
   if(data(ChatLineModel::ColumnTypeRole).toUInt() != ChatLineModel::ContentsColumn)
     return fontMetrics()->lineSpacing(); // only contents can be multi-line
 
-  ChatLineModel::WrapList wrapList = data(ChatLineModel::WrapListRole).value<ChatLineModel::WrapList>();
   _lines = 1;
-  qreal w = 0;
-  for(int i = 0; i < wrapList.count(); i++) {
-    w += wrapList.at(i).width;
-    if(w <= width) {
-      w += wrapList.at(i).trailing;
-      continue;
-    }
-    _lines++;
-    w = wrapList.at(i).width;
-    while(w >= width) {  // handle words longer than a line
-      _lines++;
-      // We do not want to compute an exact split position (that would require us to calculate glyph widths
-      // and also apply formats in this step...)
-      // Just using width should be a good estimate, but since a character can't be split in the middle, we
-      // subtract averageCharWidth as well... sometimes this won't be enough, but meh.
-      w -= width - 4*fontMetrics()->averageCharWidth();
+  WrapColumnFinder finder(this);
+  while(finder.nextWrapColumn() > 0) _lines++;
+  return _lines * fontMetrics()->lineSpacing();
+}
+
+ChatItem::WrapColumnFinder::WrapColumnFinder(ChatItem *_item) : item(_item) {
+  wrapList = item->data(ChatLineModel::WrapListRole).value<ChatLineModel::WrapList>();
+  wordidx = 0;
+  layout = 0;
+  lastwrapcol = 0;
+  lastwrappos = 0;
+  w = 0;
+}
+
+ChatItem::WrapColumnFinder::~WrapColumnFinder() {
+  delete layout;
+}
+
+int ChatItem::WrapColumnFinder::nextWrapColumn() {
+  while(wordidx < wrapList.count()) {
+    w += wrapList.at(wordidx).width;
+    if(w >= item->width()) {
+      if(lastwrapcol >= wrapList.at(wordidx).start) {
+        // first word, and it doesn't fit
+        if(!line.isValid()) {
+          layout = item->createLayout(QTextOption::NoWrap);
+          layout->beginLayout();
+          line = layout->createLine();
+          line.setLineWidth(item->width());
+          layout->endLayout();
+        }
+        int idx = line.xToCursor(lastwrappos + item->width(), QTextLine::CursorOnCharacter);
+        qreal x = line.cursorToX(idx, QTextLine::Trailing);
+        w = w - wrapList.at(wordidx).width - (x - lastwrappos);
+        lastwrappos = x;
+        lastwrapcol = idx;
+        return idx;
+      }
+      // not the first word, so just wrap before this
+      lastwrapcol = wrapList.at(wordidx).start;
+      lastwrappos = lastwrappos + w - wrapList.at(wordidx).width;
+      w = 0;
+      return lastwrapcol;
     }
+    w += wrapList.at(wordidx).trailing;
+    wordidx++;
   }
-  return _lines * fontMetrics()->lineSpacing();
+  return -1;
 }
 
-void ChatItem::layout() {
-  if(haveLayout()) return;
-  _layout = new QTextLayout(data(MessageModel::DisplayRole).toString());
+QTextLayout *ChatItem::createLayout(QTextOption::WrapMode wrapMode) {
+  QTextLayout *layout = new QTextLayout(data(MessageModel::DisplayRole).toString());
 
   QTextOption option;
-  option.setWrapMode(QTextOption::WrapAnywhere);
-  _layout->setTextOption(option);
-
-  // Convert format information into a FormatRange
-  QList<QTextLayout::FormatRange> formatRanges;
-  UiStyle::FormatList formatList = data(MessageModel::FormatRole).value<UiStyle::FormatList>();
-  QTextLayout::FormatRange range;
-  int i = 0;
-  for(i = 0; i < formatList.count(); i++) {
-    range.format = QtUi::style()->mergedFormat(formatList.at(i).second);
-    range.start = formatList.at(i).first;
-    if(i > 0) formatRanges.last().length = range.start - formatRanges.last().start;
-    formatRanges.append(range);
-  }
-  if(i > 0) formatRanges.last().length = _layout->text().length() - formatRanges.last().start;
-  _layout->setAdditionalFormats(formatRanges);
-  updateLayout();
+  option.setWrapMode(wrapMode);
+  layout->setTextOption(option);
+
+  QList<QTextLayout::FormatRange> formatRanges
+         = QtUi::style()->toTextLayoutList(data(MessageModel::FormatRole).value<UiStyle::FormatList>(), layout->text().length());
+  layout->setAdditionalFormats(formatRanges);
+  return layout;
 }
 
 void ChatItem::updateLayout() {
-  if(!haveLayout()) layout();
+  if(!haveLayout()) _layout = createLayout(QTextOption::WrapAnywhere);
 
   // Now layout
   ChatLineModel::WrapList wrapList = data(ChatLineModel::WrapListRole).value<ChatLineModel::WrapList>();
@@ -116,34 +133,15 @@ void ChatItem::updateLayout() {
   ChatLineModel::Word word = wrapList.at(0);
 
   qreal h = 0;
+  WrapColumnFinder finder(this);
   _layout->beginLayout();
   forever {
     QTextLine line = _layout->createLine();
     if (!line.isValid())
       break;
 
-    if(word.width >= width()) {
-      line.setLineWidth(width());
-      word.width -= line.naturalTextWidth();
-      word.start = line.textStart() + line.textLength();
-      //qDebug() << "setting width: " << width();
-    } else {
-      int w = 0;
-      while((w += word.width) <= width() && wordidx < wrapList.count()) {
-        w += word.trailing;
-        if(++wordidx < wrapList.count()) word = wrapList.at(wordidx);
-        else {
-          // last word (and it fits), but if we expected an extra line, wrap anyway here
-          // yeah, this is cheating, but much cheaper than computing widths ourself
-          if(_layout->lineCount() < _lines) {
-            wordidx--; qDebug() << "trigger!" << _lines << _layout->text();
-            break;
-          }
-        }
-      }
-      int lastcol = wordidx < wrapList.count() ? word.start : _layout->text().length();
-      line.setNumColumns(lastcol - line.textStart());// qDebug() << "setting cols:" << lastcol - line.textStart();
-    }
+    int col = finder.nextWrapColumn();
+    line.setNumColumns(col >= 0 ? col - line.textStart() : _layout->text().length());
 
     h += fontMetrics()->leading();
     line.setPosition(QPointF(0, h));
@@ -157,9 +155,11 @@ void ChatItem::clearLayout() {
   _layout = 0;
 }
 
+//int ChatItem::findNextWrapColumn(
+
 void ChatItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget) {
   Q_UNUSED(option); Q_UNUSED(widget);
-  layout();
+  if(!haveLayout()) updateLayout();
   _layout->draw(painter, QPointF(0,0), QVector<QTextLayout::FormatRange>(), boundingRect());
   painter->drawRect(boundingRect());
   int width = 0;
@@ -170,51 +170,6 @@ void ChatItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *option,
   }
 }
 
-/*
-void ChatItem::layout() {
-  if(!_layout.additionalFormats().count()) return; // no text set
-  if(_width <= 0) return;
-  prepareGeometryChange();
-  QFontMetrics metrics(_layout.additionalFormats()[0].format.font());
-  int leading = metrics.leading();
-  int height = 0;
-  _layout.setTextOption(textOption());
-  _layout.beginLayout();
-  while(1) {
-    QTextLine line = _layout.createLine();
-    if(!line.isValid()) break;
-    line.setLineWidth(_width);
-    if(textOption().wrapMode() != QTextOption::NoWrap && line.naturalTextWidth() > _width) {
-      // word did not fit, we need to wrap it in the middle
-      // this is a workaround for Qt failing to handle WrapAtWordBoundaryOrAnywhere correctly
-      QTextOption::WrapMode mode = textOption().wrapMode();
-      textOption().setWrapMode(QTextOption::WrapAnywhere);
-      _layout.setTextOption(textOption());
-      line.setLineWidth(_width);
-      textOption().setWrapMode(mode);
-      _layout.setTextOption(textOption());
-    }
-    height += leading;
-    line.setPosition(QPoint(0, height));
-    height += line.height();
-  }
-  _layout.endLayout();
-  update();
-}    QDateTime _timestamp;
-    MsgId _msgId;
-
-
-QRectF ChatItem::boundingRect() const {
-  return _layout.boundingRect();
-}
-
-void ChatItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget) {
-  Q_UNUSED(option); Q_UNUSED(widget);
-  _layout.draw(painter, QPointF(0, 0));
-
-}
-*/
-
 /*
 void ChatItem::mouseMoveEvent ( QGraphicsSceneMouseEvent * event ) {
   qDebug() << (void*)this << "moving" << event->pos();