Skip to content

feat(modal): change Modal.method to use dtinsightIcon and add Modal.delete #586

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat(modal): change Modal.method to use dtinsightIcon and add Modal.delete #586

wants to merge 2 commits into from

Conversation

LuckyFBB
Copy link
Collaborator

@LuckyFBB LuckyFBB commented Aug 4, 2025

变更类型

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

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

相关问题

#583

变更内容

  • 更改 Modal.method 上的默认 icon，以及对应的文本，按钮 props
  • 新增 Modal.delete 结合 comfirm 更改 icon 以及文本

详细描述

对应 Previewer

Previewer

@LuckyFBB LuckyFBB requested a review from shiqiWang0 August 12, 2025 11:15
liuxy0551
liuxy0551 reviewed Aug 12, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liuxy0551 liuxy0551 liuxy0551 left review comments

@Cythia828 Cythia828 Awaiting requested review from Cythia828

@jin-sir jin-sir Awaiting requested review from jin-sir

@JackWang032 JackWang032 Awaiting requested review from JackWang032

@shiqiWang0 shiqiWang0 Awaiting requested review from shiqiWang0

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LuckyFBB @liuxy0551