From 9c166753d7f4df9987962497f2978928a341fb96 Mon Sep 17 00:00:00 2001
From: Salem Ghoweri <salem.ghoweri@pega.com>
Date: Sun, 22 Jul 2018 09:02:12 -0400
Subject: [PATCH 1/2] fix: check to make sure the code panel-related <script>
 tag contains data before attempting to parse expected JSON. Partial fix to
 #761 as this should at least help prevent the current batch of JS errors from
 getting thrown

---
 .../uikit-workshop/src/js/modal-styleguide.js | 33 +++++++++++++------
 1 file changed, 23 insertions(+), 10 deletions(-)

diff --git a/packages/uikit-workshop/src/js/modal-styleguide.js b/packages/uikit-workshop/src/js/modal-styleguide.js
index 5ec1e3450..86026e8a3 100644
--- a/packages/uikit-workshop/src/js/modal-styleguide.js
+++ b/packages/uikit-workshop/src/js/modal-styleguide.js
@@ -116,16 +116,29 @@ var modalStyleguide = {
    * @param  {Boolean}      if the text in the dropdown should be switched
    */
   collectAndSend: function(el, iframePassback, switchText) {
-    var patternData = JSON.parse(el.innerHTML);
-    if (patternData.patternName !== undefined) {
-      patternMarkupEl = document.querySelector(
-        '#' + patternData.patternPartial + ' > .pl-js-pattern-example'
-      );
-      patternData.patternMarkup =
-        patternMarkupEl !== null
-          ? patternMarkupEl.innerHTML
-          : document.querySelector('body').innerHTML;
-      modalStyleguide.patternQueryInfo(patternData, iframePassback, switchText);
+    /**
+     * Verify <script> tag has JSON data available (not just whitespace) - helps prevents JS errors from
+     * getting thrown when certain script tags aren't rendered with partial.patternData content.
+     */
+    if (/\S/.test(el.innerHTML)) {
+      const patternData = JSON.parse(el.innerHTML);
+      if (patternData.patternName !== undefined) {
+        patternMarkupEl = document.querySelector(
+          '#' + patternData.patternPartial + ' > .pl-js-pattern-example'
+        );
+        patternData.patternMarkup =
+          patternMarkupEl !== null
+            ? patternMarkupEl.innerHTML
+            : document.querySelector('body').innerHTML;
+        modalStyleguide.patternQueryInfo(
+          patternData,
+          iframePassback,
+          switchText
+        );
+      }
+    } else {
+      // @todo: how are we handling conditional logging for debugging based on the dev environment?
+      // console.log('This <script> tag\'s JSON is empty for some reason...');
     }
   },
 

From 35c5726af130b7165b08745dd45bd10a3ac3e0a4 Mon Sep 17 00:00:00 2001
From: Salem Ghoweri <salem.ghoweri@pega.com>
Date: Sat, 28 Jul 2018 09:27:04 -0400
Subject: [PATCH 2/2] fix: change const back to var since PL's Uikit JS isn't
 run through Babel just yet.. update Prettier config to ignore Uikit's
 JavaScript for the time being

---
 .prettierignore                                    | 1 +
 packages/uikit-workshop/src/js/modal-styleguide.js | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.prettierignore b/.prettierignore
index 3a4d3d062..627831b2a 100644
--- a/.prettierignore
+++ b/.prettierignore
@@ -8,3 +8,4 @@
 packages/core/scripts/api.handlebars
 packages/core/scripts/events.handlebars
 packages/core/test/files/annotations.js
+**/uikit-workshop/src/js/**/*
diff --git a/packages/uikit-workshop/src/js/modal-styleguide.js b/packages/uikit-workshop/src/js/modal-styleguide.js
index 86026e8a3..9ed29eaa3 100644
--- a/packages/uikit-workshop/src/js/modal-styleguide.js
+++ b/packages/uikit-workshop/src/js/modal-styleguide.js
@@ -121,7 +121,7 @@ var modalStyleguide = {
      * getting thrown when certain script tags aren't rendered with partial.patternData content.
      */
     if (/\S/.test(el.innerHTML)) {
-      const patternData = JSON.parse(el.innerHTML);
+      var patternData = JSON.parse(el.innerHTML);
       if (patternData.patternName !== undefined) {
         patternMarkupEl = document.querySelector(
           '#' + patternData.patternPartial + ' > .pl-js-pattern-example'