Would've been nice to make teleport an option

[mattelen]

Would've been nice to make this an option. The old way worked perfectly for us, we don't need it teleported anywhere, but now we need to rewrite all the e2e tests to make this change work again. How is this a minor release?

Originally posted by @lehni in #1903 (comment)

[mattelen]

@lehni we could make this work with a prop, the default being that teleport won't activate. Would this work for your situation?

[oliversanders]

This update has broken our implementation too, #1903 has a number of other issues:

1. Fixed position does not respond to page size changes or scroll, other components tend to implement https://github.com/floating-ui/floating-ui for stuff like this
2. Unable to add custom classes that can target .multiselect__content-wrapper at the usage level (e.g.<VueMultiSelect :class="…" />).
3. z-index change to 9999 is somewhat excessive and goes over the top of other components (e.g. modals)
4. The PR contributor could have better solved their original problem by following https://guissmo.com/blog/bypassing-overflow-hidden-rule-in-css/ instead of using a teleport.

This change really shouldn't have been considered minor based on the impact it has, we've had no choice but to downgrade to 3.2.0.

[lehni]

@mattelen yes that would be useful indeed!

[oliversanders]

@mattelen thanks for resolving this so promptly, there's one minor thing left over from the original PR which still impacts both use-cases - the z-index: 9999; line in Multiselect.vue, this was changed from 50 to 9999:

vue-multiselect/src/Multiselect.vue

Line 743 in e0d3b48

z-index: 9999;

This isn't necessary to keep due to zIndex: 9999 being conditionally set in the isOpen watcher anyway.