diff options
author | Michael Stahl <michael.stahl@allotropia.de> | 2022-07-11 19:20:33 +0200 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2022-07-14 19:45:03 +0200 |
commit | bb8d859f8861d92ea28e217534467ea9312f8c6b (patch) | |
tree | b6d99ad84cce27ad0085b726a91e15922b2c9b7e /sdext/source/pdfimport/wrapper/wrapper.cxx | |
parent | tdf#135682 sw: fix lost selection-all when doc starts with table (diff) | |
download | core-bb8d859f8861d92ea28e217534467ea9312f8c6b.tar.gz core-bb8d859f8861d92ea28e217534467ea9312f8c6b.zip |
sw: fix spurious layout invalidation from ~SwCallLink()
This code was added in commit 56b2cf0c10d9caa01ebae1d80465e342d046a85c
"sw-collapse-empty-table-par-like-html.diff" and leaves us to guess
what it should do.
Apparently it's trying to replicate a Word feature where an empty
paragraph at the end of a table cell and preceded by a nested table is
effectively hidden, *unless* the user moves the cursor into it, at which
point it grows to its ordinary height.
The problem is that this is implemented by invalidating the position of
any table once the cursor is moved into it, causing it to be
reformatted, at potentially great expense, regardless if the cursor is
actually on a paragraph that may be hidden.
Also limit invalidations to when the cursor has actually moved to a
different node.
To fix tdf#105330, un-collapse a paragraph after Undo, requires
additionally doing the same notification from
SwUndoInsTable::UndoImpl() because the SwCallLink won't see the removed
table any more.
Mysteriously this causes the test SwLayoutWriter2 testTdf124261 to fail
in Jenkins, but only on MacOSX so impossible to debug; tb86 fails with:
layout2.cxx:2227:testTdf124261::TestBody
equality assertion failed
- Expected: 1721
- Actual : 5437
... and tb84 with:
layout2.cxx:2227:testTdf124261::TestBody
equality assertion failed
- Expected: 1721
- Actual : 3740
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136963
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
(cherry picked from commit c605283ad6785dea762feab5fdffd9d27e75c292)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137034
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
(cherry picked from commit dadaf930d14283f96cc06741d2eec6d846e59f7f)
(cherry picked from commit 158012bcd3ed56cd1c9c5b3472d42d94d5a437da)
Change-Id: Ifd55097735d3675e6b82264f455baa44e9c9e30a
Diffstat (limited to 'sdext/source/pdfimport/wrapper/wrapper.cxx')
0 files changed, 0 insertions, 0 deletions