Skip to content

Update dependencies for IGV and Plotly#21543

Merged
jmchilton merged 6 commits intogalaxyproject:devfrom
guerler:update_viz.000
Jan 13, 2026
Merged

Update dependencies for IGV and Plotly#21543
jmchilton merged 6 commits intogalaxyproject:devfrom
guerler:update_viz.000

Conversation

@guerler
Copy link
Copy Markdown
Contributor

@guerler guerler commented Jan 7, 2026

Updates igv.js to 3.7.1, plotly.js-dist to 3.3.1, and fixes a minor icon rendering issue for Nora when Galaxy is served with prefix. Additionally it updates the Charts UI package to vue 3.5.26, naive-ui 2.43.2 tailwind 4.1.18.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@guerler guerler added this to the 26.0 milestone Jan 7, 2026
@guerler guerler marked this pull request as ready for review January 7, 2026 14:53
@guerler guerler requested a review from bgruening January 7, 2026 14:53
@guerler guerler requested a review from itisAliRH January 9, 2026 16:25
confirm_button:
type: xpath
selector: '(//button[text()=" OK "])[1]'
current_genome: '[title="Human (hg18)"]'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't right at all. You can parameterize this and pass in the values you're looking for but these components are generic specifiers for how to find things in the UI not just text fixture constants. The trs_search selectors below use parameters repeatedly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the genome selector to take the genome name as a parameter instead of encoding it in the selector.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry if that came off as pedantic but I really appreciate the change - looks great to me now!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not at all, thanks for taking the time to point it out.

@jmchilton jmchilton merged commit 96856df into galaxyproject:dev Jan 13, 2026
36 of 41 checks passed
@guerler guerler deleted the update_viz.000 branch January 14, 2026 08:39
@guerler guerler added the release-testing-26.0 PRs marked for testing for the 26.0 release and issues stemming from release testing label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/visualizations kind/enhancement release-testing-26.0 PRs marked for testing for the 26.0 release and issues stemming from release testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants