On: More code review tools

[erichurkman]

I'd love a UI way to enable `?w=1` whitespace mode, especially for html/json/yaml files. I have a feeling a lot of reviewers don't know about it and spend an inordinate amount of time squinting to see what changes when it was a whitespace-only change.

[tyingq → erichurkman]

This! ?w=1 has existed for years, with no UI way to access it.

Compare two views of the same commit (found this by searching for any commit on github with the word "indent" in it)

https://github.com/douglascrockford/JSON-js/commit/1e3869cb3... (133 additions and 127 deletions)

https://github.com/douglascrockford/JSON-js/commit/1e3869cb3... (30 additions and 24 deletions)

[mwarkentin → erichurkman]

On top of that.. when you're in `?w=1` view you can't leave line-by-line comments. :facepalm:

[xPaw → erichurkman]

I have a userscript that does just that, however it needs to be updated to account for these new review tools.

https://gist.github.com/xPaw/de6ee132a2e267ef6960

[nathankleyn → erichurkman]

Some similar option for enabling patience diffing as well would be amazing. Those two options in combination would certainly make a not insignificant amount of PRs much more readable.

[alxndr → erichurkman]

I created a reminder for myself so that every six months I write an email to GitHub asking for someone to create a user option for this whitespace setting.

[wclax04 → erichurkman]

I REALLY wish I could exclude certain file types from the view.