Closed Bug 754675 Opened 12 years ago Closed 5 years ago

Highlight whitespace changes

Categories

(bugzilla.mozilla.org :: Splinter, enhancement, P5)

Production
enhancement

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: GPHemsley, Unassigned)

References

Details

As a stopgap measure until bug 654446 (or as an addition to it), it would be useful to highlight when whitespace is involved in a change. I know there are some differs that do this by changing the color of the background, which I think would be useful. I also expect it would be somewhat easier to implement than the full-fledged whitespace-excluder, and it would still go a long way towards speeding up whitespace-heavy reviews.
bug 694304 is related to this request.
(In reply to Byron Jones ‹:glob› from comment #1)
> bug 694304 is related to this request.

I'd go so far as to say it blocks this request.
Depends on: 694304
It appears that this functionality was added at some point, at least for trailing whitespace.

Not sure if it's desired to do the same for all whitespace on a line (e.g. indentation, etc.) or whether this bug can be closed.
(In reply to Gordon P. Hemsley [:gphemsley] from comment #3)
> It appears that this functionality was added at some point, at least for
> trailing whitespace.

indeed, bug 776897 implemented trailing whitespace highlighting.

> Not sure if it's desired to do the same for all whitespace on a line (e.g. indentation,
> etc.) or whether this bug can be closed.

personally i don't see value in highlighting non-trailing whitespace, but then again i didn't raise this request :)  if you feel trailing whitespace highlighting only is sufficient, feel free to WONTFIX this bug.
Priority: -- → P5

Splinter is retiring.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.