Skip to content

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

Merged
Aarebecca merged 3 commits intoreleasefrom
fix/group-attributes
Nov 25, 2024
Merged

fix: fix incorrect bounds calculate when pass stroke and lineWidth to group#1847
Aarebecca merged 3 commits intoreleasefrom
fix/group-attributes

Conversation

@Aarebecca
Copy link
Copy Markdown
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
Copy Markdown
Member

Choose a reason for hiding this comment

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

怎么是这个命名风格?

Copy link
Copy Markdown
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
Copy Markdown
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
@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 <yangtao.yangtao@antgroup.com>

* 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 <1609403959@qq.com>
Co-authored-by: wang1212 <mrwang1212@126.com>
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