Skip to content

fix render of youtube links and images in docs pages#11105

Merged
riknoll merged 1 commit into
masterfrom
dev/riknoll/arcade-fix-blog-youtube-img-render
Feb 12, 2026
Merged

fix render of youtube links and images in docs pages#11105
riknoll merged 1 commit into
masterfrom
dev/riknoll/arcade-fix-blog-youtube-img-render

Conversation

@riknoll
Copy link
Copy Markdown
Member

@riknoll riknoll commented Feb 12, 2026

fixing a couple of regressions caused by the recent marked/sanitize-html upgrades

we embed youtube videos for youtube links that are alone on a line in a markdown page. the marked upgrade broke this behavior because it was generating slightly different html that tripped up our big macro regex that replaces the generated html with the thumbnail embed

sanitize-html was also stripping all images from the generated html because img isn't in the default list of allowed tags

@ganicke FYI!

@riknoll riknoll requested a review from a team February 12, 2026 00:18
Comment thread .vscode/launch.json
"outFiles": []
},
{
"name": "pxt serve (core)",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these launch changes intentionally included?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup, we didn't have one for debugging core docs

@riknoll riknoll enabled auto-merge (squash) February 12, 2026 00:34
@riknoll riknoll merged commit 8e4569c into master Feb 12, 2026
20 checks passed
@riknoll riknoll deleted the dev/riknoll/arcade-fix-blog-youtube-img-render branch February 12, 2026 00:39
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