| commit | bb47ffbc9d36e83695aa0d01767d3f83533c04e0 | |
| author | Justin Luth <justin.luth@collabora.com> | |
| Thu, 22 Sep 2022 14:33:43 +0000 (22 10:33 -0400) | ||
| committer | Justin Luth <jluth@mail.com> | |
| Fri, 23 Sep 2022 14:05:08 +0000 (23 16:05 +0200) | ||
| tree | 8635f83de8d327a5eb0fc60f3e54d2636bbf6738 | treesnapshot (tar.gz zip) |
| parent | 03a970babc2df4957051af996d8a426e4d94f17d | commitdiff |
tdf#46733 sw: don't double-select column during deleteCol
The problem here was that under IsNewModel tables were
selecting the entire column (including merged cells)
and then using THAT selection to ExpandColumnSelection.
Well, that ends in disaster. Since NewModel already
expands to catch the cells that are contained inside
the selected cells, there is no need to expand
the selection of GetTableCell.
Change-Id: I43089d912cdd17b2599619a43e40fecc2f5dd3b5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140455
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
The problem here was that under IsNewModel tables were
selecting the entire column (including merged cells)
and then using THAT selection to ExpandColumnSelection.
Well, that ends in disaster. Since NewModel already
expands to catch the cells that are contained inside
the selected cells, there is no need to expand
the selection of GetTableCell.
Change-Id: I43089d912cdd17b2599619a43e40fecc2f5dd3b5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140455
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
| sw/source/core/frmedt/fetab.cxx | diffblobblamehistory |