From 405d97431fa374afdd5cddc6780f8949ee76a2c6 Mon Sep 17 00:00:00 2001
From: Pete Bacon Darwin <pete@bacondarwin.com>
Date: Sun, 4 Mar 2018 20:30:23 +0000
Subject: [PATCH] fixup! feat(aio): allow template to position embedded ToC
 (#22565)

PR Close #22565
---
 aio/src/styles/1-layouts/_content-layout.scss | 11 --------
 aio/src/styles/2-modules/_api-pages.scss      | 17 +++++++++++
 .../templates/api/base.template.html          | 28 ++++++-------------
 .../templates/api/lib/githubLinks.html        |  5 +++-
 4 files changed, 30 insertions(+), 31 deletions(-)

diff --git a/aio/src/styles/1-layouts/_content-layout.scss b/aio/src/styles/1-layouts/_content-layout.scss
index 7870fa3fec..c6034396da 100644
--- a/aio/src/styles/1-layouts/_content-layout.scss
+++ b/aio/src/styles/1-layouts/_content-layout.scss
@@ -37,14 +37,3 @@ aio-shell.page-docs {
   margin: 24px 0px;
   background: $lightgray;
 }
-
-.page-actions {
-  float: right;
-  .material-icons {
-    border-radius: 4px;
-    padding: 4px;
-    font-size: 20px;
-    background-color: $mist;
-    color: $mediumgray;
-  }
-}
\ No newline at end of file
diff --git a/aio/src/styles/2-modules/_api-pages.scss b/aio/src/styles/2-modules/_api-pages.scss
index b6a976dcf7..8656691a04 100644
--- a/aio/src/styles/2-modules/_api-pages.scss
+++ b/aio/src/styles/2-modules/_api-pages.scss
@@ -1,3 +1,15 @@
+.page-actions {
+  float: right;
+  .material-icons {
+    border-radius: 4px;
+    padding: 4px;
+    font-size: 20px;
+    &:hover {
+      background-color: $mist;
+    }
+  }
+}
+
 .api-body {
 
   .class-overview {
@@ -24,6 +36,11 @@
   .api-heading {
     padding: 5px 0;
     font-size: 16px;
+    font-weight: bold;
+  }
+
+  .short-description {
+    margin: 6px 0 0 10px;
   }
 
   .properties-table {
diff --git a/aio/tools/transforms/templates/api/base.template.html b/aio/tools/transforms/templates/api/base.template.html
index a5cdb7bb75..63f6490cab 100644
--- a/aio/tools/transforms/templates/api/base.template.html
+++ b/aio/tools/transforms/templates/api/base.template.html
@@ -2,32 +2,22 @@
 {% set comma = joiner(',') %}
 {% set slash = joiner('/') %}
 <article>
-    <div class="page-actions">
-      <a href="{$ github.githubEditHref(doc, versionInfo) $}" aria-label="Suggest Edits" title="Suggest Edits"><i class="material-icons" aria-hidden="true" role="img">mode_edit</i></a>
-      <a href="{$ github.githubViewHref(doc, versionInfo) $}" aria-label="View Source" title="View Source"><i class="material-icons" aria-hidden="true" role="img">code</i></a>
-    </div>
-    <div class="breadcrumb">
+  <div class="page-actions">
+    <a href="{$ github.githubEditHref(doc, versionInfo) $}" aria-label="Suggest Edits" title="Suggest Edits"><i class="material-icons" aria-hidden="true" role="img">mode_edit</i></a>
+    <a href="{$ github.githubViewHref(doc, versionInfo) $}" aria-label="View Source" title="View Source"><i class="material-icons" aria-hidden="true" role="img">code</i></a>
+  </div>
+  <div class="breadcrumb">
     <script type="application/ld+json">
       {
         "@context": "http://schema.org",
         "@type": "BreadcrumbList",
         "itemListElement": [
-          {% for crumb in doc.breadCrumbs %}
-            {$ comma() $}{
-            "@type": "ListItem",
-            "position": {$ loop.index $},
-            "item": {
-              "@id": "https://angular.io/{$ crumb.path $}",
-              "name": "{$ crumb.text $}"
-            }
-          }{% endfor %}
+          {%- for crumb in doc.breadCrumbs %}{$ comma() $}
+          { "@type": "ListItem", "position": {$ loop.index $}, "item": { "@id": "https://angular.io/{$ crumb.path $}", "name": "{$ crumb.text $}" } }{% endfor %}
         ]
       }
-      </script>
-    {% for crumb in doc.breadCrumbs %}{% if not loop.last %}
-      {$ slash() $}
-      {% if crumb.path %}<a href="{$ crumb.path $}">{$ crumb.text $}</a>{% else %}{$ crumb.text $}{% endif %}
-    {% endif %}{% endfor %}
+    </script>
+    {% for crumb in doc.breadCrumbs %}{% if not loop.last %} {$ slash() $} {% if crumb.path %}<a href="{$ crumb.path $}">{$ crumb.text $}</a>{% else %}{$ crumb.text $}{% endif %}{% endif %}{% endfor %}
   </div>
   <header class="api-header">
     {% if doc.deprecated !== undefined %}<label class="api-status-label deprecated">deprecated</label>{% endif %}
diff --git a/aio/tools/transforms/templates/api/lib/githubLinks.html b/aio/tools/transforms/templates/api/lib/githubLinks.html
index f35324b132..efba738914 100644
--- a/aio/tools/transforms/templates/api/lib/githubLinks.html
+++ b/aio/tools/transforms/templates/api/lib/githubLinks.html
@@ -4,5 +4,8 @@ https://github.com/{$ versionInfo.gitRepoInfo.owner $}/{$ versionInfo.gitRepoInf
 
 
 {% macro githubEditHref(doc, versionInfo) -%}
-https://github.com/{$ versionInfo.gitRepoInfo.owner $}/{$ versionInfo.gitRepoInfo.repo $}/edit/master/packages/{$ doc.fileInfo.realProjectRelativePath $}?message=docs({$ doc.moduleDoc.id | escape $})%3A%20describe%20your%20change...#L{$ doc.startingLine + 1 $}-L{$ doc.endingLine + 1 $}
+https://github.com/{$ versionInfo.gitRepoInfo.owner $}/{$ versionInfo.gitRepoInfo.repo $}/edit/master/packages/{$ doc.fileInfo.realProjectRelativePath $}?message=docs(
+  {%- if doc.moduleDoc %}{$ doc.moduleDoc.id.split('/')[0] $}
+  {%- if doc.docType === 'module' %}{$ doc.id.split('/')[0] $}
+  {%- else %}...{%- endif%})%3A%20describe%20your%20change...#L{$ doc.startingLine + 1 $}-L{$ doc.endingLine + 1 $}
 {%- endmacro %}
\ No newline at end of file