Esc: gaps in tooltip and notification dismissal
メンテナーはふだん 1 日以内に返信
評価
この issue はまだ評価されていません。
説明
Problem
Three gaps in the Esc handling added in #925 and #931:
- A tooltip reopens after Esc. While the target has keyboard focus, move the pointer off and back onto it twice within
tooltipOpenDelay, then press Esc. The tooltip closes, then shows again. - Esc closes the tooltip instead of a newer notification. A notification shown after a tooltip opened should take the Esc first. It does when its container is new, but not when it is added to a container that is already open and on top.
- Esc from a closed tree-select doesn't close notifications. With focus on a closed
cps-tree-select, Esc leaves open notifications as they are.
Cause
cpsTooltiponMouseEnter()schedules a new show timer without clearing the pending one, and Esc clears only the latest.cpsTooltiprecognizes an overlay opened after it by its element and z-index. A container that is already on top keeps both when a notification is added.cps-tree-selectonKeyDown()callspreventDefault()for every key except Tab, including Esc while the options are closed. The notification service skips an Esc that is already handled.
Proposal
- Clear the pending show timer before scheduling another.
- Let the notification container record each added notification, and have the tooltip compare that with what it saw when it showed.
- Mark Esc as handled in tree-select only when it closes the options, like
cps-tree-autocomplete.
- 主要言語
- TypeScript
- スター
- 9
- フォーク
- 8
- 平均マージ
- 1日 1時間
- マージ済み PR（30日）
- 23
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
AbsaOSS/cps-shared-ui のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
AbsaOSS/cps-shared-ui#875 ·
メンテナーはふだん 1 日以内に返信
-
InputComponent - expand type beyond text/number/password再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンenhancement library
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
AbsaOSS/cps-shared-ui#644 ·
メンテナーはふだん 1 日以内に返信
-
cps-table/tree-table: changing rows-per-page on an empty table doesn't emit対応中かも @lukasmatta が 3 日前に担当しました。 オープンbug library
AbsaOSS/cps-shared-ui#944 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
AbsaOSS/cps-shared-ui#940 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
AbsaOSS/cps-shared-ui#939 ·
メンテナーはふだん 1 日以内に返信
AbsaOSS/cps-shared-ui の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
microsoft/vscode-containers#655 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
dsh-tauri-desk/deepseek-harness-desktop#1477 · コメント 1 件 ·
-
[docs] RELEASE.md: wrong branch name, no check that the prepare PR merged with the right subject, manual publish step対応中かも @albertteoh が今日担当しました。 オープンdocumentation
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
jaegertracing/jaeger-ui#4739 ·
メンテナーはふだん 1 日以内に返信
-
Add: Shop On TV SDオープンcheck:passed streams:add
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信