- int flistidx = -1;
- int fmtend = -1;
- QFontMetricsF *metrics;
- QPair<quint16, quint16> wp(0, 0);
- do {
- idx = finder.toNextBoundary();
- if(idx < 0) idx = _contents.plainText.length();
- else if(finder.boundaryReasons() != QTextBoundaryFinder::StartWord) continue;
- int start = wp.first;
- while(start < idx) {
- if(fmtend <= start) {
- flistidx++;
- fmtend = _contents.formatList.count() > flistidx+1 ? _contents.formatList[flistidx+1].first
- : _contents.plainText.length();
- metrics = QtUi::style()->fontMetrics(_contents.formatList[flistidx].second);
- Q_ASSERT(fmtend > start);
+ int oldidx = 0;
+ ChatLineModel::Word word;
+ word.start = 0;
+ qreal wordstartx = 0;
+
+ QTextLayout layout(_styledMsg.plainContents());
+ QTextOption option;
+ option.setWrapMode(QTextOption::NoWrap);
+ layout.setTextOption(option);
+
+ layout.setAdditionalFormats(QtUi::style()->toTextLayoutList(_styledMsg.contentsFormatList(), length, messageLabel()));
+ layout.beginLayout();
+ QTextLine line = layout.createLine();
+ line.setNumColumns(length);
+ layout.endLayout();
+
+ while((idx = finder.toNextBoundary()) >= 0 && idx <= length) {
+
+ // QTextBoundaryFinder has inconsistent behavior in Qt version up to and including 4.6.3 (at least).
+ // It doesn't point to the position we should break, but to the character before that.
+ // Unfortunately Qt decided to fix this by changing the behavior of QTBF, so now we have to add a version
+ // check. At the time of this writing, I'm still trying to get this reverted upstream...
+ //
+ // cf. https://bugs.webkit.org/show_bug.cgi?id=31076 and Qt commit e6ac173
+ static int needWorkaround = -1;
+ if(needWorkaround < 0) {
+ needWorkaround = 0;
+ QStringList versions = QString(qVersion()).split('.');
+ if(versions.count() == 3 && versions.at(0).toInt() == 4) {
+ if(versions.at(1).toInt() <= 6 && versions.at(2).toInt() <= 3)
+ needWorkaround = 1;