-
Notifications
You must be signed in to change notification settings - Fork 376
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
Zoneless angular support #916
Comments
here is my fork: https://github.com/banned007/angular-gridster-zoneless |
Hello, |
Hmm I don't think a config option is feasible, as you would need to remove zone.js from the peerDeps (because is not a dependency on zoneless), and when you inject it, you might get some injection dependency issues, so I think the most straight forward approach would be to just publish a new package. |
Yes you are right, the most viable solution I see is to make a branch with the changes and make a version with zoneless and signals, something like 18.0.0-z to be published on npm. Will see when I would have the time for this, most probably next week. |
New version 18.0.0-z is on npm https://www.npmjs.com/package/angular-gridster2/v/18.0.0-z |
Please add support for zoneless angular, maybe a new package/repo for this? not sure how to proceed, I have a working version of zoneless (literlly just removed the calls of zone functions, and it seemed to work right out) on my fork, but not sure if this is something you should include in the main package or if you want to create a new one for supporting this feature.
The text was updated successfully, but these errors were encountered: