🤖 Artifacts: return focus to the Fullscreen button after fullscreen closes

クローズ 初心者向け
#5,694 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

関連するプルリクエストがすでにマージされています。

  • #5731 @ThomasK33 による — マージ済み

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
80/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
typescript

調査の方向性

ArtifactsPanel.test.tsxから始めて、既存の全画面表示時のフォーカス動作を理解し、その後、全画面表示を開閉するコードを見つけます。起動元がまだ存在する場合はフォーカスがそこに戻り、存在しない場合はパネルに戻ることを完了条件とします。両方のケースを確認するため、テストを実行してください。

索引モデルが issue の本文から書いたものです。

説明

enhancement

Problem: Closing Artifacts fullscreen sends focus to the panel, not to the button that opened it. This is intended today (so J/K keep working, see ArtifactsPanel.test.tsx), but J/K also work from the button, and returning to the opener is the usual dialog behavior.

Fix idea: remember the opener when fullscreen opens and return focus to it if it still exists; otherwise focus the panel.

Priority P3. Size S.

Found by the agent bug bashes in #5670 (pre-existing, outside that PR's changes).


Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high

主要言語
TypeScript
スター
2.1k
フォーク
139
平均マージ
3時間 34分
マージ済み PR(30日)
951

環境構築

  • Dockerfile または Docker Compose ファイルあり
  • プルリクエストのテンプレートなし
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

coder/xum のほかの issue

coder/xum の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。