Page MenuHomePhabricator

Replace Utilities::convertWhiteSpaceToHTML() with CSS
Open, MediumPublic

Description

Utilities::convertWhiteSpaceToHTML() is the same as the CSS white-space:pre-wrap, except that it pollutes the HTML code, which can confuse tools. Unless there’s a reason to keep it, it should be replaced by appropriate CSS styles, which makes the PHP code simpler and the output easier to work with. (The code was first introduced in rSVN29017 (code review), without much elaboration.)

Event Timeline

It's definitely not same as nowrap. It might be replaced by some other value that preserves white-space.

Sorry, typo, fixed. (The link target was correct, only the link text had a typo.)