Skip to content

fix(material/icon): do not copy event attributes - #33929

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:icon-event-attributes
Oct 5, 2026
Merged

crisbeto merged 1 commit into
angular:mainfrom
crisbeto:icon-event-attributes

Conversation

@crisbeto

@crisbeto crisbeto commented Oct 4, 2026

Copy link
Copy Markdown
Member

Adds some logic so that attributes like onclick aren't copied when creating an SVG icon from a symbol.

Adds some logic so that attributes like `onclick` aren't copied when creating an SVG icon from a symbol.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Oct 4, 2026
@pullapprove
pullapprove Bot requested review from andrewseguin and ok7sai October 4, 2026 13:15
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Oct 5, 2026
@crisbeto
crisbeto merged commit cffa9cf into angular:main Oct 5, 2026
33 of 35 checks passed
@crisbeto

crisbeto commented Oct 5, 2026

Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request Oct 5, 2026
Adds some logic so that attributes like `onclick` aren't copied when creating an SVG icon from a symbol.

(cherry picked from commit cffa9cf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: material/icon target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants