diff options
| author | Miklos Vajna <vmiklos@collabora.com> | 2019-07-16 12:17:01 +0200 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.com> | 2019-07-16 17:43:58 +0200 |
| commit | 3fb7ffeb85d87551ffcddd63b7c90c6b0e573e5f (patch) | |
| tree | c675da0ba0229301587f839eca429d61b91bc3ad /sw/qa/extras/uiwriter | |
| parent | 64fbc0e44d98614d25c81428ec024e36ce27c573 (diff) | |
sw comments on frames: avoid overlay that covers only an as-char image
As-char images had their comment arrow on the bottom left corner,
as-char ones had it at the bottom right one.
Standardize on bottom left and avoid the content overlay in case only
the frame itself is selected.
See commit cf5a3cb687a502e7f71cefb5f7001a73925bee56 (Adapt new test to
actual values observed on macOS, 2019-06-24) on why the test is not for
macOS, till somebody can check if this is working there properly.
Change-Id: I83516ea558182ecc7362f8383894ed07a16d3ad2
Reviewed-on: https://gerrit.libreoffice.org/75706
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'sw/qa/extras/uiwriter')
| -rw-r--r-- | sw/qa/extras/uiwriter/uiwriter2.cxx | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx b/sw/qa/extras/uiwriter/uiwriter2.cxx index 3c1170b80b27..eac8676a2469 100644 --- a/sw/qa/extras/uiwriter/uiwriter2.cxx +++ b/sw/qa/extras/uiwriter/uiwriter2.cxx @@ -36,6 +36,9 @@ #include <redline.hxx> #include <view.hxx> #include <cmdid.h> +#include <AnnotationWin.hxx> +#include <PostItMgr.hxx> +#include <postithelper.hxx> namespace { @@ -1602,6 +1605,22 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testImageComment) pView->GetViewFrame()->GetDispatcher()->ExecuteList(FN_INSERT_STRING, SfxCallMode::SYNCHRON, { &aItem }); pView->GetViewFrame()->GetDispatcher()->Execute(FN_CNTNT_TO_NEXT_FRAME, SfxCallMode::SYNCHRON); + +#if !defined(MACOSX) + // Make sure that the anchor points to the bottom left corner of the image. + // Without the accompanying fix in place, this test would have failed with: + // - Expected less or equal than: 1418 + // - Actual: 2442 + // The anchor pointed to the bottom right corner, so as-char and at-char was inconsistent. + Scheduler::ProcessEventsToIdle(); + SwPostItMgr* pPostItMgr = pView->GetPostItMgr(); + for (const auto& pItem : *pPostItMgr) + { + const SwRect& rAnchor = pItem->pPostIt->GetAnchorRect(); + CPPUNIT_ASSERT_LESSEQUAL(static_cast<long>(1418), rAnchor.Left()); + } +#endif + // Now delete the image. pView->GetViewFrame()->GetDispatcher()->Execute(SID_DELETE, SfxCallMode::SYNCHRON); // Without the accompanying fix in place, this test would have failed with 'Expected: 0; Actual: |