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

Fix #5 #8

Merged
merged 4 commits into from Aug 13, 2021
Merged

Fix #5 #8

merged 4 commits into from Aug 13, 2021

Conversation

tdemin
Copy link
Owner

@tdemin tdemin commented Aug 13, 2021

While renderer previously assumed there would always be a single paragraph inside the blockquote, there sometimes can be either more or none.

Test case:

> Testing text.
> Another line of testing text.

> This would get eaten by the renderer.

Test of a blank quote.

>

While renderer previously assumed there would always be a single
paragraph inside the blockquote, there sometimes can be either more or
none.

See #5.
@tdemin tdemin added this to the 0.2.0 milestone Aug 13, 2021
@tdemin tdemin self-assigned this Aug 13, 2021
@tdemin tdemin added the bug Something isn't working label Aug 13, 2021
This removes some of the leftovers of older logic in cmd/gmnhg.
Somehow the config managed to stay in .gitignore a while, causing
CI checks to fail.
This makes the renderer preserve existing line breaks in blockquotes,
provided they do not split paragraphs. Some clients/sites may use this
to form semantic around line breaks (for instance, for poems).

See #5.
@tdemin
Copy link
Owner Author

tdemin commented Aug 13, 2021

Testing file after fixes:

Test run of md2gmn after #8 applied

@tdemin tdemin linked an issue Aug 13, 2021 that may be closed by this pull request
@tdemin tdemin changed the title WIP: Fix #5 Fix #5 Aug 13, 2021
@tdemin tdemin merged commit aadb650 into master Aug 13, 2021
@tdemin tdemin deleted the bugfix/blockquote branch August 13, 2021 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blockquotes
1 participant