Bug 83532 - EDITING ODT: Right (after) indent of paragraph (from paragraph style) does not work in lists
Summary: EDITING ODT: Right (after) indent of paragraph (from paragraph style) does no...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other All
: high normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
: 79109 93336 108718 115787 142412 (view as bug list)
Depends on:
Blocks: Bullet-Number-Outline-Lists Paragraph-Indent
  Show dependency treegraph
 
Reported: 2014-09-05 15:04 UTC by Fabien Cara
Modified: 2023-05-24 16:09 UTC (History)
12 users (show)

See Also:
Crash report or crash signature:


Attachments
Three pages document illustrating the bugs (23.36 KB, application/vnd.oasis.opendocument.text)
2014-09-05 15:04 UTC, Fabien Cara
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fabien Cara 2014-09-05 15:04:21 UTC
Created attachment 105805 [details]
Three pages document illustrating the bugs

This is a two bugs in one description:

When the default style paragraph width is reduced on the right, bulleted lists in default style do not apply this modification.

In the bulleted list, when the right indentation is moved on the top rule with the cursor, the page numbering restarts to one ! This does not happen when the paragraph style is modified according to default paragraph style.
Comment 1 Yousuf Philips (jay) (retired) 2014-09-14 00:21:18 UTC
Hello Fabien,

Thank you for submitting the bug. Regarding the first bug, i think that the bullet should inherit from the already set paragraph width, as that is how it does in ms word. Regarding the second bug, it has previously been reported (bug 79303).

Steps:
1) open writer
2) modify default style by changing the after indent to 2cm
3) add a bullet
4) bullet after indent is set to 0cm
Comment 2 tommy27 2016-04-16 07:27:45 UTC Comment hidden (obsolete)
Comment 3 QA Administrators 2017-05-22 13:25:13 UTC Comment hidden (obsolete)
Comment 4 Fabien Cara 2017-05-22 14:05:14 UTC
Still present on version 5.3.3.2
Build ID: 3d9a8b4b4e538a85e0782bd6c2d430bafe583448
OS :Windows 6.2 (Win10).
Comment 5 Regina Henschel 2017-12-06 16:17:54 UTC
*** Bug 108718 has been marked as a duplicate of this bug. ***
Comment 6 Regina Henschel 2017-12-09 14:54:25 UTC
*** Bug 79109 has been marked as a duplicate of this bug. ***
Comment 7 Regina Henschel 2017-12-09 14:55:33 UTC
*** Bug 93336 has been marked as a duplicate of this bug. ***
Comment 8 Regina Henschel 2017-12-09 14:58:51 UTC
change summary to reflect the problem more exact.
Comment 9 Regina Henschel 2018-03-07 17:35:36 UTC
*** Bug 115787 has been marked as a duplicate of this bug. ***
Comment 10 roumanet 2018-03-07 17:47:47 UTC
Abiword 2.8 and Microsoft Word are ok
Open Office 4 and LibreOffice have a bug
Comment 11 roumanet 2018-05-21 14:30:24 UTC
with Libreoffice 6.0.4.2, we could see that clicking on bullets list remove the "After Text" setting (even if well formatted before).

Going to (right click) ► Paragraph ► Indents & Spacing permit to reformat correctly the bullet paragraph.

Looking at class SwBulletPortion : public SwNumberPortion in porfld.cxx (core/sw/source/core/text)
Not able to go deeper...
Comment 12 Steve Edmonds 2018-06-14 00:36:56 UTC
Going to (right click) ► Paragraph ► Indents & Spacing is a quick fix but removes association with a paragraph style attached to the list, changes to the style no longer reflect in the list.
Comment 13 roumanet 2018-06-14 06:32:26 UTC
@Fabien Cara
In the bulleted list, when the right indentation is moved on the top rule with the cursor, the page numbering restarts to one ! This does not happen when the paragraph style is modified according to default paragraph style.
► Not able to reproduce this. Could you give more details? I've tried with a selectionned bullets list (moving the small vertical arrow cursor to align with the whole text). I'm using LO6.0.4.2

@Steve Edmonds
► Could you give more detail? using Text Body style, then changing format (with reduced right marge), then selecting few text (lorem ipsum) and click on bullet icon, then changing Indents/spacing for bullets and finally changing Text body style again = changing bullets text too.
Comment 14 Steve Edmonds 2018-06-14 09:14:24 UTC
Ok, if I understood correctly this is the sequence I followed.

1.Create some text (it is Default Style), 2.apply Text Body style, 3.Reduce right margin - not the page margin but the paragraph right indent "After Text", 4.Select text and click bullet icon, 5.Change indent for bullets, 6.Adjust right margin - not the page margin but the paragraph right indent "After Text".

The right indent changes in the bulleted text as I expected.

Now if you go into styles (sidebar/F11) and apply the style Text Body to the bulleted text, the right indent is set to 0. Even if "After Text" has a setting in  the "Text Body" style.

Version: 6.0.4.2 by the way.
Comment 15 QA Administrators 2019-06-15 02:59:17 UTC Comment hidden (obsolete)
Comment 16 Fabien Cara 2019-06-15 05:38:34 UTC
I confirm that the bug is still present on version 6.2.4.2
Comment 17 QA Administrators 2021-06-15 03:41:54 UTC Comment hidden (obsolete)
Comment 18 Steve Edmonds 2021-06-15 09:46:50 UTC
As far as I can tell it is still in 7.1.4.2
Comment 19 Dieter 2021-06-20 10:01:55 UTC
*** Bug 142412 has been marked as a duplicate of this bug. ***
Comment 20 Dieter 2021-06-20 10:04:37 UTC
Still present in

Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: de7356c2e0cb099fac396808b5a86a0393b48e5f
CPU threads: 4; OS: Windows 10.0 Build 19043; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: en-GB
Calc: CL

I've changed importance to High Normal, because we have five duplicates.
Comment 21 phv 2021-09-20 14:15:33 UTC
This bug depends on bug #114287:
Paragraphs style settings for fo:text-indent and fo:margin-left has to overrule those from the list style, read section 17.20 in the spec. But LibreOffice ignores the settings in the paragraph style, in case the numbering style is not referenced in the same <style:style> element, which defines the paragraph indents.

And bug #92762 is another duplicate of this one.

Shouldn't we close #92762 and mark this report as a duplicate of #114287?

I agree that this bug affects a major feature of the word processor and should be treated with high priority.
Comment 22 Michael Stahl (allotropia) 2023-02-24 11:37:04 UTC
probably this was fixed for Writer by:

but possibly the same bug is in EditEngine as used by Drawing Shapes/Calc/etc.


commit db115bec9254417ef7a3faf687478fe5424ab378
Author:     Michael Stahl <michael.stahl@allotropia.de>
AuthorDate: Tue Feb 14 18:03:55 2023 +0100
Commit:     Michael Stahl <michael.stahl@allotropia.de>
CommitDate: Fri Feb 24 07:32:23 2023 +0000

    tdf#78510 sw,cui: split SvxLRSpaceItem for SwTextNode, SwTextFormatColl
    
    Leave editengine and non-paragraph usages of SvxLRSpaceItem as-is for now.
    
    Add new items RES_MARGIN_LEFT etc., order them so that paragraphs can
    have 3 consecutive items RES_MARGIN_FIRSTLINE..RES_MARGIN_RIGHT and
    non-paragraphs also have 2-4 consecutive items
    RES_MARGIN_RIGHT..RES_MARGIN_LEFT (only the 3 paragraph ones are
    actually used now).
    
    The HTML import filter is particularly annoying because it parses CSS
    stuff into SfxItemSets without knowing where the items will be applied,
    so it can't know whether to create SvxLeftMarginItem or
    SvxTextLeftMarginItem... the split items are created in ParseCSS1_*
    functions and then converted later if necessary.
    
    WW8 import has some weird code as well, SwWW8ImplReader::Read_LR()
    creates 3 items and then something wants to set every item on its own
    so SwWW8FltControlStack::SetAttrInDoc() turned out rather weird.
    
    Convert the paragraph dialog to handle the split items (by mapping them
    to SID_ATTR_PARA_FIRSTLINESPACE/SID_ATTR_PARA_LEFTSPACE/
    SID_ATTR_PARA_RIGHTSPACE), but the SvxRuler looks a bit more confusing
    so convert in sw shells for now and leave that for later (also unclear if
    changing these slot items like SID_ATTR_PARA_LRSPACE breaks any ABIs?).

    Change-Id: I40431821868fd3e1cceba121b5539ff9ae6befbc
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/147024
Comment 23 Justin L 2023-05-24 16:08:47 UTC
(In reply to Michael Stahl (allotropia) from comment #22)
> probably this was fixed for Writer
No. Repro 7.6+ when creating a simple Writer document from scratch.