From 3f89aeb80a4bbea8326558c557a6c17996651abe Mon Sep 17 00:00:00 2001
From: Pete Bacon Darwin <pete@bacondarwin.com>
Date: Fri, 5 Oct 2018 13:50:29 +0100
Subject: [PATCH] build(docs-infra): update CLI option rendering (#26272)

PR Close #26272
---
 aio/src/styles/2-modules/_cli-pages.scss      |  3 +-
 .../transforms/templates/cli/lib/cli.html     | 33 ++++++++-----------
 2 files changed, 15 insertions(+), 21 deletions(-)

diff --git a/aio/src/styles/2-modules/_cli-pages.scss b/aio/src/styles/2-modules/_cli-pages.scss
index a8462aa91b..7f1c9b7018 100644
--- a/aio/src/styles/2-modules/_cli-pages.scss
+++ b/aio/src/styles/2-modules/_cli-pages.scss
@@ -1,7 +1,8 @@
-.cli-name, .cli-default {
+.cli-name {
   font-weight: bold;
 }
 
 .cli-option-syntax {
   white-space: pre;
 }
+
diff --git a/aio/tools/transforms/templates/cli/lib/cli.html b/aio/tools/transforms/templates/cli/lib/cli.html
index fccf02bcf1..2658791fa3 100644
--- a/aio/tools/transforms/templates/cli/lib/cli.html
+++ b/aio/tools/transforms/templates/cli/lib/cli.html
@@ -53,16 +53,12 @@
   {% for option in options %}
     <tr class="cli-option">
       <td>
-        {% for type in option.types -%}
-        {% for alias in option.names -%}
-        <code class="cli-option-syntax">{$ renderOption(option.name, alias, type, option.default, option.enum) $}</code>
-        {% if not loop.last %}<br>{% endif %}
-        {% endfor -%}
-        {% if not loop.last %}<br>{% endif %}
-        {% endfor -%}
+        <code class="cli-option-syntax">{$ renderOption(option.name, option.type, option.default, option.enum) $}</code>
       </td>
       <td>
         {$ option.description | marked $}
+        {% if option.default !== undefined %}<p><span class="cli-default">Default:</span> <code>{$ option.default $}</code></p>{% endif %}
+        {% if option.aliases.length %}<p><span class="cli-aliases">Aliases:</span> {% for alias in option.aliases %}{$ renderOptionName(alias) $}{% if not loop.last %}, {% endif %}{% endfor %}</p>{% endif %}
       </td>
     </tr>
   {% endfor %}
@@ -71,26 +67,23 @@
 {% endif %}
 {% endmacro %}
 
-{% macro bold(isBold, contents) -%}
-<span {% if isBold %}class="cli-default"{% endif %}>{$ contents $}</span>
-{%- endmacro -%}
+{%- macro renderOptionName(name) -%}
+{% if name.length > 1 %}-{% endif %}-{$ name $}
+{%- endmacro %}
 
 {%- macro renderValues(values, default) -%}
-{% for value in values %}{$ bold(value==default, value) $}{% if not loop.last %}|{% endif %}{% endfor %}
+{$ values.join('|') $}
 {%- endmacro -%}
 
-{%- macro renderOption(name, alias, type, default, values) -%}
-{% set prefix = '--' if (name === alias and name.length > 1) else '-' %}
-{%- if type === 'boolean' -%}
-{%- if not values.length %}{$ prefix $}{$ alias $}={$ renderValues([true, false], default) $}
-{% endif -%}
-{$ prefix $}{$ bold(default, alias) $}|{$ prefix $}{$ bold(not default, alias | cliNegate) $}
+{%- macro renderOption(name, type, default, values) -%}
+{% set prefix = '--' if name.length > 1 else '-' %}
+{%- if type === 'boolean' and not values.length %}{$ prefix $}{$ name $}={$ renderValues([true, false], default) $}
 {%- elif values.length -%}
-{$ prefix $}{$ alias $}={$ renderValues(values, default) $}
+{$ prefix $}{$ name $}={$ renderValues(values, default) $}
 {%- elif type === 'string' -%}
-{$ prefix $}{$ alias $}=<var>{$ name $}</var>
+<var>{$ name $}</var>
 {%- else -%}
-{$ prefix $}{$ alias $}
+{$ prefix $}{$ name $}
 {%- endif -%}
 {%- endmacro -%}