-
Notifications
You must be signed in to change notification settings - Fork 369
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
Vue 3 Compatibility #312
base: next
Are you sure you want to change the base?
Vue 3 Compatibility #312
Conversation
Thank you so much for this. |
Thanks for your nice work! Maybe dev on full-clean branch then release a major version would be better? |
And can this PR be released as an alpha version? Maybe the temp version that supported Vue.js 3.0 be helpful to users, we can release a version first, and then continue to improve it |
@PeachScript when merge ? and can I use nuxt width used composition api ? |
After all test cases updated. But we can first release a alpha version if @nVitius confirm this PR can be released |
@PeachScript Sorry for the late reply from me. If you make a new branch on the main repo (I called mine |
@nVitius thanks for your reply ❤️ I've already create the @printscreen @productfrontenddeveloper it would be better if you could test it with your projects and feedback :D |
iam nuxt 2, vue 2 with nuxt compotion api. I can say that nuxt 3 will be out soon then. then i can try :D |
Currently, the spinner setting for the alpha version is very wired. You have to assign it like the following:
It's using an index to represent various spinner listed in the doc. A brief inspection into the codes in src shows that it is due to the processing of |
@phantomlsh Thanks for the report! I just pushed a commit that should fix that issue. |
@nVitius Thanks for your work but i tested with your fork and the issue reported by @phantomIsh persists |
There is another bug with spinner. If you use custom tag with slot="spinner" or <template #spinner> it doesn't hide the spinner after loading. It does for default spinners, but not from custom. I had to hide it myself right after the $state.complete() |
I forgot to re-run the build before committing. I pushed another commit with the updated dist files. @noeGnh Could you provide a small reproduction for the custom spinner issue? We are using |
Hi @nVitius, |
@noeGnh Thanks for setting that up. |
@nVitius Thanks for your work ! |
@PeachScript Can you publish the new build to npm? I've had the opportunity to work extensively with the Composition API over the past 6 months or so. I want to take some time soon and refactor the components in here to use it, as well as update the tests and such so we can get this PR merged in. |
Is there any update on this issue? |
Any update on this? |
I cannot have empty(no value) in the slots. It is possible with the previous version. |
@shashank261 Thanks for the report. I just pushed up a commit that should fix that issue. You can try it out by installing from my fork: |
Hi, can someone explain what has to happen for this to make it into the main release? We're waiting on this over on https://github.com/antoine92190/vue-advanced-chat Thanks! |
@PeachScript any progress ? |
@PeachScript We have been using |
vue3 testted good working |
No official vue3 support available. Installed from git with: npm i --save 'github:nvitius/vue-infinite-loading#next' (as described in <PeachScript/vue-infinite-loading#312>)
Hangs on install during "reify:nan" in "reify:audit" after these lines
|
No official vue3 support available. Installed from git with: npm i --save 'github:nvitius/vue-infinite-loading#next' (as described in <PeachScript/vue-infinite-loading#312>)
@PeachScript please pull merge. |
Does not work if more than one InfiniteLoading component on the same page |
When are you planning to make release for Vue.js 3 support? |
@printscreen is this |
@PeachScript Also interested in checking in on this progress. Thank you! |
"vue-infinite-loading": "3.0.0-alpha.0-0", is working fine for me already, please merge this PR |
EventHub causes inner state error for multiple infinite loading instances, please notice this problem. |
This patch enables compatibility with Vue 3.
Aside from a couple deprecations that were fixed, I changed the build tool to rollup in stead of webpack.
Tests will likely need to be updated.