Skip to content

breaking(dialog): 调整 Dialog 默认顶边距与关闭按钮位置 - #1036

Open
RSS1102 wants to merge 1 commit into
developfrom
rss1102/cnb-issue-67/fix/dialog-official-alignment
Open

breaking(dialog): 调整 Dialog 默认顶边距与关闭按钮位置#1036
RSS1102 wants to merge 1 commit into
developfrom
rss1102/cnb-issue-67/fix/dialog-official-alignment

Conversation

@RSS1102

@RSS1102 RSS1102 commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

🤔 这个 PR 的性质是?

勾选规则:
1.只要有新增参数,就勾选”新特性提交“
2.只修改内部bug,未新增参数,才勾选”日常 bug 修复“
3.其他选项视具体改动判断

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

🔗 相关 Issue

💡 需求背景和解决方案

Dialog 的默认内容区顶边距与关闭按钮位置与小程序官方基线各差 8px,示例页此前仅展示少量业务化场景。本次对齐官方默认样式并补齐官方示例矩阵:

  1. 默认样式对齐t_dialog.dart,无公开 API 签名变化):
    • 默认内容区 contentPaddingEdgeInsets.fromLTRB(24, 32, 24, 0) 调整为 EdgeInsets.fromLTRB(24, 24, 24, 0)(顶部 32→24);
    • 关闭按钮位置由 top: 0 / end: 0 调整为 top: 8 / end: 8
  2. 示例矩阵补齐t_dialog_page.dart):重构为「01 组件类型 / 02 组件状态 / 03 特殊类型」三组,公开展示反馈类、确认类、按钮布局、带图片、带输入、命令调用共 21 个官方场景;带图片场景复用仓库已打包资源,垂直按钮通过既有 actionsWidget 组合表达,不扩大基础 API。同步运行 generate_example_code.dart 更新代码片段。
  3. 验证结果(记录于 specs/010-dialog-official-alignment/acceptance.md):Flutter 3.32.0 与 3.47.0 下组件测试(14 tests)与示例测试(3 tests)均 PASS;flutter analyze --fatal-infos --no-pub 0 issues;Dialog 生产源码 LCOV 172/174 = 98.85%(≥95%);生成片段 --check PASS。

待人工/真机验证项(当前保持未完成)

  • 21 个入口在 Example 真实运行时打开、关闭、输入与结果返回的逐项验收;
  • 与小程序同视口尺寸的像素对照(真实运行像素证据待补充)。

📝 更新日志

  • breaking(dialog): 调整 Dialog 默认内容区顶边距(由 32px 调整为 24px)

  • breaking(dialog): 调整 Dialog 关闭按钮位置(由面板顶/右角移至距顶/右各 8px)

  • 本条 PR 不需要纳入 Changelog

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

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

  • 标题遵循 Conventional Commits 格式:type(<scope>?): 修改描述
    示例:fix(TBottomTabBar): 修复 iconText 模式底部溢出
  • “相关 Issue”处带上修复的 Issue 链接或无关联 Issue
  • 已添加对应的 Spec 链接,或已由 Review 确认本次改动无需 Spec
  • 相关文档已补充或无须补充

@github-actions

Copy link
Copy Markdown
Contributor

完成

Web Demo

APK 下载

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