Bug 158784 - Enable independent inheritance of “Spacing above” and “Spacing below” for paragraph styles
Summary: Enable independent inheritance of “Spacing above” and “Spacing below” for par...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Writer-Styles-Paragraph
  Show dependency treegraph
 
Reported: 2023-12-19 17:59 UTC by bintoro
Modified: 2024-02-01 15:30 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.