-
Notifications
You must be signed in to change notification settings - Fork 630
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
Add <summary>
to default whitelist
#216
Conversation
Since `<details>` is in there, it makes sense for `<summary>` as well since that is used inside `<details>` to define the text label/title for the collapsible element.
@leizongmin Thank you for merging it in 👍 . Do you plan to release a new minor version with the current batch of merges so we can pull in the new version into other projects? |
Yes, I will publish a new version later. |
@leizongmin I noticed that this |
Add `<summary>` to default whitelist
I have published a new version
|
@leizongmin Thank you for the new version! Will directly get on pulling that into Home-Assistant 👍. Could you also mark a release here on GitHub? Then people can easily compare the changes. EDIT: NVM, you just created it 30 seconds ago. Great! |
Since
<details>
is in there, it makes sense for<summary>
as well since that is used inside<details>
to define the text label/title for the collapsible element.See example: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details