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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: custom tags and attributes #95

Merged
merged 3 commits into from
Jan 29, 2024
Merged

Conversation

deer
Copy link
Contributor

@deer deer commented Jan 22, 2024

closes #11

This was partially done in #25, where they added details, summary, and del tags. Unfortunately they forgot the open attribute for details, which I've fixed here. I've additionally added a test covering these tags, mainly to ensure there is no regression when I switch to the unified ecosystem.

As for the actual issue, I've sadly never heard of most of the tags requested. Additionally, this seems like a game of whac-a-mole. If some new tag comes out or is supported by github, then we have to react. So I've instead moved this forward with a twist (based on #85): users can now supply their own allowedTags and allowedAttributes.

If there is enough desire in the community, then specific tags can of course be added to our default list.

@hashrock, I think this is the last one for today 馃槄

Copy link
Contributor

@hashrock hashrock left a comment

Choose a reason for hiding this comment

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

LGTM!

@hashrock hashrock merged commit b689881 into denoland:main Jan 29, 2024
1 check passed
@deer deer deleted the 11_custom_tags branch January 29, 2024 17:02
@deer deer mentioned this pull request Feb 3, 2024
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.

Whitelist various html5 tags
2 participants