-
Notifications
You must be signed in to change notification settings - Fork 20
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(pipelines): add support for collapsible groups #162
Conversation
I'll pull this down and take a look. Thanks for tagging me. |
6a9f246
to
f49c3c5
Compare
packages/module/src/pipelines/components/groups/PipelinesDefaultGroup.tsx
Outdated
Show resolved
Hide resolved
packages/module/src/pipelines/components/groups/PipelinesDefaultGroup.tsx
Outdated
Show resolved
Hide resolved
styling wip update classnames wip add proper styling, wip hover clean up unused vars, add drop shadow to action icon remove css class to fix build fix :hot build add correct styling to collapsible groups revert action label styling to default, adds isIconExternal prop add collapsible groups to subgroups layout add ref to revert to default label styling add withSelection, update padding clean up css use PF vars for css styles updates per Jeff, WIP remove data from group nodes PR feedback from Jeff: add correct params to PipelinesDefaultGroupExpanded, wrap label with layer add props to fix build fix label position remove custom css in favor of existing pf styles PR feedback from Jeff, apply pill styling to collapsed group fix build
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.
LGTM, thanks!
🎉 This PR is included in version 5.3.0-prerelease.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What
Will close #153.
WIP / help wanted:
Recursive selection of subgroups when parent group is selected☑️ Discussed with design, selection to be replaced with expand action animation to indicate expanded state
Draw location and edge anchor points of collapsed pipeline group☑️ Fixed by @jeff-phillips-18
Description
Adds support for collapsible pipelines groups and related styling as discussed with @andrew-ronaldson and @jeff-phillips-18
Type of change
Surge preview:
https://expand-collapse.surge.sh/