-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
feat(select): add props to customize toggle icons #27648
Conversation
Run & review this pull request in StackBlitz Codeflow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality works great. Nice job! Left a thread for discussing the prop names.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go once the CI errors are resolved. Nice work!
Issue number: resolves #17248
What is the current behavior?
While the
icon
shadow part allows customization of the existing toggle icon, developers do not have a way to specify a different icon to use entirely.What is the new behavior?
New props
toggleIcon
andexpandedIcon
added. (Design docs are here and here respectively.)Does this introduce a breaking change?
Other information
Docs PR: ionic-team/ionic-docs#2996
Dev build:
7.0.15-dev.11687278023.161b97d8