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

Avoid blockquote when parent style has more indent #7606

Merged
merged 1 commit into from
Oct 10, 2021

Conversation

mbrackeantidot
Copy link
Contributor

When a paragraph has an indentation different from the parent (named)
style, it used to be considered a blockquote. But this only makes sense
when the paragraph has more indentation. When a paragraph has less
indentation than its parent style, it probably shouldn't become a blockquote.
So this merge adds a check for the indentation of the parent style.

When a paragraph has an indentation different from the parent (named)
style, it used to be considered a blockquote. But this only makes sense
when the paragraph has more indentation. So this commit adds a check
for the indentation of the parent style.
@jgm jgm merged commit 0f98cbf into jgm:master Oct 10, 2021
@jgm
Copy link
Owner

jgm commented Oct 10, 2021

Thanks, this is great.

@jgm
Copy link
Owner

jgm commented Nov 2, 2021

@mbrackeantidot - do you think that the issue mentioned here could be due to this change?
https://groups.google.com/d/msgid/pandoc-discuss/d5e31c5d-b3a8-43d9-bc86-d38f11429c6bn%40googlegroups.com

@jgm
Copy link
Owner

jgm commented Nov 2, 2021

I've fixed it

jgm added a commit that referenced this pull request Nov 2, 2021
This fixes a regression introduced in pandoc 2.15 by PR #7606.
Closes #7655.
@mbrackeantidot
Copy link
Contributor Author

mbrackeantidot commented Nov 3, 2021

I see. I probably misinterpreted firstLine and hanging by not considering that they're always positive. Thanks for fixing, @jgm .

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