-
Notifications
You must be signed in to change notification settings - Fork 13
chore(deps): update dependency echarts to v6 #432
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
Conversation
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
10a08ea to
2759ebc
Compare
90e94fc to
2d55bfb
Compare
2d55bfb to
b34c785
Compare
b34c785 to
af0ac58
Compare
|
@dr-itz Only thing I could not find, is the data zoom slider shadow, which is slightly a bit up than earlier. The changelog does not mention any changes related to it. |
|
Other important thing is the This completely breaks our layout and we need to define this mode and other spacing to fit our needs. For now we keep the old behavior using - |
dr-itz
left a comment
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.
Looking good. I expected this update to require far more changes. 👍
dashboard VRT needs update
af0ac58 to
13df8fc
Compare
...shots/dashboard.spec.ts-snapshots/dashboard--tablet-dashboards-demo-chromium-light-linux.png
Show resolved
Hide resolved
13df8fc to
64c3766
Compare
dr-itz
left a comment
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.
👍
|
@dr-itz Do we need to test this in trend viewer? |
|
@akashsonune please squash the commits and make yourself the author so that renovate does not override it. Please also add a breaking change note that echarts 6.0.0 is required and maybe also link to their release here https://github.com/apache/echarts/releases/tag/6.0.0 |
yes please |
64c3766 to
a9fe247
Compare
a9fe247 to
28f0d6c
Compare
|
@akashsonune please enhance the breaking change note a little. Something like this would be nice: |
28f0d6c to
9566e6e
Compare
spike-rabbit
left a comment
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.
👍
BREAKING CHANGE: The `@siemens/charts-ng` package now requires ECharts version 6.0.0 or higher. Please update your dependencies accordingly. For details on ECharts 6, see: https://github.com/apache/echarts/releases/tag/6.0.0
9566e6e to
ee90e83
Compare
|
🎉 This PR is included in version 48.0.0-next.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
For the record: Echarts had fixed issue where the data shadow did not match the actual graph. this is the reason why there is slight change in data shaodw. |
This PR contains the following updates:
^5.6.0->^5.6.0 || ^6.0.05.6.0->6.0.0Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
apache/echarts (echarts)
v6.0.0Compare Source
grid(Cartesian),geo,polar, etc.) are supported to be declaratively laid out in the cells of matrix and calendar coordinate system. #19807 #21093 (Ovilia) #21005 #21108 (100pah)gridcomponent) axis labels and axis names overflowing the canvas, prevent axis names from overlapping with axis labels, and make them the default. #21059 (100pah) #19534 (robin-gerling) #16825 (konrad-amtenbrink)roamTrigger; supportclipongeoandseries.map; support cursor style change when hovering on the roaming area; supportpreserveAspectongeo,series.mapandseries.graph; fix the percent base ofcenterongeo,series.map,series.graphandseries.tree; enhance the behavior for roaming area overlapping. #19807#issuecomment-2974437299 (100pah)zoption for markPoint/markLine/markArea. #21117 (sz-p)z2option for markPoint/markLine/markArea. #20782 (sz-p)relativeTooption for specifying the relative target of marker position. #20166 #21042 (Ovilia)displayTransitionoption to control whether to enable the tooltip display transition. #20966 (jqqin)unboundedRangeoption. #21113 (100pah)triggerEventoption. #18164 #20907 (sz-p)tooltipDisabledfor custom series. #20447 (Ovilia)roundCapwhen changes. #20582 (Ovilia)emphasis.focusis'trajectory'with large data. #20959 (plainheart)startValueoption from theBarSeriesOptioninterface. #20901 (plainheart)blur.itemStylenot working. #21081 (mustcanbedo) #21124 (Ovilia)getSvgDataURL. #20760 (plainheart)echarts/theme/v5.jsto restore the old visual style and location settings. See #20865.echarts/src/theme/light.tsis now migrated toecharts/theme/rainbow.js.grid.outerBoundsMode: 'none'to disable the anti-overflow mechanism, and/or use optionxAxis/yAxis.axisLabel.nameMoveOverlap: falseto disable the anti-axisLabel-axisName-overlap mechanism. See #21059.center(such as the base of'33%') ongeo,series.map,series.graphandseries.treeare changed. The previous percent base is incorrect. But if you need to restore, setlegacyViewCoordSysCenterBase: true(on the root level of an echarts option). See #19807#issuecomment-2974437299.fontStyle,fontWeight,fontSize,fontFamily,textShadowColor,textShadowBlur,textShadowOffsetX,textShadowOffsetY) are changed to inherit the plain label styles. You can userichInheritPlainLabel: false(on the root level of an echarts option, or at the same level of the label style options) to restore it. See #20977New Contributors
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.