Bug 141812 - Delete table row in impress defocus (so you can't delete second row)
Summary: Delete table row in impress defocus (so you can't delete second row)
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium normal
Assignee: Caolán McNamara
URL:
Whiteboard: target:7.4.0
Keywords:
Depends on:
Blocks:
 
Reported: 2021-04-21 18:01 UTC by Telesto
Modified: 2022-04-14 14:17 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2021-04-21 18:01:29 UTC 
Description:
Delete table row in impress defocus (so you can't delete second row)

Steps to Reproduce:
1. Open Impress
2. Insert 2x3 table
3. Put cursor in the middle row
4. Press delete row in the toolbar 

Actual Results:
Row deleted blinking border

Expected Results:
Table still focused on a new row (so you can press delete row again, similar as in Writer)


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.2.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: a809b2ab2553e946431699d9d7ac3f6209cbdd6b
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 1 Telesto 2021-04-21 18:02:03 UTC 
Also in
Version: 6.1.0.0.beta2+
Build ID: 22c451df33b733440f24c1feb6380d31240d55e6
CPU threads: 4; OS: Windows 6.3; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL
Comment 2 Telesto 2021-04-21 18:03:49 UTC 
And in
Versie: 4.4.7.2 
Build ID: f3153a8b245191196a4b6b9abd1d0da16eead600
Locale: nl_NL

and in
Versie: 4.1.0.4 
Build ID: 89ea49ddacd9aa532507cbf852f2bb22b1ace28

and in
LibreOffice 3.5.7.2 
Build ID: 3215f89-f603614-ab984f2-7348103-1225a5b
Comment 3 Oksana Ivanova 2021-07-31 23:10:36 UTC 
confirm in 

Win 7

Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: ac80ec817eb07c77a51bc0729985a473c734182e
CPU threads: 8; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: Skia/Raster; VCL: win
Locale: en-US (ru_RU); UI: en-US
Calc: CL
Comment 4 Telesto 2022-04-11 10:42:04 UTC 
repro
Version: 7.4.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: efe854bf9b6daff3d0ecf6e3d04bd9a50bfaa3f3
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL Jumbo
Comment 5 Telesto 2022-04-11 10:46:02 UTC 
@Caolan
A bit opportunistic, so feel free to ignore.. But I personally see this as some  basic functionality and likely not to hard to solve.
Comment 6 Caolán McNamara 2022-04-14 10:25:08 UTC 
https://gerrit.libreoffice.org/c/core/+/133008 looks promising
Comment 7 Commit Notification 2022-04-14 14:11:08 UTC 
Caolán McNamara committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/ae0e7e3918284c31a91acca0f733919926ae3a62

tdf#141812 keep focus in an impress table cell when row/col deleted

It will be available in 7.4.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 8 Caolán McNamara 2022-04-14 14:17:17 UTC 
seems to basically work and the above commit shows the location of the svx table stuff for any potential further adjustment