fix: 自定义 icon 支持跨域 close #2513

[lijinke666]

👀 PR includes

🐛 Bugfix

• Solve the issue and close 🐛自定义 icon 渲染会导致 canvas 无法正常导出图片 #2513

🔧 Chore

• Test case
• Docs / demos update
• CI / workflow
• Release version
• Other ()

📝 Description

通过 customSVGIcons 自定义在线链接的 icon 时支持跨域, 防止 Canvas 转图片时报错

🖼️ Screenshot

Before	After

🔗 Related issue link

🔍 Self-Check before the merge

• Add or update relevant docs.
• Add or update relevant demos.
• Add or update test case.
• Add or update relevant TypeScript definitions.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
antvis-s2	❌ Failed (Inspect)			Jan 19, 2024 10:21am

[github-actions]

Size Change: +20 B (0%)

Total Size: 441 kB

Filename	Size	Change
./packages/s2-core/dist/index.min.js	176 kB	+20 B (0%)

ℹ️ View Unchanged

Filename	Size
./packages/s2-core/dist/style.min.css	389 B
./packages/s2-core/node_modules/@testing-library/dom/dist/@testing-library/dom.cjs.js	21.7 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/@testing-library/dom.esm.js	21.2 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/@testing-library/dom.umd.js	60.3 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/@testing-library/dom.umd.min.js	33.9 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/config.js	1.16 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/DOMElementFilter.js	1.87 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/event-map.js	1.18 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/events.js	1.55 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/get-node-text.js	300 B
./packages/s2-core/node_modules/@testing-library/dom/dist/get-queries-for-element.js	738 B
./packages/s2-core/node_modules/@testing-library/dom/dist/get-user-code-frame.js	810 B
./packages/s2-core/node_modules/@testing-library/dom/dist/helpers.js	1.07 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/index.js	1.14 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/label-helpers.js	1.05 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/matches.js	1.14 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/pretty-dom.js	1.47 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/all-utils.js	337 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/alt-text.js	573 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/display-value.js	713 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/index.js	418 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/label-text.js	1.8 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/placeholder-text.js	491 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/role.js	2.67 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/test-id.js	498 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/text.js	958 B
./packages/s2-core/node_modules/@testing-library/dom/dist/queries/title.js	718 B
./packages/s2-core/node_modules/@testing-library/dom/dist/query-helpers.js	1.81 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/role-helpers.js	2.82 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/screen.js	1.51 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/suggestions.js	1.5 kB
./packages/s2-core/node_modules/@testing-library/dom/dist/wait-for-element-to-be-removed.js	748 B
./packages/s2-core/node_modules/@testing-library/dom/dist/wait-for.js	2.38 kB
./packages/s2-react/dist/index.min.js	68 kB
./packages/s2-react/dist/style.min.css	3.43 kB
./packages/s2-vue/dist/index.min.js	20.5 kB
./packages/s2-vue/dist/style.min.css	1.68 kB

_{compressed-size-action}

[github-actions]

你好, @lijinke666 CI 执行失败, 请点击 [Details] 按钮查看, 并根据日志修复。

Hello, @lijinke666 CI run failed, please click the [Details] button for detailed log information and fix it.

[codecov]

Codecov Report

Attention: 176 lines in your changes are missing coverage. Please review.

Comparison is base (1adc25d) 76.48% compared to head (630180c) 76.80%.
Report is 124 commits behind head on master.

Files	Patch %	Lines
packages/s2-core/src/cell/base-row-cell.ts	71.72%	22 Missing and 19 partials ⚠️
packages/s2-core/src/facet/frozen-facet.ts	82.01%	2 Missing and 32 partials ⚠️
packages/s2-core/src/facet/pivot-facet.ts	79.12%	4 Missing and 15 partials ⚠️
packages/s2-core/src/data-set/pivot-data-set.ts	80.32%	1 Missing and 11 partials ⚠️
...ckages/s2-core/src/facet/header/base-frozen-row.ts	85.36%	0 Missing and 6 partials ⚠️
...s/s2-core/src/facet/layout/build-gird-hierarchy.ts	82.35%	1 Missing and 5 partials ⚠️
...eraction/base-interaction/click/data-cell-click.ts	50.00%	3 Missing and 3 partials ⚠️
packages/s2-core/src/data-set/table-data-set.ts	66.66%	0 Missing and 5 partials ⚠️
packages/s2-core/src/facet/header/row.ts	58.33%	5 Missing ⚠️
packages/s2-core/src/facet/layout/layout-hooks.ts	0.00%	3 Missing and 1 partial ⚠️
... and 23 more

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2524      +/-   ##
==========================================
+ Coverage   76.48%   76.80%   +0.31%     
==========================================
  Files         240      249       +9     
  Lines       12312    12764     +452     
  Branches     2630     2749     +119     
==========================================
+ Hits         9417     9803     +386     
+ Misses       1377     1364      -13     
- Partials     1518     1597      +79     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lijinke666]

🎉 This PR is included in version @antv/s2-v1.54.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀