Add template-indent rule

[mmkal]

Fixes #580

Todo:

• Fill out markdown docs
• Support comments

[fisker]

I don't see why it couldn't go in separately.

Because your template literal can be anywhere, use something like /* HTML */ is easier than write a selector.

Imaging

foo.innerHTML = /* HTML */ `
// ...
`;

$(foo).html(/* HTML */ `
// ...
`)

div.insertAdjacentHTML(
	'beforebegin',
	  /* HTML */ `
	  // ...
	  `
);

Object.assign(div, {
	innerHTML: /* HTML */ `
	`,
});

You don't have to list all those cases in selectors, and it can work with Prettier .

It's much more complex

It's very simple, use sourceCode.getTokenBefore(..., {includeComments: true}), and check that comment.

[mmkal]

@fisker anything else you feel should be added here?

[fisker]

I'll take a look ASAP.

[fisker]

We need test for

foo = outdent`
before
before${
expression
}after
after
`;

foo = outdent`
  before
  before${
expression
}after
  after
`;

foo = outdent`
  before
  before${
	outdent`
inner
	`
}after
  after
`;

foo = outdent`
  before
  before${
	normalTemplate`
inner
	`
}after
  after
`;

[mmkal]

@fisker added tests in 84ada62. Note that this rule is unopinionated about how parent variables are indented, and only cares about the template literals themselves, so

outdent`
  before
  before${
	outdent`
inner
	`
}after
  after
`;

becomes

outdent`
  before
  before${
	outdent`
		inner
	`
}after
  after
`;

etc. even though the outer template is indented with spaces and the inner with tabs. It should work nicely with other rules that do care about the indentation level of the parent, like prettier or whatever. Good call to make sure this behaviour doesn't get regressed though. 👍

[fisker]

LGTM