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: Escape HTML tags in tag docstrings so they don't break our docs #90

Merged
merged 3 commits into from
Jun 18, 2024

Conversation

gadenbuie
Copy link
Contributor

@gadenbuie gadenbuie commented Jun 14, 2024

We're using the name and title of the htmltools.tags functions in https://shiny.posit.co/py/api/core/ui.tags.html but they're unprotected HTML, resulting in a broken table.

Fixes posit-dev/py-shiny#1337

This was reported on Discord and then by Randy. Thanks @gregswinehart for the push to find the fix.

I haven't re-generated docs. I'm hoping @schloerke might have the time to carry this fix through to https://shiny.posit.co/py

@gadenbuie gadenbuie requested review from wch and schloerke June 14, 2024 18:48
scripts/generate_tags.py Outdated Show resolved Hide resolved
@schloerke schloerke merged commit 284a7b5 into main Jun 18, 2024
15 checks passed
@schloerke schloerke deleted the gadenbuie-patch-1 branch June 18, 2024 21:07
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.

Docs: Tags in htmltools.tags are appearing as HTML elements rather than code snippets
2 participants