Skip to content

Highcharts conversion to Plotly JS#1822

Merged
aestoltm merged 143 commits intoubccr:xdmod11.0from
aestoltm:xdmod_plotly_conversion
May 7, 2024
Merged

Highcharts conversion to Plotly JS#1822
aestoltm merged 143 commits intoubccr:xdmod11.0from
aestoltm:xdmod_plotly_conversion

Conversation

@aestoltm
Copy link
Copy Markdown
Contributor

Description

These changes are for the Summary, Usage and Metric Explorer Tabs.

When finding a bug please compare with an instance running Highcharts so we know if this is an existing bug or not.

Features not in Plotly JS yet

  • Hover events styling marker and line sizes
  • Shadow

Motivation and Context

Plotly JS has no license requirement which is desirable for Open XDMoD.

Tests performed

Tested on docker and dev-port. Will be converting the Playwright tests to work with Plotly JS.

Checklist:

  • The pull request description is suitable for a Changelog entry
  • The milestone is set correctly on the pull request
  • The appropriate labels have been added to the pull request

@aestoltm aestoltm added this to the 11.0.0 milestone Feb 14, 2024
@aestoltm aestoltm added Category:Metric Explorer Metric Explorer / Usage javascript Pull requests that update Javascript code php Pull requests that update Php code labels Feb 14, 2024
@aestoltm aestoltm merged commit a919f07 into ubccr:xdmod11.0 May 7, 2024
@aestoltm aestoltm deleted the xdmod_plotly_conversion branch May 16, 2024 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category:Metric Explorer Metric Explorer / Usage javascript Pull requests that update Javascript code php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants