Fixed docs indentation in rootScope.js

[Zizzamia]

No description provided.

[petebacondarwin]

LGTM

[caitp]

Not LGTM --- I don't think this is the right fix for this :( Fixing the trimming behaviour in dgeni would make more sense. Otherwise the code looks really weird.

[caitp]

@petebacondarwin --- Why don't we pick out the leading whitespace from each line in each sample, and remove the length of the shortest one from each line which starts with it? It would take more time to process, but we'd get correct results without making the codebase look weird. Given that code snippets are typically not very long, it seems reasonable to me.

[petebacondarwin]

@caitp - we do this already with jsdoc comments but I am sure there was some reason why it didn't work for code blocks. One, perhaps more attractive, solution is to leave the stars at the start of each line:

  /*
   * ```
   * someCode();
   * ```
   */

Since we strip off leading whitespace and stars.

I'll have a look at the dgeni code again.

[petebacondarwin]

So I looked at the code again. If there is a dgeni template that uses nunjucks code or marked "tags" then their contents will have its indentation trimmed. E.g. only the underlined whitespace below will be trimmed:

{% code %}
____function doStuff() {
____  SomeCodeThatIsIndented();
____}
{% endcode %}

If there is an inline example, using <example> tag then each file will have its indentation trimmed. E.g. only the underlined whitespace will be trimmed:

<example>
   <file ...>
____<div>
____  Some file contents
____</div> 
  </file>
</example>

But at the moment the code blocks within markdown (indicated by ```) are not being trimmed since dgeni is not aware of them. They are passed straight through to the marked parser.

[petebacondarwin]

Looks like there might be a hook in marked that can let us trim the code: https://github.com/chjj/marked#block-level-renderer-methods

[petebacondarwin]

Well it will be closed when PR #9093 lands.

[petebacondarwin]

Now its landed!