From d07982b104de5dc2b54bef09c071500ce22cf539 Mon Sep 17 00:00:00 2001 From: Andrew den Exter Date: Wed, 9 May 2012 16:14:34 +1000 Subject: Fix cursorToX for right to left text with trailing whitespace. QTextLine::cursorToX returned the line width for cursor positions outside the width of a wrapped right to left line because the leading space width was always calculated as 0. Returning a non-zero width for the leading space does cause problems for other uses of QTextEngine::alignLine() though as the textAdvance already doesn't include the leading/trailing space so subtracting it there double accounts for it. Task-number: QTBUG-24801 Change-Id: I56cbb139814c32813bebb49de8c045b29154a958 Reviewed-by: Jiang Jiang --- .../auto/gui/text/qtextlayout/tst_qtextlayout.cpp | 440 +++++++++++++++++++++ 1 file changed, 440 insertions(+) (limited to 'tests') diff --git a/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp b/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp index e5238865fb..3a1cd48bef 100644 --- a/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp +++ b/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp @@ -57,6 +57,10 @@ #define TESTFONT_SIZE 12 +Q_DECLARE_METATYPE(QTextOption::WrapMode) +Q_DECLARE_METATYPE(Qt::LayoutDirection) +Q_DECLARE_METATYPE(Qt::AlignmentFlag) + class tst_QTextLayout : public QObject { Q_OBJECT @@ -80,6 +84,12 @@ private slots: void noWrap(); void cursorToXForInlineObjects(); void cursorToXForSetColumns(); + void cursorToXForTrailingSpaces_data(); + void cursorToXForTrailingSpaces(); + void horizontalAlignment_data(); + void horizontalAlignment(); + void horizontalAlignmentMultiline_data(); + void horizontalAlignmentMultiline(); void defaultWordSeparators_data(); void defaultWordSeparators(); void cursorMovementFromInvalidPositions(); @@ -525,6 +535,436 @@ void tst_QTextLayout::cursorToXForSetColumns() QCOMPARE(line.cursorToX(1), (qreal) TESTFONT_SIZE); } +void tst_QTextLayout::cursorToXForTrailingSpaces_data() +{ + qreal width = TESTFONT_SIZE * 4; + + QTest::addColumn("wrapMode"); + QTest::addColumn("textDirection"); + QTest::addColumn("alignment"); + QTest::addColumn("cursorAt0"); + QTest::addColumn("cursorAt4"); + QTest::addColumn("cursorAt6"); + + // Aligned left from start of visible characters. + QTest::newRow("ltr nowrap lalign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignLeft + << qreal(0) + << width + << qreal(TESTFONT_SIZE * 6); + + // Aligned left from start of visible characters. + QTest::newRow("ltr wrap lalign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignLeft + << qreal(0) + << width + << width; + + // Aligned right from end of whitespace characters. + QTest::newRow("ltr nowrap ralign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignRight + << qreal(TESTFONT_SIZE * -2) + << qreal(TESTFONT_SIZE * 2) + << width; + + // Aligned right from end of visible characters. + QTest::newRow("ltr wrap ralign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignRight + << qreal(TESTFONT_SIZE) + << width + << width; + + // Aligned center of all characters + QTest::newRow("ltr nowrap calign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * -1) + << qreal(TESTFONT_SIZE * 3) + << qreal(TESTFONT_SIZE * 5); + + // Aligned center of visible characters + QTest::newRow("ltr wrap calign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * 0.5) + << qreal(width) + << qreal(width); + + // Aligned right from start of visible characters + QTest::newRow("rtl nowrap ralign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignRight + << width + << qreal(0) + << qreal(TESTFONT_SIZE * -2); + + // Aligned right from start of visible characters + QTest::newRow("rtl wrap ralign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignRight + << width + << qreal(0) + << qreal(0); + + // Aligned left from end of whitespace characters + QTest::newRow("rtl nowrap lalign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignLeft + << qreal(TESTFONT_SIZE * 6) + << qreal(TESTFONT_SIZE * 2) + << qreal(0); + + // Aligned left from end of visible characters + QTest::newRow("rtl wrap lalign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignLeft + << qreal(TESTFONT_SIZE * 3) + << qreal(0) + << qreal(0); + + // Aligned center of all characters + QTest::newRow("rtl nowrap calign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * 5) + << qreal(TESTFONT_SIZE * 1) + << qreal(TESTFONT_SIZE * -1); + + // Aligned center of visible characters + QTest::newRow("rtl wrap calign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * 3.5) + << qreal(0) + << qreal(0); +} + +void tst_QTextLayout::cursorToXForTrailingSpaces() +{ + QFETCH(QTextOption::WrapMode, wrapMode); + QFETCH(Qt::LayoutDirection, textDirection); + QFETCH(Qt::AlignmentFlag, alignment); + QFETCH(qreal, cursorAt0); + QFETCH(qreal, cursorAt4); + QFETCH(qreal, cursorAt6); + + QTextLayout layout("%^& ", testFont); + + QTextOption o = layout.textOption(); + o.setTextDirection(textDirection); + o.setAlignment(alignment); + o.setWrapMode(wrapMode); + layout.setTextOption(o); + + layout.beginLayout(); + QTextLine line = layout.createLine(); + line.setLineWidth(TESTFONT_SIZE * 4); + layout.endLayout(); + + QCOMPARE(line.cursorToX(0), cursorAt0); + QCOMPARE(line.cursorToX(4), cursorAt4); + QCOMPARE(line.cursorToX(6), cursorAt6); +} + +void tst_QTextLayout::horizontalAlignment_data() +{ + qreal width = TESTFONT_SIZE * 4; + + QTest::addColumn("wrapMode"); + QTest::addColumn("textDirection"); + QTest::addColumn("alignment"); + QTest::addColumn("naturalLeft"); + QTest::addColumn("naturalRight"); + + // Aligned left from start of visible characters. + QTest::newRow("ltr nowrap lalign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 6); + + // Aligned left from start of visible characters. + QTest::newRow("ltr wrap lalign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 3); + + // Aligned right from end of whitespace characters. + QTest::newRow("ltr nowrap ralign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignRight + << qreal(TESTFONT_SIZE * - 2) + << width; + + // Aligned right from end of visible characters. + QTest::newRow("ltr wrap ralign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignRight + << qreal(TESTFONT_SIZE) + << width; + + // Aligned center of all characters + QTest::newRow("ltr nowrap calign") + << QTextOption::NoWrap + << Qt::LeftToRight + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * -1) + << qreal(TESTFONT_SIZE * 5); + + // Aligned center of visible characters + QTest::newRow("ltr wrap calign") + << QTextOption::WrapAnywhere + << Qt::LeftToRight + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * 0.5) + << qreal(TESTFONT_SIZE * 3.5); + + // Aligned right from start of visible characters + QTest::newRow("rtl nowrap ralign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignRight + << qreal(TESTFONT_SIZE * -2) + << width; + + // Aligned right from start of visible characters + QTest::newRow("rtl wrap ralign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignRight + << qreal(TESTFONT_SIZE * 1) + << width; + + // Aligned left from end of whitespace characters + QTest::newRow("rtl nowrap lalign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 6); + + // Aligned left from end of visible characters + QTest::newRow("rtl wrap lalign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 3); + + // Aligned center of all characters + QTest::newRow("rtl nowrap calign") + << QTextOption::NoWrap + << Qt::RightToLeft + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * -1) + << qreal(TESTFONT_SIZE * 5); + + // Aligned center of visible characters + QTest::newRow("rtl wrap calign") + << QTextOption::WrapAnywhere + << Qt::RightToLeft + << Qt::AlignHCenter + << qreal(TESTFONT_SIZE * 0.5) + << qreal(TESTFONT_SIZE * 3.5); +} + +void tst_QTextLayout::horizontalAlignment() +{ + QFETCH(QTextOption::WrapMode, wrapMode); + QFETCH(Qt::LayoutDirection, textDirection); + QFETCH(Qt::AlignmentFlag, alignment); + QFETCH(qreal, naturalLeft); + QFETCH(qreal, naturalRight); + + QTextLayout layout("%^& ", testFont); + + QTextOption o = layout.textOption(); + o.setTextDirection(textDirection); + o.setAlignment(alignment); + o.setWrapMode(wrapMode); + layout.setTextOption(o); + + layout.beginLayout(); + QTextLine line = layout.createLine(); + line.setLineWidth(TESTFONT_SIZE * 4); + layout.endLayout(); + + QRectF naturalRect = line.naturalTextRect(); + QCOMPARE(naturalRect.left(), naturalLeft); + QCOMPARE(naturalRect.right(), naturalRight); +} + + +void tst_QTextLayout::horizontalAlignmentMultiline_data() +{ + qreal width = TESTFONT_SIZE * 8; + + QString linebreakText("^%$&\u2028^%&*^$"); + QString wrappingText("^%$&^%&*^$"); + QString wrappingWhitespaceText("^%$& ^%&*^$"); + + QTest::addColumn("text"); + QTest::addColumn("textDirection"); + QTest::addColumn("alignment"); + QTest::addColumn("firstLeft"); + QTest::addColumn("firstRight"); + QTest::addColumn("lastLeft"); + QTest::addColumn("lastRight"); + + Qt::LayoutDirection textDirection[] = { Qt::LeftToRight, Qt::RightToLeft }; + QByteArray textDirectionText [] = { "ltr ", "rtl " }; + for (int i = 0; i < 2; ++i) { + // Aligned left from start of visible characters. + QTest::newRow(textDirectionText[i] + "linebreak lalign") + << linebreakText + << textDirection[i] + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 4) + << qreal(0) + << qreal(TESTFONT_SIZE * 6); + + // Aligned left from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-text lalign") + << wrappingText + << textDirection[i] + << Qt::AlignLeft + << qreal(0) + << width + << qreal(0) + << qreal(TESTFONT_SIZE * 2); + + // Aligned left from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-ws lalign") + << wrappingWhitespaceText + << textDirection[i] + << Qt::AlignLeft + << qreal(0) + << qreal(TESTFONT_SIZE * 4) + << qreal(0) + << qreal(TESTFONT_SIZE * 6); + + // Aligned right from start of visible characters. + QTest::newRow(textDirectionText[i] + "linebreak ralign") + << linebreakText + << textDirection[i] + << Qt::AlignRight + << qreal(TESTFONT_SIZE * 4) + << width + << qreal(TESTFONT_SIZE * 2) + << width; + + // Aligned right from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-text ralign") + << wrappingText + << textDirection[i] + << Qt::AlignRight + << qreal(0) + << width + << qreal(TESTFONT_SIZE * 6) + << width; + + // Aligned left from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-ws ralign") + << wrappingWhitespaceText + << textDirection[i] + << Qt::AlignRight + << qreal(TESTFONT_SIZE * 4) + << width + << qreal(TESTFONT_SIZE * 2) + << width; + + // Aligned center from start of visible characters. + QTest::newRow(textDirectionText[i] + "linebreak calign") + << linebreakText + << textDirection[i] + << Qt::AlignCenter + << qreal(TESTFONT_SIZE * 2) + << qreal(TESTFONT_SIZE * 6) + << qreal(TESTFONT_SIZE * 1) + << qreal(TESTFONT_SIZE * 7); + + // Aligned center from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-text calign") + << wrappingText + << textDirection[i] + << Qt::AlignCenter + << qreal(0) + << width + << qreal(TESTFONT_SIZE * 3) + << qreal(TESTFONT_SIZE * 5); + + // Aligned center from start of visible characters. + QTest::newRow(textDirectionText[i] + "wrap-ws calign") + << wrappingWhitespaceText + << textDirection[i] + << Qt::AlignCenter + << qreal(TESTFONT_SIZE * 2) + << qreal(TESTFONT_SIZE * 6) + << qreal(TESTFONT_SIZE * 1) + << qreal(TESTFONT_SIZE * 7); + } +} + +void tst_QTextLayout::horizontalAlignmentMultiline() +{ + QFETCH(QString, text); + QFETCH(Qt::LayoutDirection, textDirection); + QFETCH(Qt::AlignmentFlag, alignment); + QFETCH(qreal, firstLeft); + QFETCH(qreal, firstRight); + QFETCH(qreal, lastLeft); + QFETCH(qreal, lastRight); + + QTextLayout layout(text, testFont); + + QTextOption o = layout.textOption(); + o.setTextDirection(textDirection); + o.setAlignment(alignment); + o.setWrapMode(QTextOption::WrapAnywhere); + layout.setTextOption(o); + + layout.beginLayout(); + QTextLine firstLine = layout.createLine(); + QTextLine lastLine; + for (QTextLine line = firstLine; line.isValid(); line = layout.createLine()) { + line.setLineWidth(TESTFONT_SIZE * 8); + lastLine = line; + } + layout.endLayout(); + + qDebug() << firstLine.textLength() << firstLine.naturalTextRect() << lastLine.naturalTextRect(); + + QRectF rect = firstLine.naturalTextRect(); + QCOMPARE(rect.left(), firstLeft); + QCOMPARE(rect.right(), firstRight); + + rect = lastLine.naturalTextRect(); + QCOMPARE(rect.left(), lastLeft); + QCOMPARE(rect.right(), lastRight); +} + void tst_QTextLayout::defaultWordSeparators_data() { QTest::addColumn("text"); -- cgit v1.2.3