-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Migrate NGL viewer to npm package #18946
Conversation
…we probably want to isolate this to static/dist
Thank you @dannon for helping with this project. This looks gtm +1. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works for me flawlessly! Its always cool to see proteins spinning :)
I think there are a few things we want to adjust with the published version of the chart, but this is a good step forward for the ngl viewer specifically in any event using the new publishing/installation mechanisms -- going to go ahead and merge this iteration in. |
This PR was merged without a "kind/" label, please correct. |
This PR replaces the Galaxy dependent backbone-based charts-ngl plugin with a separate npm package using the Vue3-based charts package.
How to test the changes?
(Select all options that apply)
License