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

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NWYLZW
Copy link
Collaborator

@NWYLZW NWYLZW commented Nov 25, 2023

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

无。

💡 需求背景和解决方案

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

📝 更新日志

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

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

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

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

Copy link
Contributor

github-actions bot commented Nov 25, 2023

失败

@NWYLZW
Copy link
Collaborator Author

NWYLZW commented Nov 25, 2023

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
Copy link
Collaborator Author

NWYLZW commented Nov 25, 2023

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

NWYLZW added a commit to zodui/zodui that referenced this pull request Nov 25, 2023
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.

1 participant