Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extra newline in empty attribute #306

Closed
Nainoia-Inc-Admin opened this issue Jan 17, 2021 · 3 comments
Closed

Extra newline in empty attribute #306

Nainoia-Inc-Admin opened this issue Jan 17, 2021 · 3 comments
Assignees
Labels
Milestone

Comments

@Nainoia-Inc-Admin
Copy link

Nainoia-Inc-Admin commented Jan 17, 2021

Moving from version 4.1.25 to 4.3.1 resulted in significant newline changes in some of my layout. I need help understanding how to correct my layout XML. The layout and two version examples are attached below.

LayoutXMLSnippet.txt

Example as preferred with version 4.1.25.pdf

Example with extra space with version 4.3.1.pdf

@pgundlach
Copy link
Member

Looks like a bug, I'll investigate

@pgundlach pgundlach changed the title Version 4.3.1 result in extra newline. Is this expected? How do I change my layout XML? Extra newline in empty attribute Jan 19, 2021
@pgundlach pgundlach self-assigned this Jan 19, 2021
@pgundlach pgundlach added the Bug label Jan 19, 2021
@pgundlach pgundlach modified the milestones: Version 4.4, 4.3.2 Jan 19, 2021
@pgundlach
Copy link
Member

Version 4.3.1

<data>
    <elt att1="a" att2="" />
    <elt att1="a" att2="" />
    <elt att1="a" att2="" />
</data>
<Layout xmlns="urn:speedata.de:2009/publisher/en"
    xmlns:sd="urn:speedata:2009/publisher/functions/en">

    <Record element="data">
        <PlaceObject>
            <Textblock>
                <Paragraph>
                  <ForAll select="elt">
                    <Value select="@att1"/>
                    <Value select="@att2"/>
                  </ForAll>
                </Paragraph>
            </Textblock>
        </PlaceObject>
    </Record>
</Layout>

Should render as aaa in one line.

@pgundlach
Copy link
Member

This is fixed in version 4.3.2

pgundlach added a commit that referenced this issue Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants