Page MenuHomePhabricator

diff.css uses the proprietary -moz-pre-wrap instead of the cross-browser, standards-compliant pre-wrap
Closed, InvalidPublic

Description

Author: rememberthedot

Description:
Line 36 of diff.css states "white-space: -moz-pre-wrap;"

Please change this to "white-space: pre-wrap;" which works on all standards-compliant browsers and not just Mozilla-based ones.


Version: unspecified
Severity: enhancement

Details

Reference
bz13797

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:08 PM
bzimport set Reference to bz13797.
bzimport added a subscriber: Unknown Object (MLST).

rememberthedot wrote:

Here is a link to the relevant CSS 2.1 specification page: http://www.w3.org/TR/CSS21/text.html#white-space-prop

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

rememberthedot wrote:

Sorry, I should have noticed that. It probably wouldn't hurt to remove -moz-pre-wrap anyway because there are very few Mozilla browsers left that don't understand the standard pre-wrap.

rememberthedot wrote:

Sorry again, I see that pre-wrap isn't yet implemented in Firefox. I thought it was. See http://developer.mozilla.org/en/docs/CSS:white-space

Just forget I ever filed this bug.