Skip to content

feat(copy): remove title/hideTooltip, add tooltip #479

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

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

LuckyFBB
Copy link
Collaborator

@LuckyFBB LuckyFBB commented Sep 3, 2024

变更类型

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

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

相关问题

#477

变更内容

删除现有的 title/hideTooltip 属性

更改为 tooltip 属性,React.ReactNode | TooltipProps

传入了 tooltip 属性就展示 组件

tooltip 为 React.ReactNode 时转换为 { title: tooltip }

大体设计保持和 Form.Item 的 tooltip 一致

后续目标让 RC 的所有 tooltip 保持一致

详细描述

对应 Previewer

@shiqiWang0
Copy link
Collaborator

这样改之后是不是需要传 tooltip 才给改造后的 copy 组件展示;没改之前的话, 默认就可以展示 复制的 tooltip,这样子产品是不是 用这个组件 就需要传 tooltip 了

@LuckyFBB
Copy link
Collaborator Author

LuckyFBB commented Sep 3, 2024

这样改之后是不是需要传 tooltip 才给改造后的 copy 组件展示;没改之前的话, 默认就可以展示 复制的 tooltip,这样子产品是不是 用这个组件 就需要传 tooltip 了

是会有这个问题,可以改成 tooltip = “复制”,如果不需要展示 Tooltip 的话就得设置 tooltip = “”

@LuckyFBB LuckyFBB requested a review from mortalYoung September 5, 2024 03:57
Copy link
Collaborator

@mortalYoung mortalYoung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@mortalYoung mortalYoung merged commit f8ca178 into DTStack:master Sep 9, 2024
1 check passed
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.

None yet

4 participants