[apache/echarts]壮举:添加了 barSeries.startValue

2024-05-11 104 views
6

添加 barSeries.startValue 供用户指定条形图的开始位置。

已解决的问题
  • 修复#15463
  • 修复#16840
  • 修复#17077
之前的详细信息 :问题是什么?

正如问题中所述。用户无法指定柱的起始值。

问题#16840 问题#17077
图像 图像
之后:这个 PR 中是如何修复的?

用户现在可以指定barSeries.startValue设置柱的起始值。堆栈和多轴也经过测试可以正确运行。

问题#16840(起始值=-200) 问题#17077(起始值=-150)
#16840 之后 #17077 之后
使用新 API 的相关测试用例或示例

bar-startValue.html

回答

4

感谢您的贡献!社区将尽快对其进行审核。同时,请查看编码标准和 Wiki 关于如何发出拉取请求

本 PR 中需要对文档进行更改。另请向apache/echarts-doc提交文档更改的 PR ,并更新 PR 描述中的问题 id。当文档 PR 被合并时,维​​护者将删除标签PR: awaiting doc

5
  • 这可能会修复 #15463 你能在 PR 描述中添加它吗
2

当然,已经完成了。

0

这看起来是一个有用的功能,只是需要更多的测试用例。例如,测试为轴定义最小值或最大值的情况,或者r极坐标条何时从 0 开始的情况。

并且您需要运行视觉测试来测试此 PR 造成的差异。

npm run test:visual

选择最新的每晚构建来与源代码中的一个构建进行比较。

6

@Ovilia 添加了额外的测试用例,并且所有视觉测试都通过了。

1

另请添加startValue使用 dataZoom 组件的测试。

7

@Olivia 直觉很好dataZoom!我测试了一下,发现startValue不兼容dataZoom。我修改了代码,现在它可以很好地与原始代码配合使用,dataZoom并且可以更自然地合并到原始代码中。现在也min/max由用户设置具有比 更高的优先级startValue。另一个变化是把startValue移到了下面axis而不是series,我认为这更合理。 、priority、polar等相关测试用例dataZoom均已添加。请检查一下。

8

@jiaawulin001 合并被阻止,