Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#17,944 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
82/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
react, typescript
Lĩnh vực
frontend

Hướng nghiên cứu

Đọc apps/web và kiểm tra PullRequestDetailPanel.tsx, tập trung vào các nút merge và auto-merge cùng kiểu dáng gọn của chúng trong phần đầu trang. Tái hiện sự cố với bảng chi tiết PR hẹp và kiểm tra mỗi nút thay đổi thế nào sau lần nhấp đầu tiên. Hoàn thành khi trạng thái đã kích hoạt được phân biệt rõ bằng mắt ở dạng chỉ có biểu tượng của cả hai nút; issue không nêu tên tệp kiểm thử nào.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
TypeScript
Star
24.8k
Fork
6.4k
Merge trung bình
6 giờ 23 phút
Pull request đã merge (30 ngày)
203

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của pingdotgg/t3code

Tất cả issue của pingdotgg/t3code

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.