[k] Change indenting standard in templates

RESOLVED FIXED in 2.0

Status

support.mozilla.org
Knowledge Base Software
P3
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: jsocol, Assigned: jsocol)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

(Assignee)

Description

8 years ago
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.
(Assignee)

Comment 1

8 years ago
Paul: r?

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

Fixed indentation and added modelines (and a few small whitespace/indent-level fixes).
Reviewed. Modelines work, yay! r+, though see my last comment. at http://github.com/jsocol/kitsune/commit/ec22589f1a03ee63b28a0b1742b9d2e14255c707
(Assignee)

Comment 3

8 years ago
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.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Summary: Change indenting standard in templates → [k] Change indenting standard in templates
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.