-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
ui: Adds ability to show a 'partial' list in list-collections #10174
Conversation
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.
Very cool 😎 LGTM
67443f6
to
05ba731
Compare
🍒 If backport labels were added before merging, cherry-picking will start automatically. To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/366012. |
🍒✅ Cherry pick of commit 0e4db3f onto |
* ui: Add support for showing partial lists in ListCollection * Add CSS for partial 'View more' button, and move all CSS to /components * Enable partial view for intention permissions
🍒✅ Cherry pick of commit 0e4db3f onto |
* ui: Add support for showing partial lists in ListCollection * Add CSS for partial 'View more' button, and move all CSS to /components * Enable partial view for intention permissions
We have a component that we use quite a lot called
<ListCollection />
, we use it for our 'virtual/DOM-recycling scrolling' and just for normal 'native scrolling' lists.This PR adds a
@partial
attribute to this component, letting you specify that you only want a certain number of rows to show initially. When this attribute is set, we also render a 'View more' button to let the user expand the list to show all the rows instead of just a 'partial' amount.We also took the opportunity here to move the
list-collection
styles into theapp/components/list-collection
folder, so everything is co-located in the one folder.