Skip to content
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(MultiSelect): new option to display title of the selected items #5886

Closed

Conversation

TiagoCunhaF
Copy link

This PR creates a new option that allows the title of the selected items to be displayed and not just the counter as it is currently.

It was also added the option to display the counter together with the titles because when many items are selected, it may not be possible for all of them to be displayed, in which case scrolling is possible.

This proposed change was made thinking about the usability of the user who will see which items were selected even after closing the selection list, and also allows any of the items to be deselected.

 showSelectedText: PropTypes.bool -> If true, selected itens is rendered in title

image

 showSelectedCounter: PropTypes.bool -> If true, show selected itens counter. Default is true.

image

ezgif com-video-to-gif

Any suggestions will be highly appreciated.

@TiagoCunhaF TiagoCunhaF requested a review from a team as a code owner April 19, 2020 08:13
@ghost ghost requested review from emyarod and joshblack April 19, 2020 08:13
@netlify
Copy link

netlify bot commented Apr 19, 2020

Deploy preview for carbon-elements ready!

Built with commit da33199

https://deploy-preview-5886--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Apr 19, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit da33199

https://deploy-preview-5886--carbon-components-react.netlify.app

@netlify
Copy link

netlify bot commented Apr 19, 2020

Deploy preview for carbon-elements ready!

Built with commit 2429b05

https://deploy-preview-5886--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Apr 19, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 2429b05

https://deploy-preview-5886--carbon-components-react.netlify.app

@TiagoCunhaF TiagoCunhaF requested a review from a team as a code owner April 19, 2020 09:18
@rafaelmaiach
Copy link

I really liked it! It would be a great addition to components

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC we decided not to pursue this enhancement, but we may need to reexamine this proposal

ref #2422 #2571 #2640

@asudoh
Copy link
Contributor

asudoh commented Apr 21, 2020

Thank you for the link @emyarod! Closing for now and let's continue the discussion at #2571. As #2571 states, the interest is in seeing the UX that works well with many selections.

@asudoh asudoh closed this Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants