---
# System prepended metadata

title: GitHub PR Review UI Feedback

---

# GitHub UI Feedback

### General / Cross-cutting

- Please let us control (hide/show, disable/enable) *which* backlinks are displayed on an issue or PR.
    - Somebody in a random repo pushing a commit referencing this issue/PR almost certainly has nothing to do with us, and just becomes useless noise.

### Reviews:

- Confusing when reviewing commit-by-commit (thanks to the PR authors who try very hard to make atomic commits and make reviews easier):
    - The UI does not make it very clear which review comment belongs to which commit
- There's too much whitespace: even a single line comment takes up a *bunch* of space, if the PR has more than a few comments, then it means a *ton* of scrolling.
- There's no fine-grained support for marking a review comment as blocking or non-blocking.
- When in PR review view, large diff gets collapsed by default, but I still need to read the diff anyway?

### Pull requests

- The magic word "fixes" or "closes" etc. is very subtle in the PR description and can be hard to notice. Sometimes I might write "would close #xxx" or something to that effect and the magic behavior would kick in.
    - Can we opt-in to magic phrases?
- Stacked PRs and Reviews
    - It's a massive pain currently to manage a series of PRs, that are rebased on top of each other. 

### Issue/Pull Request Tracker:
- Allow sorting by last commit/last review
    - Last activity for issues/pr is considered to be any activity - which includes comments and label changes which results in difficulty for people triaging them. 


### Feature nice-to-haves: [concerns that are not important ]
- Adding label changes as events to the pr 
    - Labels can automatically change on issue/pr events such as CI failing, reviewed, new commits added.
- Mutually exclusive labels / "state machines"?
- Blocking and unblocking of issues/prs on other issues/pr including those external to the repository 
- Comment *trees*: the current linear feed of comments make it very hard to follow discussions especially if there are >2 people involved.
- Allow a reason to be added directly to an issue/pr being closed rather than having to add a separate comment
- Allow to filter out comments from an issue/pr so you can view only - comments by author and/or reviewer,  label changes, etc 
 
### Already raised (ignore this section)

- "Load more" is hostile to every stakeholder: to the issue/PR author, the reviewers, the triage people, to users.
    - Maybe allowing *folding*/*hiding* the set of stuff you don't want to see makes more sense. 