Skip to content

Conversation

@goneall
Copy link
Member

@goneall goneall commented Aug 25, 2022

This was not caught previously due to an issue with the LicenseListPublisher.

This was not caught previously due to an issue with the LicenseListPublisher.
@jlovejoy
Copy link
Member

@goneall - I noticed this tag, - which I don't think I've seen before. what is the spacing="none" for?

@goneall
Copy link
Member Author

goneall commented Aug 26, 2022

@jlovejoy

what is the spacing="none" for?

this is just to control the display - the default is to put a space before and after optional text. If you use spacing="none" it will remove any spacing.

We have to use this type of attribute/tagging since XML kind of ignores most white space.

BTW - I didn't modify the spacing - just adding a missing word for this PR.

@jlovejoy jlovejoy added this to the 3.19 (documentation) milestone Sep 1, 2022
@jlovejoy jlovejoy added the minor updates to file URL update, notes update, etc. label Sep 1, 2022
@jlovejoy jlovejoy merged commit 51e45b9 into main Sep 1, 2022
@jlovejoy jlovejoy deleted the goneall-patch-3 branch September 1, 2022 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor updates to file URL update, notes update, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants