Skip to content

fix: allow 'expanded' attribute in Purifier#40501

Merged
erwinmombay merged 5 commits intoampproject:mainfrom
erwinmombay:fix/accordion-expanded-mustache
Apr 9, 2026
Merged

fix: allow 'expanded' attribute in Purifier#40501
erwinmombay merged 5 commits intoampproject:mainfrom
erwinmombay:fix/accordion-expanded-mustache

Conversation

@erwinmombay
Copy link
Copy Markdown
Member

@erwinmombay erwinmombay commented Apr 7, 2026

Allow 'expanded' to be preserved during HTML sanitization. This is needed for amp-accordion

erwinmombay and others added 5 commits April 6, 2026 17:04
Allow 'expanded' to be preserved during HTML sanitization. This is needed for amp-accordion (and Bento accordion) when rendered inside templates like amp-mustache, where state attributes on standard tags (like <section>) were being stripped.

Includes test case for standard elements.
Allow 'expanded' to be preserved during HTML sanitization. This is needed for amp-accordion (and Bento accordion) when rendered inside templates like amp-mustache, where state attributes on standard tags (like <section>) were being stripped.

Includes test case for standard elements.
@erwinmombay
Copy link
Copy Markdown
Member Author

@danielrozenberg could you review this. thanks!

@erwinmombay
Copy link
Copy Markdown
Member Author

need to do a manual test before merging

@erwinmombay
Copy link
Copy Markdown
Member Author

verified the fix

@erwinmombay erwinmombay merged commit 44604cf into ampproject:main Apr 9, 2026
52 checks passed
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