3 ms·
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 in
by erichurkman 11y ago
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.
- nathankleyn 11y agoSome 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.
- wclax04 11y agoI REALLY wish I could exclude certain file types from the view.
- tyingq 11y agoThis! ?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/1e3869cb398ddf58d3d52efd735067093dc5bf3e https://github.com/douglascrockford/JSON-js/commit/1e3869cb3... (133 additions and 127 deletions) https://github.com/douglascrockford/JSON-js/commit/1e3869cb398ddf58d3d52efd735067093dc5bf3e?w=1 https://github.com/douglascrockford/JSON-js/commit/1e3869cb3... (30 additions and 24 deletions)
- xPaw 11y agoI 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 https://gist.github.com/xPaw/de6ee132a2e267ef6960
- mwarkentin 11y agoOn top of that.. when you're in `?w=1` view you can't leave line-by-line comments. :facepalm:
- alxndr 11y agoI 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.