fix some formatting issues with the roadmap

getzaymo · Jul 9, 2018 · 0055759 · 0055759
1 parent 08be92b
commit 0055759
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 6 deletions.
diff --git a/assets/sass/pages/_roadmap.scss b/assets/sass/pages/_roadmap.scss
@@ -30,7 +30,8 @@
     text-align: center;
 
     h4,
-    p {
+    p,
+    ul:not(.tags) {
       color: $color-white;
       margin: 0 0 5px 0;
     }
@@ -83,12 +84,19 @@
           font-weight: 400;
         }
 
-        p {
+        p,
+        ul:not(.tags) {
           margin-top: 0;
           font-size: 10pt;
           line-height: 1.5;
           color: #444;
         }
+
+        ul:not(.tags) {
+          padding-left: 1.1em;
+          list-style-type: square;
+        }
+
       }
 
     }

diff --git a/content/includes/roadmap.json b/content/includes/roadmap.json
@@ -71,7 +71,7 @@
         "issue": {
           "url": "https://github.com/ampproject/amphtml/issues/16555",
           "title": "Instrument & tweak legacy ad performance with no impact to UX",
-          "description": "- [ ] Legacy ads (ads that consist of JS) on AMP pages some times have indeterministic behavior in regards to how quickly they appear on screen. This can happen in two different situations: 1) When the AMP page loads and looks entirely loaded by the ads take close to 5-6 seconds to load 2) Ads further down the viewport (even within 5 viewports) are not rendered even when scrolling down 10 seconds after pausing in the first viewport. ",
+          "description": "-  Legacy ads (ads that consist of JS) on AMP pages some times have indeterministic behavior in regards to how quickly they appear on screen. This can happen in two different situations: 1) When the AMP page loads and looks entirely loaded by the ads take close to 5-6 seconds to load 2) Ads further down the viewport (even within 5 viewports) are not rendered even when scrolling down 10 seconds after pausing in the first viewport. ",
           "labels": [
             {
               "url": "https://api.github.com/repos/ampproject/amphtml/labels/Category:%20Ads",
@@ -151,7 +151,7 @@
         "issue": {
           "url": "https://github.com/ampproject/amphtml/issues/16535",
           "title": "Add new analytics for stories",
-          "description": "- [ ] Add an analytics trigger for when the user education overlay is shown (#15420)\r\n- [ ] Add an analytics trigger for when the video spinner is shown",
+          "description": "-  Add an analytics trigger for when the user education overlay is shown (#15420)\r\n-  Add an analytics trigger for when the video spinner is shown",
           "labels": [
             {
               "url": "https://api.github.com/repos/ampproject/amphtml/labels/Category:%20AMP%20Story",
@@ -209,7 +209,7 @@
         "issue": {
           "url": "https://github.com/ampproject/amphtml/issues/16533",
           "title": "Design/test a new desktop experience for stories",
-          "description": "- [ ] Create a prototype of a new desktop experience\r\n- [ ] Test the prototype\r\n- [ ] Integrate feedback\r\n- [ ] Design a launch/rollout plan\r\n- [ ] Roll out the new desktop experience",
+          "description": "-  Create a prototype of a new desktop experience\r\n-  Test the prototype\r\n-  Integrate feedback\r\n-  Design a launch/rollout plan\r\n-  Roll out the new desktop experience",
           "labels": [
             {
               "url": "https://api.github.com/repos/ampproject/amphtml/labels/Category:%20AMP%20Story",

diff --git a/scripts/import_roadmap.js b/scripts/import_roadmap.js
@@ -78,7 +78,10 @@ async function importRoadmap() {
     card.issue = {
       url: issue.data.html_url,
       title: issue.data.title.replace(/\[Master [fF]eature\] /, ''),
-      description: issue.data.body.replace('Feature description:\r\n\r\n', '').split('\r\n\r\n')[0],
+      description: issue.data.body
+        .replace('Feature description:\r\n\r\n', '')
+        .replace(/\[ \]/g, '')
+        .split('\r\n\r\n')[0],
       labels: (issue.data.labels || []).map(label => {
 
         return label.name.startsWith('Category:') ? { url: label.url, name: label.name.replace('Category:', ''), color: label.color } : false;