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

fix with analogjs #3

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

fix with analogjs #3

wants to merge 5 commits into from

Conversation

mhmmd7777
Copy link

@mhmmd7777 mhmmd7777 commented Jan 27, 2025

Fix #1

@jan-lostak
Copy link

I believe that resolving the bug should not require incorporating a large number of additional dependencies into the project. My intention was to integrate Cornerstone3D into my existing Angular project seamlessly. However, implementing these changes would introduce a multitude of other issues and complications, potentially hindering the overall stability and maintainability of the application. Adding numerous dependencies can increase the complexity of the project, lead to longer build times, and create compatibility challenges with existing libraries and frameworks. Therefore, it's essential to find a solution that addresses the bug without significantly expanding the project's dependency footprint, ensuring that the application remains robust and easier to manage in the long run.

@daemon29
Copy link

I agree with @jan-lostak on this. This will help for anyone want to start a fresh project with angular and cornerstone. But for an existing project, migrate to a new build system is a pain in the ass, as thing will break down, not compatible and stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with Demo Project - Empty Viewport
3 participants