dprint / dprint-plugin-markdown

Markdown code formatting plugin for dprint.
MIT License
25 stars 9 forks source link

Various indent issues in markdown lists in quotes #98

Closed lucacasonato closed 4 months ago

lucacasonato commented 7 months ago

Describe the bug

Various indents are added to lists in quotes in front of the quote rather than after. Additionally, line width is not correctly enforced in lists inside of quotes.

https://dprint.dev/playground/#code/HwAgtCAyD2BOCmBbEBLADgZwK7ICbQBs4QMUAXEAQ0XjIBoQBjaAOw3kbNq1it3RQZGKFgHMQ8AuQbtcIfBJRYMiaLgBQoECC6I0xEcP64sLClgoFKAIzjwJFfEQQhElUS0pUpARyyUAOnVNcCg7ZHRsPEJiUgpqWgZmNg4uMh4+ASERcUlpEng5BXglFTUdJH1eQxRjU3NLGzsHeRiXNw8vSl9-IJCIGAQIzBxW5xJyKhp6JlZ2Tm5eSn40QWExCSkZ2VbFZVVcIK0QAAMzgCsMEO0AM2hoa0pYa9Oz9SA/plugin/markdown

Input Code

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod
>   tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod.
>   ```js
>   foobar
>   ```

Expected Output

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod
>   tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod
>   tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod.
>   ```js
>   foobar
>   ```

Actual Output

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod
  > tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.

> - Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod.
  > ```js
  > foobar
  > ```
dsherret commented 4 months ago

Fixed by https://github.com/dprint/dprint-plugin-markdown/pull/94 (I had this pr opened a long time ago and just forgot to merge it)