558228 - [k] Change indenting standard in templates

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Software, task, P3)

Description

[James Socol [:jsocol, :james]]

While 4 spaces >> tabs for source, when we have to send the text down the wire, we should be mindful of the extra 3 characters every time.

AMO uses two-space indenting in their templates, and I vote we follow that standard.

Comment 1

[James Socol [:jsocol, :james]]

Paul: r?

http://github.com/jsocol/kitsune/commit/ec22589f1a03ee63b28a0b1742b9d2e14255c707

Fixed indentation and added modelines (and a few small whitespace/indent-level fixes).

Comment 2

[Paul Craciunoiu [:paulc]]

Reviewed. Modelines work, yay! r+, though see my last comment. at http://github.com/jsocol/kitsune/commit/ec22589f1a03ee63b28a0b1742b9d2e14255c707

Comment 3

[James Socol [:jsocol, :james]]

http://github.com/jsocol/kitsune/commit/bd3d5b8c9131b3a08c5dd59a2cc1fc47c0bea0b6

I'm wary about setting a single way to do that without thinking a lot more about it. I don't want to create a document of nothing but special cases. In general: do what's clear.