[Bug]: Escape does not return to the previous thread from Pull Requests or Usage
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 23k
- Forks
- 5.9k
- Avg merge
- 11h 14m
- Merged PRs (30d)
- 357
Description
Before submitting
- I searched existing issues and did not find a duplicate for the Pull Requests page.
- I included enough detail to reproduce or investigate the problem.
Area
apps/web
Steps to reproduce
- Open a thread in T3 Code.
- Navigate to the Pull Requests page or the Usage page.
- With no dialog or popover open, press Escape.
- Repeat for the other page.
- Compare with opening Settings from a thread and pressing Escape.
Expected behavior
Escape should return to the thread I was working in, consistent with Settings. If a dialog or popover is open, Escape should dismiss it first without leaving the page.
Actual behavior
Escape does not return from either Pull Requests or Usage to the previous thread. The same action works from Settings, making keyboard navigation inconsistent.
Impact
Minor bug or occasional failure. Returning to the thread requires manual navigation.
Related work
- #1350 requested the existing Escape-to-go-back behavior in Settings.
- Open PR #10158 already adds Escape navigation to Usage. This report records the current inconsistency on both pages; the Usage portion overlaps with that PR, while Pull Requests remains uncovered.
Workaround
Select the previous thread in the sidebar.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in apps/web by comparing Escape handling on Pull Requests and Usage with the working Settings behavior, and review the overlap with PR #10158. Verify that Escape first dismisses an open dialog or popover, otherwise returns to the previous thread from both pages, without breaking the existing Settings behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100