Bug 150518 - Sparkline dialogs have Close button instead of Cancel button
Summary: Sparkline dialogs have Close button instead of Cancel button
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.4.0.3 release
Hardware: All All
: medium trivial
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: needsDevAdvice
Depends on:
Blocks: Sparklines
  Show dependency treegraph
 
Reported: 2022-08-20 14:12 UTC by grofaty
Modified: 2023-01-09 05:53 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
close_button.jpg (87.93 KB, image/jpeg)
2022-08-20 14:20 UTC, grofaty
Details
sample_document.ods (8.31 KB, application/vnd.oasis.opendocument.spreadsheet)
2022-08-20 14:20 UTC, grofaty
Details

Note You need to log in before you can comment on or make changes to this bug.
Description grofaty 2022-08-20 14:12:01 UTC
Description:
New feature in LibreOffice Cals 7.4 is Sparkline. User interface problem. When opening Sparkline dialog there are "Close" and "OK" buttons. The problem is "Close" it behaves like "Cancel" button in other dialogs. It is small inconsistency bug in naming "Close" button, but it would be more appropriate to name it "Cancel" button.

Steps to Reproduce:
1. Type in values in cells:
A1: 5
B1: -6
C1: 7
2. Select cells from A1:C1
3. Right mouse click inside selection.
4. Select Sparkline | Insert Sparkline.
5. New dialog opens in "Output range" field type in D1 and click on OK button.
6. Sparkline appears in D1 cell. Right click on D1 and select Sparkline | Edit Sparkline Group and click on check box in front of Negative points and click on OK button.
7. Right click on D1 cell and select Sparkline | Edit Sparkline Group. Now uncheck the Negative points checkbox. But now click on Close button.
8. Nothing happens. So it looks like Close button is actually functioning as Cancel button. Close button is now inconsistent without other options like right clik on cell and select Delete. Dialog opens with Cancel and OK buttons, which is expected.

Actual Results:
Button is called "Close" button.

Expected Results:
Button is expected to be named "Cancel" button like other similar dialogs already in LibreOffice.


Reproducible: Always


User Profile Reset: No



Additional Info:
Help | About info:

Version: 7.4.0.3 / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: sl-SI (en_US.UTF-8); UI: en-US
Flatpak
Calc: threaded
Comment 1 grofaty 2022-08-20 14:20:01 UTC
Created attachment 181906 [details]
close_button.jpg

Dialog image.
Comment 2 grofaty 2022-08-20 14:20:49 UTC
Created attachment 181907 [details]
sample_document.ods

Sample document you can follow starting step 7.
Comment 3 Stéphane Guillou (stragu) 2022-09-09 17:21:48 UTC
I can see the Close button in:

Version: 7.4.0.3 / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded

It looks like most insertion and formatting dialogs have a "Cancel" button but there are also a few dialogs with a "Close" button. I'm not sure what the logic is behind picking one over the other.

grofaty, can you point to a dialog that applies the changes when the "Close" button is pressed? For example, the Insert > Hyperlink dialog doesn't apply the changes if the Close button is pressed.
Comment 4 Grobe 2023-01-08 15:49:20 UTC
In 7.4.3.2 (Linux Fedora) there is still OK/Close instead of OK/Cancel.
Comment 5 grofaty 2023-01-08 17:28:26 UTC
@Stephane, sorry for late response. Your question: "Can you point to a dialog that applies the changes when the "Close" button is pressed?"

I can't. That is not what I have been claiming. I am trying to say, there are two buttons: Close and Cancel. The effect of Close button in all cases I have described are always the same as Cancel button. If two buttons have the same effect, then I see no reason in one occasion to use Close and in another occasion to use Cancel. My point is, following the simplicity rule, one of the buttons (Close or Cancel) should be removed and only one of them be used. In majority of the programs there is Cancel button and to follow user familarity with user interface, we should follow the Cancel button.

In example, you are referring to the Insert | Hyperlink. If we apply the logic:
1. In URL field type in some address like: www.test.com
2. Now lets follow what do the buttons: Reset, Apply, Close and OK perform.
Reset: it clear all of the data fields. Just like selecting text in the fields and right click and Delete.
Apply: Apply the setting (in this case insert hyperlink into the selected Calc cell) and leave the dialog open.
OK: OK should always!!! mean exactly the same action as Apply (so inserting the hyperlink into the selected cell) and!!! additionally close the dialog.
Close: In this case what happens, program executes Reset (cleaning all of the data in all of the fields) and closes the dialog. This is classical Cancel operation. Why? Because we lose the data we have manually typed into the URL field. Close in this case is incorrectly used, it should be Cancel button. Cancel means lose everything you have set in the dialog and perform no operation in the Calc cells.

When does Close button make sense to be used instead of Cancel button? Close should be used if there is e.g. Delete button inside dialog and after we click the Delete, then action is finished, there is no way to Cancel Delete effect, in this case Close button is appropriate to indicate, we can't change our mind and cancel the option. For example if there is file dialog. We open file dialog, select file and we click on Delete button and so we deleted the file. In this case Cancel button can not be used any more because we have deleted the file and file is irreversible deleted. We can't use Cancel anymore, because we can't reverse the delete file action. The only option in this case would be Close button. The Close button means: There is no other action to be applied or there is no other action to be canceled, the only thing that can be done is close down the dialog.
Comment 6 Stéphane Guillou (stragu) 2023-01-09 05:53:25 UTC
Thank you for the reply! That makes sense. Marking as New, and NeedsDevAdvice for what I assume is an easyhack.

Also in:

Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: ec2f1d73936c9d8cee83c0887170e9ecb8f044ba
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded