Skip to content

WIP: feat(Chart): Sync only zoom behaviour between charts in a group

Andrei Stoicescu requested to merge astoicescu/syncOnlyZoomInCharts into master

What does this MR do?

At the moment, members of a chart group are syncing interactions that are unwanted, all because of the larger scope of the echarts.connect() method behaviour.

Example 1: Chart lines disabling between two charts in the same group

sync_01

Example 2: Crosshairs are synced between charts in the same group

sync_02

The only interaction syncing that is wanted is the one of the zooming behaviour.

This MR attempts to only add syncing for zooming.

Does this MR meet the acceptance criteria?

Conformity

  • Code review guidelines.
  • GitLab UI's contributing guidlines.
  • If it changes a Pajamas-compliant component's look & feel, the MR has been reviewed by a UX designer.
  • If it changes GitLab UI's documentation guidelines, the MR has been reviewed by a Technical Writer.
  • If the MR changes a component's API, integration MR(s) have been opened in the following projects to ensure that the @gitlab/ui package can be upgraded quickly after the changes are released:
  • Added the ~"component:*" label(s) if applicable.

Merge request reports