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.

Sign up for GitHub

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

Jump to bottom

fix: tooltip for multi view #6142

Merged

pearmini merged 1 commit into v5 from fix-tooltip-multiview

Mar 21, 2024

Member

pearmini commented Mar 21, 2024

close: #6130

问题：多视图 tooltip 展示失败。
原因：通过 style 获得 rect bbox 没有考虑 transform，会导致拾取不到。
解决办法：通过 getGeometryBounds + transform 来获得真实的 bbox。


          fix: tooltip for multi view

c8bd058

pearmini force-pushed the fix-tooltip-multiview branch from 23ea05e to c8bd058 Compare

March 21, 2024 05:23

coveralls commented Mar 21, 2024

Pull Request Test Coverage Report for Build 8370113610

Details

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 86.466%

Totals
Change from base Build 8354524575:	0.02%
Covered Lines:	10263
Relevant Lines:	11502

💛 - Coveralls

xiaoiver approved these changes

View reviewed changes

pearmini merged commit 411e1f4 into v5

2 checks passed

pearmini deleted the fix-tooltip-multiview branch

March 21, 2024 07:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet