This is an incredibly simple feature which to me is "obvious" but after so many module iterations have been left out still. Am i overlooking an obvious detail of why this was left out ?

Please include this in the module as it makes code much more readable.

thx.

CommentFileSizeAuthor
#3 Screen Shot 2013-06-04 at 01.13.23.png30.12 KBcameron tod
codefilter.pre_.css_.patch474 byteslitwol

Comments

litwol’s picture

litwol commented
Title: div.codefilter should preserve whitespaces22 » div.codefilter should preserve whitespaces
haysuess’s picture

haysuess commented

I can't believe this needs a patch also!

Except it should be:

white-space: pre-wrap; so it preserves them and wraps them so the text doesn't flow outside the code area.

cameron tod’s picture

cameron tod commented
Status: Active » Closed (won't fix)
StatusFileSize
new Screen Shot 2013-06-04 at 01.13.23.png30.12 KB

I don't think this is required, it should be handled in codefilter_fix_spaces().

Here's an example:
Codefilter whitespace preserved

A bigger issue perhaps is that this module hasn't had a recommended D6 release since 2008 (!). Part of that is that it is used on Drupal.org, so any update will be subject to the scrutiny of the entire Drupal community. The 6.x-1.x codebase has several improvements, so I need to work on getting a 1.1 out.