Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

[Bug]: Compact PR header merge button gives no visible cue that it needs a second click

Chiusa Adatta ai principianti
  • Qualcosa non va?
#17,944 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
react, typescript
Ambito
frontend

Direzione di ricerca

Leggi apps/web ed esamina PullRequestDetailPanel.tsx, concentrandoti sui pulsanti di merge e auto-merge e sul loro stile compatto nell’intestazione. Riproduci il problema con un pannello dei dettagli della PR stretto e verifica come cambia ciascun pulsante dopo il primo clic. Il lavoro è completato quando lo stato armato è visibilmente distinto nella versione con la sola icona per entrambi i pulsanti; l’issue non indica alcun file di test.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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
  1. Open a mergeable pull request in the PR detail panel.
  2. Make the panel narrow enough that the header buttons collapse to icon only (@max-[30rem]/pr-header).
  3. 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.

Lingua principale
TypeScript
Stelle
24.8k
Fork
6.4k
Merge medio
7h 9m
PR unite (30g)
244

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di pingdotgg/t3code

Tutte le issue di pingdotgg/t3code

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.