component
component copied to clipboard
fix: invisible axis labels after data changed
fix antvis/G2#6373
帮忙加个单测。
帮忙加个单测。
这个单测可以怎么写,是写jest的test还是dev的用例。发现之前使用canvas的单元测试例子很少,issue legend那里倒是有不过是skip的,是要类似那样写吗
我先试着写了一个测试,看看还要不要改
单测是 ok 的,只要能修改代码前报错,修改代码后通过即可。
@Sociosarbis 你这个 pr 改动很大,可以描述你的改动思路吗?
单测是 ok 的,只要能修改代码前报错,修改代码后通过即可。
@Sociosarbis 你这个 pr 改动很大,可以描述你的改动思路吗?
https://github.com/antvis/component/pull/328/files#diff-952fad19eb6df82060dc1ed9745576b356283c6ff4fa326de8360e868edbb99bR263-R271
https://github.com/antvis/component/pull/328/files#diff-b131542d356aeec217531cdbe2ffa00e90b10daa5927996d3679c2973916f759R362-R367
主要的改动是这两处,第一处是收集所有的animation,等到所有的animation结束以后再去计算overlap, 同时也在开始的时候也计算一次,目的是保证开始和结束的状态是正确的。原来在enter的时候会立刻调一次(这样会把还未处理exit考虑进去),还有update的animation结束时调用,然而enter和update的回调都比exit早执行。感觉在处理完所有属性设置和动画以后,再后处理overlap会合理些,毕竟个人感觉overlap主要是关注最后的结果。 第二处是为了保证_transitions的类型跟声明保持一致,原来有可能是多层的嵌套,避免错误使用。
@Sociosarbis hi,这个 PR 导致一些截图发生变化(你可以在 Details查看详情),如果确认变更是正确的,请更新下并提交相应截图。
@Sociosarbis hi,这个 PR 导致一些截图发生变化(你可以在 Details查看详情),如果确认变更是正确的,请更新下并提交相应截图。
好,这几天有空的时候,我仔细看看👀
把立刻执行overlapHandler,改成只在动画结束后才执行就通过了。本身也觉得立刻执行是会有些问题