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...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

salvis’s picture

Here's the patch.

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

salvis’s picture

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.

salvis’s picture

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

salvis’s picture

Status: Needs work » Fixed

Committed to 8/7/6.

Status: Fixed » Closed (fixed)

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