Skip to content
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

fix: invisible axis labels after data changed #328

Merged
merged 5 commits into from
Dec 9, 2024

Conversation

Sociosarbis
Copy link
Contributor

@hustcc
Copy link
Member

hustcc commented Sep 23, 2024

帮忙加个单测。

@Sociosarbis
Copy link
Contributor Author

Sociosarbis commented Sep 24, 2024

帮忙加个单测。

这个单测可以怎么写,是写jest的test还是dev的用例。发现之前使用canvas的单元测试例子很少,issue legend那里倒是有不过是skip的,是要类似那样写吗

我先试着写了一个测试,看看还要不要改

@hustcc
Copy link
Member

hustcc commented Dec 3, 2024

单测是 ok 的,只要能修改代码前报错,修改代码后通过即可。

@Sociosarbis 你这个 pr 改动很大,可以描述你的改动思路吗?

@Sociosarbis
Copy link
Contributor Author

单测是 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的类型跟声明保持一致,原来有可能是多层的嵌套,避免错误使用。

@Aarebecca
Copy link
Contributor

@Sociosarbis hi,这个 PR 导致一些截图发生变化(你可以在 Details查看详情),如果确认变更是正确的,请更新下并提交相应截图。

@Sociosarbis
Copy link
Contributor Author

@Sociosarbis hi,这个 PR 导致一些截图发生变化(你可以在 Details查看详情),如果确认变更是正确的,请更新下并提交相应截图。

好,这几天有空的时候,我仔细看看👀

@Sociosarbis
Copy link
Contributor Author

把立刻执行overlapHandler,改成只在动画结束后才执行就通过了。本身也觉得立刻执行是会有些问题

@Aarebecca Aarebecca merged commit b2c4352 into antvis:master Dec 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

柱状统计图的x轴部分名称显示不出来
3 participants