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

Support attributes on hier, BULLETS and ITEMS, fixes #91

Merged
merged 8 commits into from
Mar 22, 2022
Merged

Conversation

longhotsummer
Copy link
Contributor

@longhotsummer longhotsummer commented Mar 18, 2022

  • support block attributes for BULLETS and ITEMS
  • ensure empty body documents are valid xml
  • fix hcontainers for crossheadings at the top level

@longhotsummer longhotsummer changed the title Support attributes on hier, BLOCKLIST and ITEMS Support attributes on hier, BLOCKLIST and ITEMS, fixes Mar 18, 2022
@longhotsummer longhotsummer changed the title Support attributes on hier, BLOCKLIST and ITEMS, fixes Support attributes on hier, BULLETS and ITEMS, fixes Mar 18, 2022
@longhotsummer
Copy link
Contributor Author

Closes #61

@longhotsummer longhotsummer marked this pull request as draft March 22, 2022 07:59
@longhotsummer
Copy link
Contributor Author

crossHeadings are still broken. Working on it.

@longhotsummer longhotsummer marked this pull request as ready for review March 22, 2022 08:30
@longhotsummer
Copy link
Contributor Author

Crossheadings fixed

<body>
<section eId="hierarchicalStructure_1__sec_nn_1">
<crossHeading eId="hierarchicalStructure_1__sec_nn_1__crossHeading_1">crossheading one</crossHeading>
<wrapUp>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oof I don't like this! 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:D What's the alternative? We can also determine that the section doesn't have any "real" content before the blockList, so the blockList then goes into an hcontainer? That's not particularly pretty either.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah it is what it is! :)

goose-life
goose-life previously approved these changes Mar 22, 2022
@longhotsummer longhotsummer merged commit 47d1e09 into master Mar 22, 2022
@longhotsummer longhotsummer deleted the attribs branch March 22, 2022 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants