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): set some elements as slot to promote customization #63

Merged
merged 4 commits into from
Dec 13, 2022

Conversation

gretchelin
Copy link
Collaborator

@gretchelin gretchelin commented Dec 12, 2022

This PR exposes selection, max-selection, select-all and option item label content as slot to accommodate custom rendering needs. Also created two slot position in option to enable prepend and appending custom to item list.

Storybook story and vuepress docs for this component has been updated to reflect these changes.

…ser customization

docs(multiselect): add new  story for new selection and max selection in storybook
docs(multiselect): add new slots in vuepress docs
@gretchelin gretchelin force-pushed the feat/multiselect-slots branch from 7ebc761 to 6434a28 Compare December 12, 2022 05:12
@gretchelin gretchelin marked this pull request as ready for review December 12, 2022 06:12
docs(multiselect): add new story for select all slot
docs(multiselect): add select all slot to vuepress docs entry
@gretchelin gretchelin force-pushed the feat/multiselect-slots branch from 2fd336f to b3028d2 Compare December 12, 2022 07:50
docs(multiselect): add story for append.item and prepend.item slot
docs(multiselect): add append.tem and prepend.item slot to vuepress entry
…omization

docs(multiselect): add new story for option label slot
docs(multiselect): add option label slot in vuepress doc entry
@gretchelin gretchelin changed the title feat(multiselect): selection and max-selection slot feat(multiselect): set some elements as slot to promote customization Dec 13, 2022
@gretchelin
Copy link
Collaborator Author

@gravitano might want to check into slot naming. I didn't see any rule about slot naming for this project, so idk if the slot names I set up in this PR are okay or should be changed to a specific pattern.

@gravitano gravitano merged commit f8b8456 into gitsindonesia:main Dec 13, 2022
@gretchelin gretchelin deleted the feat/multiselect-slots branch December 13, 2022 23:15
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.

2 participants