Make cursor open hand again after clicking on internal link

CCBUG: 421437

Make cursor open hand again after clicking on internal link
CCBUG: 421437
da934a0a · Oliver Sander · Nate Graham · c69e8a5b · da934a0a
Commit da934a0a authored May 25, 2020 by Oliver Sander Committed by Nate Graham Oct 13, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 0 deletions

autotests/parttest.cpp autotests/parttest.cpp +7 -0

No files found.
--- a/autotests/parttest.cpp
+++ b/autotests/parttest.cpp
@@ -314,6 +314,13 @@ void PartTest::testClickInternalLink()
    QTest::mouseMove(part.m_pageView->viewport(), QPoint(width * 0.17, height * 0.05));
    QTest::mouseClick(part.m_pageView->viewport(), Qt::LeftButton, Qt::NoModifier, QPoint(width * 0.17, height * 0.05));
    QTRY_COMPARE(part.m_document->currentPage(), 1u);
+
+    // make sure cursor goes back to being an open hand again.  Bug 421437
+    // TODO: This test for bug 421437 has been committed when there was no fix for the bug
+    // available yet.  That's why the QTRY_COMPARE_WITH_TIMEOUT is preceded by QEXPECT_FAIL.
+    // Please remove the QEXPECT_FAIL together with the fix for bug 421437.
+    QEXPECT_FAIL("", "Please remove this QEXPECT_FAIL once bug 421437 is fixed!", Continue);
+    QTRY_COMPARE_WITH_TIMEOUT(part.m_pageView->cursor().shape(), Qt::OpenHandCursor, 1000);
 }

 // Test for bug 421159, which is: When scrolling down with the scroll bar