Bug 163725 - "Delete Columns" and "Delete Rows" labels now include gratuitous comment "(for uprotected cells)"
Summary: "Delete Columns" and "Delete Rows" labels now include gratuitous comment "(fo...
Status: RESOLVED MOVED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
24.8.2.1 release
Hardware: All All
: medium minor
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Cell-Add-Delete
  Show dependency treegraph
 
Reported: 2024-11-01 22:44 UTC by Eyal Rozenberg
Modified: 2024-11-15 13:17 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Eyal Rozenberg 2024-11-01 22:44:58 UTC
At some point in the recent past, the labels for the "Delete Columns" and "Delete Rows" commands was changed to: "Delete Columns (for uprotected cells)" and "Delete Rows (for uprotected cells)".

This is a bad change. First, usually, no cells are protected at all in the sheet, so the caveat is irrelevant.

Second, when the user is told that columns/rows may not be deleted, s/he is led to believe that there actually _are_ protected cells which s/he needs to do something about - which is not the case.

Third, this makes the labels very long, which makes context menus very wide and the Sheet menu on the main menu bar wider than it could have been - this, while it is doubtful that a user would assume deletion works on protected cells - as they are protected.
Comment 1 Heiko Tietze 2024-11-04 10:28:39 UTC
This was decided for bug 136003.
Comment 2 Eyal Rozenberg 2024-11-13 23:31:48 UTC
(In reply to Heiko Tietze from comment #1)
> This was decided for bug 136003.

Commented there as well. We can't widen the menu for everyone and have menu items with "fine print", just to put a band-aid on some bug.
Comment 3 Eyal Rozenberg 2024-11-13 23:38:31 UTC
And to be more specific regarding UX:

* The vast majority of users rarely protect and unprotect cells.
* In fact, most users probably _never_ protect or unprotect any cells, ever, and may not have even taken the time to understand what that means.
* As the user, I want to delete _these_ cells, that I've selected. WTF is the deal with "for unprotected cells"? Is LibreOffice sending its lawyer to talk to me? If these cells are protected and can't be deleted, why isn't this grayed out? And if they aren't protected, why is the app adding fine print to the command name? "Delete cells, maybe, under certain conditions, and some of them, possibly." Ugh.
Comment 4 Cor Nouws 2024-11-14 10:45:34 UTC
(In reply to Eyal Rozenberg from comment #2)
> Commented there as well.
metoo
Comment 5 Heiko Tietze 2024-11-15 13:17:30 UTC
We discussed the topic in the design meeting and agreed to revert the patch and do have the information at the protect sheet dialog. Reopening the other ticket...