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(statustag): change icon font-size and docs #504

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

LuckyFBB
Copy link
Collaborator

@LuckyFBB LuckyFBB commented Oct 8, 2024

变更类型

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

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

相关问题

#487
#483 statusTag 修改

变更内容

更改 icon 大小为 16px

修改 statusTag 的文档

详细描述

对应 Previewer

@liuxy0551
Copy link
Collaborator

看变更好像没改到 #483

@LuckyFBB
Copy link
Collaborator Author

LuckyFBB commented Oct 8, 2024

看变更好像没改到 #483

image

483 中关于 StatusTag 的部分

@liuxy0551
Copy link
Collaborator

看变更好像没改到 #483

image 483 中关于 StatusTag 的部分

这样改动就可以把重复了几次的 ”图片样式“ 改好是吗,是的话我就没问题了

image

@LuckyFBB
Copy link
Collaborator Author

LuckyFBB commented Oct 8, 2024

看变更好像没改到 #483

image 483 中关于 StatusTag 的部分

这样改动就可以把重复了几次的 ”图片样式“ 改好是吗,是的话我就没问题了

image

这个是 Image 的组件,暂时没改这个,留着和 Image 组件的相关内容一起修改吼

@liuxy0551
Copy link
Collaborator

看变更好像没改到 #483

image 483 中关于 StatusTag 的部分

这样改动就可以把重复了几次的 ”图片样式“ 改好是吗,是的话我就没问题了
image

这个是 Image 的组件,暂时没改这个,留着和 Image 组件的相关内容一起修改吼

#483 中应该去除 Image 的内容,或者 Image 的文档问题随当前 PR 一同修改

@LuckyFBB
Copy link
Collaborator Author

LuckyFBB commented Oct 8, 2024

看变更好像没改到 #483

image 483 中关于 StatusTag 的部分

这样改动就可以把重复了几次的 ”图片样式“ 改好是吗,是的话我就没问题了
image

这个是 Image 的组件,暂时没改这个,留着和 Image 组件的相关内容一起修改吼

#483 中应该去除 Image 的内容,或者 Image 的文档问题随当前 PR 一同修改

更新了 #483 和当前 PR 的内容

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

@LuckyFBB LuckyFBB merged commit 0695b73 into DTStack:master Oct 24, 2024
1 check passed
@LuckyFBB LuckyFBB deleted the fix/487 branch October 24, 2024 08:18
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.

3 participants