Handling unidentified/death keys in the keyboard history UI
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 48/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
- 領域
- frontend
調査の方向性
Start at app/static/js/app.js line 72, where the keyboard history relies on KeyboardEvent.key. Reproduce the alt+L case with the documented MacOS layout across Chrome, Safari, and Firefox, then verify that handling for Unidentified and Dead keys produces consistent, useful history in the UI without changing backend or HID behavior.
索引モデルが issue の本文から書いたものです。
説明
I noticed one quirk in the UI, regarding the keypress history in the status bar. Note: this only affects the UI, not the backend or HID interface!
When the client machine is configured to use a keyboard layout that doesn’t provide a mapping for certain keys or key combinations, the keyboard history shows inconsistent results across browsers. From the view point of the target machine, however, things work as expected. So this issue is specific to using non-EN-US keyboard layouts.
I tested this on a MacOS machine with a customised keyboard layout, which doesn’t define a mapping for the alt+L key combination. So pressing alt+L on the client machine wouldn’t produce any character on the client machine itself. However, on the target machine, the alt+L combination may be well-defined, e.g. if the target machine uses a German keyboard layout where alt+L maps to the @ character. In this scenario, TinyPilot would forward the alt+L combination to the target machine, and produce the @ there.
In the TinyPilot’s web UI, however, I see these results in the status bar:
- Chrome displays:
alt+Unidentified(note: it’s notundefined!) - Safari displays:
alt+Dead - Firefox displays:
alt+l(as I would expect)
As stated in the documentation, the key property of the KeyboardEvent object may be Unidentified or Dead:
- If the KeyboardEvent represents the press of a dead key, the key value must be "
Dead".- If the key cannot be identified, the returned value is
Unidentified.
For the keyboard history, we rely on that key property, which is why we see this.
- 主要言語
- Python
- スター
- 3.5k
- フォーク
- 291
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
tiny-pilot/tinypilot のほかの issue
-
bug medium
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
tiny-pilot/tinypilot#1419 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
tiny-pilot/tinypilot#1929 · コメント 3 件 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
tiny-pilot/tinypilot#1899 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
tiny-pilot/tinypilot#1896 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
tiny-pilot/tinypilot#1882 ·
tiny-pilot/tinypilot の issue をすべて見る
似ている issue
-
agent-ready documentation needs-triage
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
-
documentation
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" オープン
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
-
instance instance add
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
searxng/searx-instances#939 · コメント 1 件 ·
-
area-deployment area-integrations triage:bot-seen
難易度 2/5 半日 初心者へのやさしさ 86/100