[Bug]: Compact PR header merge button gives no visible cue that it needs a second click
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 82/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- react, typescript
- 领域
- frontend
调研方向
阅读 apps/web 并检查 PullRequestDetailPanel.tsx，重点关注 merge 和 auto-merge 按钮及其紧凑的页眉样式。使用较窄的 PR 详情面板重现问题，并检查每个按钮在首次点击后如何变化。完成标准是：两个按钮在仅显示图标的形式下，已启用状态都能明显区分；该 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 分钟
- 30 天内合并 PR
- 215
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 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可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭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 天内回复
相似的 Issue
-
afk-ok area:data-quality importer size:S
难度 2/5 1-3 小时 新手友好度 74/100
enorm-labs/event-junkie#3301 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
one-zero-eight/apply-bot#39 ·
-
难度 2/5 1-3 小时 新手友好度 72/100
bigbluebutton/bbb-ui-components-react#108 ·
维护者通常 2 天内回复
-
area: app type: bug
难度 2/5 1-3 小时 新手友好度 70/100
ExpidetGames/code-reviewer#308 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复