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

Needed to import from dist/index.umd to avoid having multiple Vue instances #53

Open
afontcu opened this issue Dec 18, 2020 · 3 comments
Labels
needs reproduction We need a minimal runnable reproduction from you to investigate your report

Comments

@afontcu
Copy link

afontcu commented Dec 18, 2020

Hi mate! I was running into some issues with vue-simple-portal in Nuxt because it was importing another Vue instance so I was getting the usual “$attrs is readonly” “$listeners is readonly” errors.

I got it fixed by changing the import statement to

- import { Portal } from '@linusborg/vue-simple-portal'
+ import { Portal } from '@linusborg/vue-simple-portal/dist/index.umd'

but I wanted to double check if this is intended behavior or if I'm actually doing something wrong here 😇

Thanks!

@LinusBorg
Copy link
Owner

Hm. Seems weird. Not sure where to start, would have to be related to the build config...

Can you create a small repro?

@LinusBorg LinusBorg added the needs reproduction We need a minimal runnable reproduction from you to investigate your report label Dec 31, 2020
@afontcu
Copy link
Author

afontcu commented Dec 31, 2020

Sure! I'll set up a minimal repro and see if the issue is still there – I wanted to make sure it wasn't an expected behavior

@atrunelle
Copy link

We were having the same issue in Nuxt using 0.1.4, but it seems 0.1.5 fixed the problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs reproduction We need a minimal runnable reproduction from you to investigate your report
Projects
None yet
Development

No branches or pull requests

3 participants