Issue 119401 - suspicious property fo:margin="100%" in paragraph style 'first line indent'
Summary: suspicious property fo:margin="100%" in paragraph style 'first line indent'
Status: CONFIRMED
Alias: None
Product: xml
Classification: Code
Component: code (show other issues)
Version: AOO340-dev
Hardware: PC All
: P3 Normal with 2 votes (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-22 16:06 UTC by Regina Henschel
Modified: 2013-12-03 13:39 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Dummy text with first line indent (20.25 KB, application/vnd.oasis.opendocument.text)
2012-05-22 16:06 UTC, Regina Henschel
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Regina Henschel 2012-05-22 16:06:01 UTC
Created attachment 77578 [details]
Dummy text with first line indent

Unzip the attached document and look into styles.xml. Notice that the page style "First_20_line_20_indent" has a paragraph property fo:margin="100%". This makes no sense, because the reference for 100% is the _width_ of the parent element, here the body. The property fo:margin is a shorthand and will be expanded to the four margin settings for rendering. This results in setting the top and bottom margin to the body width. The left and right margins are overwritten with the immediately following explicit margin-left and margin-right settings.

1. Because margin-left and margin-right are set explicitly no shorthand should be used, but margin-top and margin-bottom should be set explicitly too. The correct values can be found in the paragraph style itself or in the link up chain of paragraph styles or set to default "0cm".

2. The value "100%" should not be used at all, because the reference is the width of the parent element, which never can be the right value for margins.

The error becomes obvious, when you export the document to XHTML. The export filter translates the fo:margin property to the corresponding css, which results in a wrong rendering. But the bug is not in the filter, it is already in the odt-file.

This error might be introduced with the fix for issue 117001. For example in DEV300m70 this error does not exist.
Comment 1 Terry Yang 2012-06-11 09:17:59 UTC
Verify repro on AOO3.4 rev.1327774
Comment 2 Michael.AMES 2013-12-03 13:39:10 UTC
Is there any update for this bug?