-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update action list to v3.10.0 #285
Conversation
* - badge (not currently supported) | ||
* - onAction | ||
* | ||
* @property item |
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.
Needed to remove this and pass individual properties through to support the new role
property passed in from polaris-action-list/section
items=(array | ||
(hash | ||
text="I'm helpful" | ||
helpText="Yay I'm helping!" |
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.
Yay!
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.
💯
Updates
polaris-action-list
and child components to match Polaris v3.10.0. I haven't added the new scroll functionality since we don't currently need it anywhere and don't have an implementation of the PolarisScrollable
component that we'd need for it anyway.