Bug 114631 - Calc and Impress UI - putting checkmarks on multiple entries in 'Spacing' and 'Align' menus possible
Summary: Calc and Impress UI - putting checkmarks on multiple entries in 'Spacing' and...
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
(earliest affected) release
Hardware: All All
: medium normal
Assignee: Andreas Heinisch
Keywords: bibisected, bisected, implementationError, needsDevEval, topicUI
Depends on:
Blocks: Main-Menu
  Show dependency treegraph
Reported: 2017-12-21 17:53 UTC by Mike
Modified: 2021-11-26 21:23 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Screenshots (627.55 KB, application/pdf)
2017-12-21 17:58 UTC, Mike

Note You need to log in before you can comment on or make changes to this bug.
Description Mike 2017-12-21 17:53:11 UTC
If you change in Impress the line spacing via clicking on a preset in the format menu (like “1.5”), the original setting (“1”) remains checked. The result is that both settings are checked in the menu. Only one would be correct.

The same issue also appears in ‘Format’ → ‘Align’. Impress has this issue since the menu was introduced in LO 5.0 Writer has not the issue, even though it has the same menus (‘Spacing’  and ‘Align’). In LO 5.0 are these menus already implemented in Impress, but not in Writer. Writer has them since LO 5.1, but no issue with them since then.

Steps to Reproduce:
1. Start Impress
2. Choose any template
3. Click on the slide
4. Enter a word (like ‚Lorem‘)
5. Press Enter
6. Enter another word (like ‚Ipsum‘)
7. Mark both lines
8. Open menu format → spacing
Setting should be „Line Spacing: 1“
9. Choose from menu ‚Line Spacing: 1.5“
Spacing changes to 1.5 lines.
10. Open menu format → spacing
11. Now both settings (1 & 1.5) are marked.
12. Choose ‚Line Spacing: 2‘
Spacing changes to 2 lines.
13. All three spacing options are marked: 1, 1.5 and 2

Same with 'Align'

Actual Results:  
It is possible to put checkmarks on multiple entries in 'Spacing' and 'Align' menu. Like 'Line Spacing: 1" AND 'Line Spacing: 2'. 

Expected Results:
If I choose a different setting, then the checkmark from the original setting should disappear.

Reproducible: Always

User Profile Reset: Yes

OpenGL enabled: Yes

Additional Info:
I searched for “impress menu spacing”, but found no dupe.

Version: (x64)
Build ID: 9e7caee0173c5ec2d8b5cfa6aedfd1c9674d442b
CPU threads: 4; OS: Windows 6.1; UI render: default; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2017-12-19_22:55:34
Locale: de-DE (de_DE); Calc: group threaded

I could also reproduce it in Ubuntu

User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36 Edge/16.16299
Comment 1 Mike 2017-12-21 17:58:22 UTC
Created attachment 138576 [details]
Comment 2 Aron Budea 2017-12-21 22:10:37 UTC
Confirmed using LO / Windows 7.
Comment 3 Mike 2018-01-27 19:05:17 UTC
Polite ping: Could somebody check if Calc is also affected? :-) (I don't know Calc. Issue affects Impress, but not Writer.)
Comment 4 Aron Budea 2018-01-27 22:07:22 UTC
Yes, it affects Calc as well, eg. you can set paragraph spacing in comments.

I dug up the commits that added these entries to the menubar, but I have no idea how toggling works, and where that can be specified.
Looks like a good easy hack candidate, adding the respective keywords.

For reference, these commits add the menu entries to the menus:
tdf#91781 Reorganize writer's menu bar

rearrange Impress's format menu bar
Comment 5 Yousuf Philips (jay) (retired) 2018-01-28 16:12:21 UTC
Likely ' menu:style="radio"' need to be added to these menu items, like they are there in writer

Comment 6 Aron Budea 2018-01-28 19:22:12 UTC
Thanks for the hint Jay! The question is, why isn't it there for the line spacing menu entries?
Comment 7 QA Administrators 2019-01-29 03:48:23 UTC Comment hidden (obsolete)
Comment 8 Aron Budea 2019-08-24 23:51:04 UTC
The Impress parts are fixed by the following commits.
In Calc/Writer comments, still multiple spacings get selected ( / d3d13140f0036c53aa74820b41acfeffa3572168).

author		Ilhan Yesil <ilhanyesil@gmx.de>	2018-06-08 10:49:41 +0200
committer	Thorsten Behrens <Thorsten.Behrens@CIB.de>	2018-06-12 22:14:21 +0200
tdf#118063 Impress: Deselected spacing remained checked

author		Ilhan Yesil <ilhanyesil@gmx.de>	2018-05-28 11:05:30 +0200
committer	Katarina Behrens <Katarina.Behrens@cib.de>	2018-05-29 09:45:50 +0200
tdf#117847 Impress: Deselected alignment remained checked
Comment 9 QA Administrators 2021-08-24 03:48:06 UTC Comment hidden (obsolete)
Comment 10 Andreas Heinisch 2021-11-26 11:09:32 UTC
Hi! I fixed the line spacing issue for comments in Writer. However, I could not find the menu item for the line spacing in Calc. Maybe someone of you could enlight me :)
Comment 11 Aron Budea 2021-11-26 13:28:11 UTC
Hi Andreas! Thanks for that, I checked now, and these menu items have been removed by the following commit for bug 136071. So, this can be closed (please add a reference to the change that fixed this in Writer).

Comment 12 Andreas Heinisch 2021-11-26 21:23:45 UTC
Andreas Heinisch committed a patch related to this issue.
It has been pushed to "master":


tdf#114631 - Writer: Deselected line spacing items remained checked

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:

Affected users are encouraged to test the fix and report feedback.