[Dialog] 支持焦点落点可配置、Enter 键确认与焦点循环(含 DialogPlugin 对齐)
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 18/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- react, typescript
- 領域
- frontend
調査の方向性
Start with dialog/hooks/useDialogEsc.js, where activateDialog() currently focuses only the .t-dialog__ctx container, then dialog/DialogCard.js (renderConfirmBtn hardcodes the primary theme and renderFooter passes custom footers through unchanged), and config-provider/ConfigContext.js for the dialog defaults. Done means defaultFocus, Enter-to-confirm with its textarea, IME and confirmLoading guards, and Tab focus trap work the same for Dialog and DialogPlugin, with a stable data-role="confirm" marker on the primary button.
索引モデルが issue の本文から書いたものです。
説明
这个功能解决了什么问题
环境
- tdesign-react: 1.18.2
- React: 18
- 浏览器: Chrome 140 / Safari 18(均复现)
背景与诉求
- 不同弹窗诉求不同:二次确认希望焦点落「确定」,危险操作(删除/锁定)希望落「取消」,表单类弹窗希望落输入框。落点应由参数决定,并支持全站默认值。
- 可配置的键盘确认:只要弹窗打开,按 Enter 就应等同于点击「确定」。当前焦点默认落在弹窗容器上,回车完全无响应,键盘链路断裂。
期望行为
- 焦点落点可配置:提供
defaultFocus,支持'confirm' | 'cancel' | 'none' | (() => HTMLElement | null);组件默认值建议保持现状('none')以兼容存量,但通过globalConfig.dialog.defaultFocus可全站统一策略。 - Enter 触发确认,与焦点解耦:弹窗打开期间按 Enter 等同于点击「确定」,不要求焦点位于主按钮;Esc 仍关闭。需内置边界:焦点在
textarea/可换行输入控件、IME 组合输入过程中不触发,confirmLoading期间不重复触发。可用confirmOnEnter开关。 - Tab 焦点循环:Tab / Shift+Tab 在弹窗内循环,不逃逸;关闭后焦点归还触发元素(
restoreFocus)。 - 上述能力在声明式
Dialog与命令式DialogPlugin上行为一致。
你建议的方案是什么
建议 API(P0,其余能力可另开 issue)
interface DialogProps {
/** 打开后焦点落点;组件默认建议保持现状('none'),由业务按需指定 */
defaultFocus?: 'confirm' | 'cancel' | 'none' | (() => HTMLElement | null);
/** Enter 触发确认,默认 true */
confirmOnEnter?: boolean;
/** 焦点循环,默认 true */
focusTrap?: boolean;
/** 关闭后焦点归还触发元素,默认 true */
restoreFocus?: boolean;
}
配套要求:
globalConfig.dialog暴露defaultFocus/confirmOnEnter/focusTrap的默认值,便于全站统一;- 为「主操作按钮」提供稳定语义标记(例如统一
data-role="confirm",自定义 footer 下由业务显式声明),让defaultFocus: 'confirm'与 Enter 确认在任何 footer 形态下都能命中,而不是靠类名匹配。
根因(源码定位)
dialog/hooks/useDialogEsc.js的activateDialog()只做lastDialog.current.focus(),把焦点交给容器.t-dialog__ctx,既不涉及按钮,也没有任何落点配置。- 组件层只监听 Esc,无 Enter、无 focus trap;
autofocus在 Dialog 内不生效(参考 #2794),消费方只能在onOpened里补刀,且开场动画期间 TDesign 会抢焦点,时序脆弱。 dialog/DialogCard.js的renderConfirmBtn硬编码theme: "primary"(theme="danger"弹窗也渲染 primary),renderFooter对自定义footer直接parseTNode,不提供「主操作按钮」语义标记。于是:- 默认 footer:主按钮是
.t-button--theme-primary; - 自定义 footer:业务常用
<Button theme="danger">,类名是.t-button--theme-danger。
我们的补丁只认--theme-primary时,一批自定义 footer 弹窗(如分支锁定)主按钮依然拿不到焦点——只要组件不给语义,消费方就只能不断猜选择器。
- 默认 footer:主按钮是
config-provider/ConfigContext.js中dialog.confirmBtnTheme已声明默认值,但全量检索es/无任何消费方,属未生效配置。
我们当前的 workaround(供参考,非长期方案)
在业务封装层:① onOpened 里按类名猜主按钮并 focus();② 在 document 上挂 keydown 捕获实现 Tab 循环;③ 用 :focus 补焦点环(程序化 focus 不命中 :focus-visible,Button 无可见焦点);④ 对 DialogPlugin.confirm 做同构包装复制同一套逻辑。
Enter 确认目前只能自己绑,还得自行判断 textarea、IME 组合输入、confirmLoading 等边界,规则与组件内部(如 Esc 的 dialogStack)不同步就会出现误触发或漏触发。这套补丁必须随组件实现变化持续维护,因此希望能力下沉到组件层。
- 主要言語
- TypeScript
- スター
- 973
- フォーク
- 379
- 平均マージ
- 4日 7時間
- マージ済み PR(30日)
- 15
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Tencent/tdesign-react のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Tencent/tdesign-react#4465 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
[Table] reserveSelectedRowOnPaginate=false 时表头全选包含其他页数据対応中かも @dvd233 が 32 日前に担当しました。 オープン🐞 bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Tencent/tdesign-react#4416 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
-
[Chatbot] 似乎不支持ag-ui中的interrupts再び着手できるかも @LzhengH が 52 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン💪🏻 enhancement
Tencent/tdesign-react#4400 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
💪🏻 enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
Tencent/tdesign-react#4253 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
💪🏻 enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
Tencent/tdesign-react#4246 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
Tencent/tdesign-react の issue をすべて見る
似ている issue
-
Add: Shop On TV SDオープンcheck:passed streams:add
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信
-
bug implicit-state
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
book-layout.spec fails when the similar-books rail holds other projects' books対応中かも @praetorianer777 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
praetorianer777/GoTome#373 ·
メンテナーはふだん 1 日以内に返信
-
area:cards enhancement ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
fil-donadoni/tolaria#8883 ·
メンテナーはふだん 1 日以内に返信
-
xsd:integer cast from a string rejects negative numbers, a leading + and surrounding whitespaceオープンbug 🐛
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
comunica/comunica#1960 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信