Page MenuHomePhabricator

Diffs do not indicate change in leading whitespace
Closed, ResolvedPublic

Description

Author: timwi

Description:
BUG MIGRATED FROM SOURCEFORGE
http://sourceforge.net/tracker/index.php?func=detail&aid=959622&group_id=34373&atid=411192
Originally submitted by Gordon Tran (buffered) 2004-05-24 20:24

When adding the leading space for preformatted text,
the diff will attempt to highlight the new space in red,
which has no visible effect.

http://test.wikipedia.org/w/index.php?
title=TestWikipedia:Sandbox&curid=19088&diff=2213&oldi
d=2212


Version: unspecified
Severity: normal

Details

Reference
bz286

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 7:11 PM
bzimport set Reference to bz286.
bzimport added a subscriber: Unknown Object (MLST).

timwi wrote:

*** Bug 305 has been marked as a duplicate of this bug. ***

ayg wrote:

This can be fixed with

.diffchange { text-decoration: underline; }

or

.diffchange { background-color: something other than default; }

plus

td.diff-addedline, td.diff-deletedline, td.diff-context {

white-space: pre-wrap;
white-space: -moz-pre-wrap;

}

The latter point (which is a necessary component) will only work on some
browsers, however, such as Opera and Gecko-based browsers. For a universal fix,
spaces would have to be converted automatically to  . A stylesheet
solution packaged in by default would be better than nothing, though, in my
opinion, and next-gen browsers will hopefully all support pre-wrap (although IE
7 doesn't yet, at least).

  • This bug has been marked as a duplicate of bug 13466 ***