1404: Need a way to supress or hide whitespace-only diffs

shanka******@gmai***** (Google Code) (Is this you? Claim this profile.)
Nov. 22, 2009
What version are you running? Beta


What's the URL of the page containing the problem? Intranet


What steps will reproduce the problem?
1. View a diff which has hundreds of whitespace-only changes (e.g. trailing
blanks removal, or trivial re-indenting), but only 1 or 2 "real" differences
2.
3.

What is the expected output? What do you see instead?

   Trying to find that "real" difference (i.e. where more than
   just whitespace has been changed) is really difficult. 

   What I'd like to see is either an option to suppress or fold
   whitespace-only diff blocks, or some key/button to jump to
   "next non-whitespace diff".

What operating system are you using? What browser?

   Firefox on Windows.

Please provide any additional information below.

   We have some aggressive "style cops" around here who force
   people to strip out all existing trailing spaces from any
   file they edit. (Or else, they turn that flag on in Eclipse
   or Visual C++). Makes reviewing those changes a hellacious
   experience..
david
#1 david
In 1.1, there's a "hide whitespace changes" toggle at the bottom right of the header
box with all the review request info.
  • +NotABug