search

angular-slider / ngx-slider

Self-contained, mobile friendly slider component for Angular based on angularjs-slider
https://angular-slider.github.io/ngx-slider/
MIT License
388 stars 175 forks source link

Support Angular 13+ (Ivy distribution) #296

Closed jefbarn closed 9 months ago

jefbarn commented 2 years ago

From Angular 13:

Processing legacy "View Engine" libraries:

mateusz-sawit commented 2 years ago

Seems to be done on fork https://github.com/rp3e11/ngx-slider

rp3e11 commented 2 years ago

Yes I forked it and upgrade it to v13.

It works fine in my project, but I have to do further testing. To upgrade I also had to upgrade the demo-app which seems fine at first glance but I have check it in more detail.

I had one issue with performance since ngUpdate called updateTicks which in turn calls detectChanges() and that lead to infinite callback loop. I moved detectChanges inside an if statement such that it is only called if the ticks are changed. This might be a problem for other uses and I have to investigate that further.

I didn't manage to run the e2e tests yet, but that seems to be related to a version mismatch between chromium driver and my installed chrome, so I could't execute the tests.

rp3e11 commented 2 years ago

I had another look at the remaining work to finish the upgrade to v13.

The state right now is, that I can install and work with the slider in my project. However, I only use a subset of all the features so I have no clue, whether there are any remaining issues for all the other features. Further the demo-app in particular the typedoc documentation isn't properly working after the upgrade. The way I see it right now, there are three tasks that would have to be completed before it could be merged back:

Fixing all the above will take a significant amount of time. I could do the bulk of the work, but before I would start it, I would have to be sure, that it would be merged back by the maintainers and that they are happy with the breaking upgrades from protractor to cypress and typedoc. Is there a maintainer who would do that?

jymaps commented 2 years ago

Does the fork solve the issue? How do I use it in my project? Is it enough to run ng build?

jymuk commented 2 years ago

@rp3e11 Can you help me out please with a guide on how to install this fork in our app?

rp3e11 commented 2 years ago

Sorry for the late reply.

I am not working with this fork anymore since I haven't heard back from maintainers and the library is quite complex and I don't understand the implementation to a high enough level, that I could maintain it myself (i.e. fix bugs). So I wouldn't recommend using my fork in the long run.

If you still want to use it, there are a couple of options. The easiest one (in my opinion) is to clone it and use npm link as described in this stackoverflow thread. The build instructions are in the file DEVELOPERS.md.

danisss9 commented 1 year ago

Hello,

I created a fork with Angular 14/15 support based on @rp3e11 fork. In case you want to use it https://www.npmjs.com/package/ngx-slider-v2 https://github.com/danisss9/ngx-slider

vinlos commented 1 year ago

Hello,

I created a fork with Angular 14/15 support based on @rp3e11 fork. In case you want to use it https://www.npmjs.com/package/ngx-slider-v2 https://github.com/danisss9/ngx-slider

Thank you for your work. If you need help in maintaining this package, don't hesitate to contact me

whyboris commented 1 year ago

✅ confirming that danisss9/ngx-slider Fork works with Angular 16 🚀

ananthay commented 1 year ago

are there any plans of merging danisss9/ngx-slider ?

piotrdz commented 9 months ago

Closing old issues about the same subjects. Please follow discussion in more recent issues regarding this subject (https://github.com/angular-slider/ngx-slider/issues/351, https://github.com/angular-slider/ngx-slider/issues/355).