Right now krumo's string output is wrapped (like HTML), i.e. multiple spaces and newlines are ignored. I see no benefit in removing this information, and I propose to add

white-space: pre;

to the div.krumo-preview style in order to preserve white space.

In fact, I think this is a bug, because if you need to see the text as it is in the variable, you can't...

Files: 
CommentFileSizeAuthor
#1 devel.krumo_.preformatted.1262694.1.patch1.68 KBsalvis
FAILED: [[SimpleTest]]: [MySQL] Invalid patch format in devel.krumo_.preformatted.1262694.1.patch.
[ View ]

Comments

StatusFileSize
new1.68 KB
FAILED: [[SimpleTest]]: [MySQL] Invalid patch format in devel.krumo_.preformatted.1262694.1.patch.
[ View ]

Here's the patch.

The skin.css files all have CR/LFs, so the added line has the CR, too.

Status:Active» Needs review

Anyone care to comment on this?

Status:Needs review» Needs work

The last submitted patch, devel.krumo_.preformatted.1262694.1.patch, failed testing.

Ah, yes, the testbot doesn't like the line endings (see #1). Never mind...

Status:Needs work» Fixed

Committed to 8/7/6.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.