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

[Icon] 支持传入SVG直接渲染 #27

Closed
richerfu opened this issue Jun 24, 2022 · 9 comments
Closed

[Icon] 支持传入SVG直接渲染 #27

richerfu opened this issue Jun 24, 2022 · 9 comments
Assignees
Labels

Comments

@richerfu
Copy link
Contributor

这个功能解决了什么问题

自定义渲染内容

你建议的方案是什么

传入SVG,直接渲染

import { Icon } from 'tdesign-icons-react';
const svg = <Svg />
const Com = () => <Icon conponent={svg} />
@github-actions
Copy link

👋 @southorange1228,感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

@github-actions
Copy link

github-actions bot commented Jun 24, 2022

♥️ 有劳 @uyarn 尽快确认问题。
确认有效后将下一步计划和可能需要的时间回复给 @southorange1228 。

@honkinglin
Copy link

为何不直接渲染 Svg 组件呢。。

@richerfu
Copy link
Contributor Author

richerfu commented Jun 24, 2022

为何不直接渲染 Svg 组件呢。。

简化使用成本 比如 颜色 大小等API,可以比较统一。
第三方的用户可以使用这个统一输出类似于tdesign-icons-react的库
如果要支持树摇,必须把SVG作为dom结构进行渲染,所以提出这个能力。对比了下业内的大部分组件库 应该都是支持的

@uyarn
Copy link
Collaborator

uyarn commented Jun 24, 2022

我考虑一下 先transfer到icon仓库吧

@uyarn uyarn transferred this issue from Tencent/tdesign-react Jun 24, 2022
@github-actions
Copy link

👋 @southorange1228,感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

@richerfu
Copy link
Contributor Author

richerfu commented Jul 5, 2022

激活下此issue,大佬看下是否考虑纳入本方案或者有其他替代方案 @uyarn

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

这个 Issue 被标记为了过时 stale ,因为它已经持续 30 天没有任何活动了。删除 stale 标签或评论,否则将在 7 天内关闭。

@github-actions github-actions bot added the Stale label Aug 4, 2022
@github-actions
Copy link

此 Issue 被自动关闭,因为它自被标记为过时 stale 以来已闲置 7 天。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants