logoalt Hacker News

embedding-shapetoday at 1:47 PM1 replyview on HN

> The improvements to PR review have been nice though

I dunno, probably the worst UX downgrade so far, almost no PRs are "fully available" on page load, but requires additional clicks and scrolling to "unlock" all the context, kind of sucks.

Used to be you loaded the PR diff and you actually saw the full diff, except really large files. You could do CTRL+F and search for stuff, you didn't need to click to expand even small files. Reviewing medium/large PRs is just borderline obnoxious today on GH.


Replies

epistasistoday at 1:59 PM

I find it impossible to use the current diff view for most codebases, and spend tons of time clicking open all available sections...

They have somehow found the worst possible amount of context for doing review. I tend to pull everything down to VS Code if I want to have any confidence these days.