Bug 75805

Summary: "Apply" button in "insert" -> "Hyperlink" closes dialog
Product: LibreOffice Reporter: a07cd040897db54e103c
Component: CalcAssignee: Thomas Arnhold <thomas-libo>
Status: RESOLVED FIXED    
Severity: normal CC: cno
Priority: medium    
Version: 4.2.1.1 release   
Hardware: Other   
OS: All   
See Also: https://bugs.freedesktop.org/show_bug.cgi?id=56456
https://bugs.freedesktop.org/show_bug.cgi?id=63271
https://bugs.freedesktop.org/show_bug.cgi?id=75578
https://bugs.freedesktop.org/show_bug.cgi?id=74040
https://bugs.freedesktop.org/show_bug.cgi?id=74588
Whiteboard:
Crash report or crash signature: Regression By:
Attachments: A screenshot of the insert-hyperlink dialog.

Description a07cd040897db54e103c 2014-03-05 16:34:42 UTC
Created attachment 95175 [details]
A screenshot of the insert-hyperlink dialog.

The dialog that appears by clicking "Insert" -> "Hyperlink" is closed, when "Apply" is selected. I think a click on that button should only apply the changes made and *not* close the dialog at all.
Comment 1 Cor Nouws 2014-03-05 17:03:31 UTC

*** This bug has been marked as a duplicate of bug 63271 ***
Comment 2 a07cd040897db54e103c 2014-07-07 11:33:05 UTC
This is *not* a duplicate of bug 63271 it is the complete opposite. Furthermore i feel that "fixing" bug 63271 *introduced* the behaviour i am criticising.

Although 63271 is correctly marked as "RESOLVED WONTFIX" the "patch" seems to be included into the source.

The "Apply" button should *never* close a dialog window - it should only apply/adopt the changes made. And that's exactly how it worked *before* "fixing" bug 63271. Now - when applying the changes also closes the window - it feels broken.

May be an additional button "OK" is needed to solve this discrepancy.

"OK"    - Apply changes AND close the dialog
"Apply" - Only apply the changes
"Close" - Discard changes and close the dialog
Comment 3 penttila 2014-08-11 13:16:31 UTC
See also discussion about bug 56456 (added by felix.stadler in history)
Comment 4 Thomas Arnhold 2014-08-17 11:44:22 UTC
Proposed fix: https://gerrit.libreoffice.org/10946