Skip to content

Conversation

@Bitshifter-9
Copy link

@Bitshifter-9 Bitshifter-9 commented Nov 17, 2025

Added transformHtmlListIndent to preserve list indentation inside raw HTML blocks in slide Markdown, preventing Vue compile errors caused by Prettier removing spaces.

Integrated the transformer early in the Markdown pipeline.

Added regression tests and a snapshot for the

+ list scenario to ensure future safety.
this closes issue #2337

@netlify
Copy link

netlify bot commented Nov 17, 2025

Deploy Preview for slidev ready!

Name Link
🔨 Latest commit 69a4ab5
🔍 Latest deploy log https://app.netlify.com/projects/slidev/deploys/691bfb9807689800080d6a1d
😎 Deploy Preview https://deploy-preview-2367--slidev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kermanx kermanx changed the title Fix list indentation inside raw HTML blocks fix: list indentation inside raw HTML blocks Nov 18, 2025
This adds a transformer that ensures lists inside HTML blocks maintain
proper indentation to prevent Vue parse errors when prettier-plugin-slidev
removes the required indentation.

Note: This is a workaround. The proper fix should be in prettier-plugin-slidev
to preserve indentation during formatting. This transformer ensures
compatibility even when Prettier removes the required indentation.

Fixes slidevjs#2337
Skip processing lists inside Vue components (v-*, PascalCase, template)
to prevent breaking v-clicks and other Vue component functionality.

Fixes Cypress test failure for deep nested lists.
The pkg-pr-new GitHub app is only installed on the upstream repository,
not on forks. This causes CI to fail on forks. Skip this step unless
running on the upstream repository.
@Bitshifter-9 Bitshifter-9 changed the title fix: list indentation inside raw HTML blocks fix: list indentation inside raw HTML blocks #2337 Nov 18, 2025
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.

1 participant