Skip to content

Fix slot tags uncleaned in HTML String#6948

Merged
matthewp merged 5 commits into
withastro:mainfrom
cyanxiao:fix-astro-slot-uncleaned
May 1, 2023
Merged

Fix slot tags uncleaned in HTML String#6948
matthewp merged 5 commits into
withastro:mainfrom
cyanxiao:fix-astro-slot-uncleaned

Conversation

@cyanxiao
Copy link
Copy Markdown
Contributor

Changes

  • <astro-slot /> as a placeholder for slots should be cleaned in the HTML String that is rendered. Previously, only </astro-slot>s were removed but <astro-slot name="xxx">s were left untouched.
  • Fix astro-slot no closed tag #6373

Testing

Added a test to see if <astro-slot />s stay in the rendered HTML string.

Docs

No docs affected.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 29, 2023

🦋 Changeset detected

Latest commit: dfa6cd7

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the pkg: astro Related to the core `astro` package (scope) label Apr 29, 2023
@cyanxiao cyanxiao changed the title Fix astro slot uncleaned Fix slot tags uncleaned in HTML String Apr 29, 2023
@matthewp matthewp merged commit 50975f2 into withastro:main May 1, 2023
@astrobot-houston astrobot-houston mentioned this pull request May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

astro-slot no closed tag

2 participants