-
Notifications
You must be signed in to change notification settings - Fork 294
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
Support Angular 13 #515
Comments
It looks a lot like this package has been abandoned. The last tag was Feb 2020 and there are no accepted pull request since Nov 2020. @rintoj @speige @kykint are listed as maintainers but they all haven't interacted with this repository since 2020. I guess we'll have to look for an alternative, which is really sad, because this package is just the best for the job, as far as i know. |
@iharbeck published an angular 13 version on npm: |
Maybe now Angular 14 |
Currently working on Angular 14 version, waiting for fxLayout to be released for Angular 14 to complete final testing. |
@iharbeck Awesome! Keep us posted! Is there anywhere we can check out the progress? |
@lincolnthree @pjc2007 @iharbeck/ngx-virtual-scroller@14.0.4 https://www.npmjs.com/package/@iharbeck/ngx-virtual-scroller |
Thankyou @iharbeck . I have updated this and Angular to 14. So far so good. |
@iharbeck Actually, I just realised I did not point to the correct npm (@iharbeck/ngx-virtual-scroller@14.0.4) I've now done that, deleted my node_modules, reinstalled, but am getting this error (in both IDE and when I compile) Any ideas why this would be? |
just change the import to @iharbeck as well. The IDE should fix it if you remove the import and use the code completion. |
Thankyou again @iharbeck . The above worked and now have it incorporated, and all looks good my initial tests in my use case (our QA now be testing the app) |
I'm having this error with that version :(
|
try to add following dependencies to your project: |
@iharbeck will your improvements be merged into this repo? |
@ewalddieser the rintoj repository is not maintained anymore. You can use my fork instead. |
@iharbeck That's really cool. Is there a way for you to take over the main repo? So that people don't get confused and also could just install using I'm currently evaluating this lib and so far it looks really promising. The requirement is to render really long documents (laws) where the chapters are loaded lazily. It works already nicely when normally scrolling up and down. The difficulty is that user should also be able to quickly scroll hundreds of chapters up/down by dragging the scroll bar. I tried to solve that by creating placeholder chapter objects with estimated heights and then load the real chapters when they are scrolled into view. This works, but not good enough. The document then jumps back and forth awkwardly. |
I am really stuck in Angular 12 and it seems that @iharbeck's package supports only versions 13 and 14. |
@marian-simonca |
@iharbeck thank you for your fork: https://github.com/iharbeck/ngx-virtual-scroller 🤝 I think we'll be able to get this repository updated sometime in the near future too 🤞 cheers! |
@whyboris does this imply the maintainer is going to also update to Angular 15? I'm in the process of updating the code to Angular 14 (including strict typing) and was planning to raise a PR back to iHarbeck's repository. Should I retarget here or just let the maintainer take care of it? Edit: I did raise a PR for Angular 14 (minus 2 lines which need help on). |
@iharbeck I'm not sure what's happening on this repo but do you think you'll do Angular 15 in your fork? |
@gloriousjob Just released version 15, please give a try. |
Thanks, @iharbeck! Works well! |
@iharbeck thanks for the v15 fork! also, it seems like you need both ngx-virtual-scroller and @iharbeck/ngx-virtual-scroller to make it work? I tried to open the demo folder in stackblitz but it seems like there is an issue with the path src/virtual-scroller in list-with-api.component.ts opening rintoj's demo folder in stackblitz seemed to work fine though |
There is another library (not a fork) that I think deserves a look: It's actually quite simple, supports responsive changes to the viewport size very well, and is quite configurable. Just make sure you specify both [itemHeight] and [itemWidth], or it wont render more than one item. I think with a little love, and maybe a few features brought over from this lib, this one has a LOT of potential. We are going to try it in our app and see how it goes. |
Dear @iharbeck, I tried your fork, and I have the following build error. Can you pls suggest a workaround?
I'm on Angular 13.3.11. The folder of |
@ichepurnoy please add "skipLibCheck": true, in tsconfig.json |
Angular 13 enables Ivy and removes the View Engine. Please update dependencies and configs to support Angular 13.
The following warning shows during a compile:
⠙ Generating browser application bundles (phase: setup)...Processing legacy "View Engine" libraries:
Encourage the library authors to publish an Ivy distribution.
NOTE: Someone has already opened a PR, which appears to be just running through the ng update's from where the code was at:
#514
The text was updated successfully, but these errors were encountered: