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

fix(refs T35139): Allow dynamically use slots available in vue-multiselect #629

Merged
merged 6 commits into from
Nov 7, 2023

Conversation

salisdemos
Copy link
Contributor

@salisdemos salisdemos commented Nov 7, 2023

If we have to / want to use a specific slot in the multiselect, it wasn't possible by now.
with this changes, we can define any slot that is available in vue-multiselect (https://vue-multiselect.js.org/#sub-slots) to use it.

To test this, You have to add a slot (like singleLabel) in some occurence, where You use dp-multiselect. The Content should be rendered.

Ticket https://yaits.demos-deutschland.de/T35139

package.json Outdated
@@ -98,7 +98,7 @@
"react-dom": "^18.2.0",
"storybook": "7.5.3",
"storybook-addon-vue-slots": "^0.9.21",
"style-dictionary": "^3.7.1",
"style-dictionary": "3.8.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

should it be part of this PR?

Copy link
Contributor Author

@salisdemos salisdemos Nov 7, 2023

Choose a reason for hiding this comment

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

no. thanks. reverted them.
0190abd
3ff5847

Copy link
Contributor

@sakutademos sakutademos left a comment

Choose a reason for hiding this comment

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

code review, lgtm

@salisdemos salisdemos merged commit 5b1827d into main Nov 7, 2023
2 checks passed
@salisdemos salisdemos deleted the f_advance_multiselect_slots branch November 7, 2023 11:54
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