Bug 158979 - Remember last option at Merge Cells
Summary: Remember last option at Merge Cells
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.3.0.3 release
Hardware: All All
: medium enhancement
Assignee: K Stefun
URL: https://ask.libreoffice.org/t/how-do-...
Whiteboard: target:25.2.0
Keywords: difficultyBeginner, easyHack, skillCpp
Depends on:
Blocks: Calc-Merge-Split Dialog-Remember-Settings
  Show dependency treegraph
 
Reported: 2024-01-02 16:48 UTC by Tracey
Modified: 2024-12-21 00:53 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
sample file (12.24 KB, image/png)
2024-01-02 16:48 UTC, Tracey
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tracey 2024-01-02 16:48:28 UTC
Created attachment 191703 [details]
sample file

Every time I merge cells I have to choose the first option.

add a default option to Merge Cells for one of the three choices.

Thanks, Tracey
Comment 1 Stéphane Guillou (stragu) 2024-03-27 00:43:40 UTC
There's a few ways this could be envisaged (not necessarily exclusive to each other):
- remember last setting (per session or in profile)
- "set as default" option in the dialog
- option in Options > Calc
- only expert configuration to change it app-wide
- giving access to new Uno commands so they can be added to toolbars or menus by user

I'll let the UX/design team decide what is best here.

See also bug 147837, in which a change of default / an indicator of hidden cells is discussed.
Comment 2 Stéphane Guillou (stragu) 2024-03-27 00:47:06 UTC
(We've got the 3 options since LO 5.3)
Comment 3 Heiko Tietze 2024-04-03 09:09:37 UTC
KISS: remember the last option (per session)

Code pointer: 

sc/source/ui/miscdlgs/mergecellsdialog.cxx 
just add a static variable to initialize one of the three radio boxes
Comment 4 Commit Notification 2024-12-04 10:08:21 UTC
anonymotter committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/913854864096605bcea23a6174834fb5750ddad0

tdf#158979 Add static variable for last used merge option

It will be available in 25.2.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 5 Long Meza 2024-12-20 07:33:29 UTC Comment hidden (spam)
Comment 6 Stéphane Guillou (stragu) 2024-12-20 19:40:44 UTC
Verified the fix with own master build:

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

anonymotter, I think you can mark this report as "resolved - fixed". Unless you had something else to commit?