Skip to content

Conversation

@Eomnational
Copy link

@Eomnational Eomnational commented Nov 12, 2025

变更类型

请选择以下选项以描述 PR 的类型:

  • Bug 修复(修复现有问题)
  • [ √] 新功能(添加了一个功能)
  • 代码优化(性能改进、代码重构)
  • 文档更新
  • 单测新增或修改
  • 其他(请说明):

相关问题

#620

变更内容

详细描述

https://eomnational.github.io/dt-react-component/components/tiny-tag

对应 Previewer

@LuckyFBB LuckyFBB marked this pull request as draft November 12, 2025 03:52
@shiqiWang0
Copy link
Collaborator

shiqiWang0 commented Nov 12, 2025

你好,感谢你的 PR,这里我有几个疑问
从 TinyTag 设计之初,是考虑需要展示 10px 的 text; 你这边提供的 MR

  1. 从 demo 中可以看到 icon 属性传值 <span style={{ fontSize: '12px' }}>⭐ , 可能会比较大; 需要给一个 10px 的大小;在这种情况下, 我们的组件其实可以 <TinyTag value="⭐ 推荐" /> 直接这样使用;
  2. 另外用 icon 属性, 是非常灵活的,那就可以在使用上 将<span style={{ fontSize: '12px' }}>⭐⭐⭐⭐⭐</span> 作为icon 的属性值, 那在这种情况下,我们的组件是支持 <TinyTag value="⭐ ⭐⭐⭐ 推荐" />,可以兼容处理掉这种展示问题;
image
  1. 从另一个例子, 支持 icon 传入自定义的 svg ,但是这个 svg 设置的样式,我的疑问是为什么 这个 icon 做了absolute 绝对定位;
    我觉得是可以支持 icon 进行传入丰富我们的组件能力,可能仍需要考虑下这个 icon 展示可能存在的传入的值,以及如果不如预期传的时候,可能会导致展示问题;

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