tdesign-react icon indicating copy to clipboard operation
tdesign-react copied to clipboard

fix(radio): group can't render outline variant by options prop

Open NWYLZW opened this issue 2 years ago • 3 comments

🤔 这个 PR 的性质是?

  • [x] 日常 bug 修复
  • [ ] 新特性提交
  • [ ] 文档改进
  • [ ] 演示代码改进
  • [ ] 组件样式/交互改进
  • [ ] CI/CD 改进
  • [ ] 重构
  • [ ] 代码风格优化
  • [ ] 测试用例
  • [ ] 分支合并
  • [ ] 其他

🔗 相关 Issue

无。

💡 需求背景和解决方案

Radio 组件的 variant 无法处理 outline 类型的输入。

📝 更新日志

  • fix(Radio): variant 为 outline 时不支持使用 options 的方式进行渲染

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

  • [x] 文档已补充或无须补充
  • [x] 代码演示已提供或无须提供
  • [x] TypeScript 定义已补充或无须补充
  • [x] Changelog 已提供或无须提供

NWYLZW avatar Nov 25 '23 20:11 NWYLZW

失败

github-actions[bot] avatar Nov 25 '23 20:11 github-actions[bot]

df1c95a1-665c-4784-8d03-9cd15e52a1bf

Radio 的 这个 variant 的默认值感觉有点问题,如果是 outline 的话 在使用 options 渲染的时候时候就没办法决定是不是使用 Radio.Button 进行渲染了

7ee3157a-0743-4023-ad3c-a86aecef31b8 7ceb6dcc-d551-4dcc-a0fa-d2241d024a4c

如果这俩个都是 variant outline 的话就没办法使用 options 数据渲染的时候选择某一种样式展示了 我之前以为 上面的是对应的 <Radio options={xx}/> 下面的是对应的 <Radio variant='outline' options={xx}/>

我看了一下 Button 那边的定义现在 Radio.Button 的默认样式其实就是沿用的 Button variant=outline 是不是 Radio 的默认类型写错,应该默认是 undefined,这样子才能解释和用的出来

然后现在 react 这边好像之所以能跑好像是因为 bug,恰好因为我没处理 variant outline 所以才使得看起来好像没有问题了。

NWYLZW avatar Nov 25 '23 20:11 NWYLZW

单元测试之所以挂了是因为 <Radio.Group /> === <Radio.Group variant='outline' />

NWYLZW avatar Nov 25 '23 20:11 NWYLZW