Agaric Design Collective

Ask Agaric: Whitespace at the end of lines in a Drupal module or theme?

By Benjamin Melançon
on 09 Oct
0 comments

Description

Is this annoying or a useful feature. My IDE removes trailing whitespace. The disadvantage is that it takes you more time to see the real changes/fixes.

It *is* a good thing that your editor deletes whitespace at the end of lines, that conforms to coding standards. While your making small typo fixes like that, you should fix the indentation also; I see some things where you removed space at the end but the indentation was left at far too much; each progressive indent should be two spaces (and no tabs used at all).

So while fixing typos, please see:

http://drupal.org/coding-standards

(For committing to Drupal core, however, note that you don't want a patch to do both code cleanup and functionality improvements at the same time— this should be done in separate patches.)

 

Post new comment
The content of this field is kept private and will not be shown publicly.
  • Allowed HTML tags: <a> <em> <strong> <cite> <code> <ul> <ol> <li> <dl> <dt> <dd> <blockquote> <h1> <h2> <h3> <h4> <h5> <h6> <small> <pre> <strike> <sub> <sup> <kbd> <s>
  • Lines and paragraphs break automatically.
  • Web page addresses and e-mail addresses turn into links automatically.
  • You may post code using <code>...</code> (generic) or <?php ... ?> (highlighted PHP) tags.

More information about formatting options

CAPTCHA
This question is for testing whether you are a human visitor and to prevent automated spam submissions.
Image CAPTCHA
Copy the characters (respecting upper/lower case) from the image.