Skip to content

改进第四组交互质量所需的 UI 基础能力 - #54

Merged
xiaoland merged 1 commit into
mainfrom
fix/dropdown-search-affordance
Sep 23, 2026
Merged

xiaoland merged 1 commit into
mainfrom
fix/dropdown-search-affordance

Conversation

@xiaoland

Copy link
Copy Markdown
Member

目的与范围

这是 client-web 第四组交互质量的 UI 生产者依赖,消费任务为 InKCre/client-web#118,用户教程继续在 InKCre/docs#31 更新;不单独扩大为 Wizard 或异步请求框架。

  • Dropdown 搜索区增加使用 spacing Token 的留白、搜索图标与可点击聚焦区域。
  • 保留三点式内容加载,补充紧凑 spinner、可见状态名称和简单 InkSkeleton;等待动画响应 reduced-motion。
  • Dialog 的交互禁用与按钮自身 loading 分离。保存转圈,取消仅禁用;默认和自定义 footer 均保留关闭锁与失败恢复。
  • 五个 spacing Token 使用 rem;已有 Figma 值导入拒绝单位意外回退。
  • 同步 Story、设计标准、迁移说明、Skill 源与生成物、Changesets。

验证

完整 pnpm check 通过,包括类型、lint、生成物、Token 导入夹具、独立 tarball 安装与公开入口、23 个 Story / 125 个 Variant 构建。

实际构建浏览器检查:自定义 footer 保存时仅保存按钮转圈,取消禁用;Escape 不能关闭提交中的 Dialog;失败保留草稿并恢复按钮,重试可以成功。Loading 的可访问状态名可唯一定位。实际 Web 源码联调已验证 Settings Dropdown 的搜索图标、留白、桌面与 360px 容器表现;完整隔离数据库 E2E 11/11 通过,包括默认/自定义 Dialog 的取消按钮不转圈。

这些 Web 结果是源码联调证据,不是正式包或公开 client-web preview 验收。需要先通过正常 Version PR 发布,再由 client-web 锁定正式版本复验。

风险与回退

Dialog 自定义动作过去隐式继承 spinner,现在必须由执行按钮显式绑定 isLoading;交互禁用仍由 Dialog 提供,迁移说明已列出。rem 在默认 16px 根字号下维持既有间距,较大根字号随之扩大。消费者应复核长文本与窄容器。

无数据库迁移、凭据或生产部署配置变化。回退时先撤销消费者对新组件的使用,再回退 UI 版本;不从分支发布包。

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying inkcre-web-design with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0b5d9da
Status: ✅  Deploy successful!
Preview URL: https://3ce6e304.design-dd4.pages.dev
Branch Preview URL: https://pr-54.design-dd4.pages.dev

View logs

@xiaoland
xiaoland marked this pull request as ready for review September 23, 2026 13:19
@xiaoland
xiaoland merged commit c3f2370 into main Sep 23, 2026
3 checks passed
@xiaoland xiaoland mentioned this pull request Sep 23, 2026

This branch was successfully deployed

1 active deployment
preview — 0b5d9dae Deployed Sep 23, 2026 by xiaoland via ui-web preview cleanup #20
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