Bug 158784

Summary:	Enable independent inheritance of “Spacing above” and “Spacing below” for paragraph styles
Product:	LibreOffice	Reporter:	bintoro <kalle>
Component:	Writer	Assignee:	Not Assigned <libreoffice-bugs>
Status:	NEW ---
Severity:	enhancement	CC:	eyalroz1, heiko.tietze, stephane.guillou
Priority:	medium
Version:	Inherited From OOo
Hardware:	All
OS:	All
See Also:	https://bugs.documentfoundation.org/show_bug.cgi?id=88559 https://bugs.documentfoundation.org/show_bug.cgi?id=68263
Whiteboard:
Crash report or crash signature:		Regression By:
Bug Depends on:
Bug Blocks:	107833

Description bintoro 2023-12-19 17:59:52 UTC

Description:
Setting either fo:margin-top or fo:margin-bottom for a paragraph style causes both values to become direct properties of the style. It would be great to be able to set just one and continue to inherit the other.


Steps to Reproduce:
1. Create a style that derives from Body Text.
2. Adjust the value of “Spacing > Below paragraph” for the new style.
3. Adjust the value of “Spacing > Above paragraph” for Body Text.

Actual Results:
The “Above paragraph” spacing for the newly created style remains as it was after step 2. It is no longer inherited from the parent style.

Expected Results:
The “Above paragraph” spacing for the newly created style should reflect the new value set in step 3. Since the property was never explicitly modified on the child style, it should continue to be inherited.


Reproducible: Always


User Profile Reset: No

Additional Info:
More generally, the UI would benefit from a visual indicator of which settings are being inherited and which ones are set (or about to be set!) directly on the style being edited. For example, inherited values could be displayed in parentheses.

Comment 1 Mike Kaganski 2023-12-19 20:10:32 UTC

Related change: commit 7e7b69829db63e64b8aed8d03c6eaed6d8f1a27c (and following commits) split left/right indents, enabling their independent inheritance.

Comment 2 Stéphane Guillou (stragu) 2024-01-03 01:17:18 UTC

Same in OOo 3.3.

Makes sense to me, but let's confirm with UX/Design team.

Comment 3 Heiko Tietze 2024-01-03 09:37:41 UTC

Yes, we treat indentation separately and should do the same for spacing.