[Bug]: Compact PR header merge button gives no visible cue that it needs a second click
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 82/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- react, typescript
- 領域
- frontend
調査の方向性
apps/webを読み、PullRequestDetailPanel.tsxを調べます。特に、mergeボタンとauto-mergeボタン、およびヘッダーのコンパクトなスタイルに注目します。幅の狭いPR詳細パネルで問題を再現し、最初のクリック後に各ボタンがどう変化するかを確認します。両方のボタンで、アイコンのみの表示でも armed 状態が視覚的に区別できれば完了です。このissueではテストファイルは指定されていません。
索引モデルが issue の本文から書いたものです。
説明
Before submitting
- I searched existing issues and did not find a duplicate.
- I included enough detail to reproduce or investigate the problem.
Area
apps/web
Steps to reproduce
- Open a mergeable pull request in the PR detail panel.
- Make the panel narrow enough that the header buttons collapse to icon only (
@max-[30rem]/pr-header). - Click the header Merge button (or Enable auto-merge) once.
Expected behavior
The button visibly shows that it is armed and a second click will merge, even in its icon-only form.
Actual behavior
Since #17710, the first click arms the button and the only visible change is the label switching to "Confirm merge?" / "Confirm auto-merge?". In the compact header that label is hidden (@max-[30rem]/pr-header:hidden in PullRequestDetailPanel.tsx), so the button looks unchanged. Apart from the hover tooltip, nothing tells you a second click is needed, and the armed state expires after 3 seconds. The first click looks like it did nothing.
Approve workflows and the menu items are not affected because their labels never hide.
Impact
Minor bug or occasional failure
Version or commit
main @ 1f093d119b (#17710) and later
Proposed fix
Show the armed state in a way that survives the icon-only form. For example, swap the merge glyph for a confirm icon (e.g. CheckIcon) while armed, alongside the existing label change. Revert already switches to the destructive variant when armed, which would be another option. The change is limited to those two buttons in PullRequestDetailPanel.tsx. I'm happy to open a PR once the direction is approved.
- 主要言語
- TypeScript
- スター
- 24.8k
- フォーク
- 6.4k
- 平均マージ
- 5時間 30分
- マージ済み PR（30日）
- 215
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pingdotgg/t3code のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
[Bug]: Diff panel hides the current branch name and the arrow overlaps the base ref when the panel is narrow対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンaccepted bug via-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
pingdotgg/t3code#19048 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
[Bug] Windows repository cloning fails while creating an existing drive root対応中かも @pujitha24 が 3 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
[Bug]: iOS still shows "Failed to connect. Retrying" on every resume (follow-up to #5695)対応中かも @otavio が 2 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
pingdotgg/t3code の issue をすべて見る
似ている issue
-
useFragment does not require `variables` when the fragment's variables type has required fieldsオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
apollographql/apollo-client#13493 ·
メンテナーはふだん 1 日以内に返信
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backends対応中かも @roboomp が今日担当しました。 オープンbug prio:p2 providers triaged
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
can1357/oh-my-pi#15798 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
code-yeongyu/oh-my-openagent#10172 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
PDFCraftTool/pdfcraft#189 ·
-
Add the Yahoo chart API duplicate last-day row (yfinance#573) to incidents.md (about 1 hour)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンdata docs good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 80/100
pacocartones/market-data-sanity-checker#160 ·
メンテナーはふだん 1 日以内に返信