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(markdown-renderer): preserve list margin spacing #215

Merged
merged 1 commit into from
May 8, 2024

Conversation

nijel
Copy link
Contributor

@nijel nijel commented Apr 22, 2024

The extra spacing can be before of after leader, so try to preserve it on both positions.

Fixes #213

test/test_markdown_renderer.py Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented May 5, 2024

Coverage Status

coverage: 94.207% (+0.009%) from 94.198%
when pulling 207b675 on nijel:preserve-list-spacing
into 06f2a93 on miyuchina:master.

@pbodnar
Copy link
Collaborator

pbodnar commented May 5, 2024

@nijel, thanks for your PR! I've found some small details and fixed them right away - can you please review my commit and squash it to your original commit if you find it OK?

The extra spacing can be before of after leader, so try to preserve it
on both positions.

Fixes miyuchina#213
@nijel nijel force-pushed the preserve-list-spacing branch from e53b6fd to 207b675 Compare May 6, 2024 06:05
@nijel
Copy link
Contributor Author

nijel commented May 6, 2024

Looks good, I've just squashed that.

@pbodnar pbodnar added this to the 1.4.0 milestone May 8, 2024
@pbodnar pbodnar merged commit 0653065 into miyuchina:master May 8, 2024
9 checks passed
@nijel nijel deleted the preserve-list-spacing branch May 10, 2024 07:50
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.

Keep list item indentation in MarkdownRenderer
3 participants