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: fix incorrect bounds calculate when pass stroke and lineWidth to group #1847

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

Aarebecca
Copy link
Contributor

  • 修复给 Group/CustomElement 等非可见元素设置 stroke lineWidth 等参数时,包围盒计算异常的问题。

基于 PARSED_STYLE_LIST 属性列表进行样式处理,需要设置 enableMassiveParsedStyleAssignOptimizationtrue

修复前(绿色为包围盒区域):
image

修复后:
image

@Aarebecca Aarebecca changed the base branch from next to release November 22, 2024 07:08
@Aarebecca Aarebecca changed the title Fix/group attributes fix: fix incorrect bounds calculate when pass stroke and lineWidth to group Nov 22, 2024
@@ -0,0 +1,53 @@
import { Group, Path, Rect, runtime } from '@antv/g';

export async function group_with_stroke(context) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

怎么是这个命名风格?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug 测试用例我就按照具体的问题原因命名的

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

其他的用例基本也是这样命名的

@Aarebecca Aarebecca merged commit a2b137e into release Nov 25, 2024
3 checks passed
@Aarebecca Aarebecca deleted the fix/group-attributes branch November 25, 2024 02:23
Aarebecca added a commit that referenced this pull request Nov 25, 2024
* fix: fixed the issue where RenderCache could not cache RenderPipeline (#1839)

* chore: add changeset

* fix: fix incorrect bounds calculate when pass stroke and lineWidth to group (#1847)

* fix: fix incorrect bounds calculate when pass stroke and lineWidth to group

* chore: commit changeset

* test: fix test case

* update test (#1846)

* chore: fix test

* chore: update vite version

* chore: update test snapshots

* chore: remove pnpm-lock.yaml

* chore: update workflows

---------

Co-authored-by: Aaron <[email protected]>

* chore: update workflows (#1849)

* chore(release): bump version (#1845)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: RE <[email protected]>
Co-authored-by: wang1212 <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

2 participants