Render force-push compare diff inline in the PR conversation, not as a separate page #197515
Replies: 2 comments
-
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
🏷️ Discussion Type
Product Feedback
Body
Summary
When a contributor force-pushes to a PR branch, GitHub adds a timeline entry like:
The hash range links to a separate
/compare/<old>..<new>page, which navigates the reviewer away from the PR. Reviewing what changed between iterations requires opening it in a new tab and context-switching back to the PR.Request
Make the compare diff render inline as a collapsible/toggleable section within the PR conversation timeline, similar to how review comments show diff context inline.
Why this matters
The current friction is one of the main reasons maintainers ask contributors to push incremental commits instead of force-pushing clean revisions. If reviewing force-push iterations were as easy as expanding a section in the conversation, more projects could allow contributors to maintain clean commit histories during review.
Example
Today:
8139a28...ef78ad8→ navigates to a separate compare pageProposed:
8139a28...ef78ad8→ expands an inline diff view inside the PR conversation, collapsible like outdated review commentsRelated
Beta Was this translation helpful? Give feedback.
All reactions