From d3158f5c7e4a4f4916be9887a9e12b294a5bf360 Mon Sep 17 00:00:00 2001
From: Wyatt Johnson <wyattjoh@gmail.com>
Date: Fri, 1 Jun 2018 15:14:49 -0600
Subject: [PATCH] GraphiQL Fixes (#1671)

* bug: fixed graphiql style, upgraded
---
 views/api/graphiql.njk | 103 ++++++++++++++++++++++-------------------
 1 file changed, 56 insertions(+), 47 deletions(-)

diff --git a/views/api/graphiql.njk b/views/api/graphiql.njk
index b726918f3..219046b1f 100644
--- a/views/api/graphiql.njk
+++ b/views/api/graphiql.njk
@@ -5,56 +5,82 @@
 {% block css %}
 {# Include the base development pieces #}
 {{ super() }}
+<link href="//cdn.jsdelivr.net/npm/graphiql@0.11.11/graphiql.css" rel="stylesheet" />
 <style type="text/css">
-  html, body, #graphiql {
+  html, body {
     height: 100%;
     margin: 0;
     overflow: hidden;
     width: 100%;
   }
+  #graphiql {
+    position: absolute;
+    right: 0;
+    left: 0;
+    bottom: 0;
+    top: 56px;
+  }
   *, ::after, ::before {
     box-sizing: inherit;
   }
 </style>
-<link href="//cdn.jsdelivr.net/graphiql/0.9.1/graphiql.css" rel="stylesheet" />
 {% endblock %}
 
 {% block js %}
+<script src="//cdn.jsdelivr.net/es6-promise/4.0.5/es6-promise.auto.min.js"></script>
 <script src="//cdn.jsdelivr.net/fetch/0.9.0/fetch.min.js"></script>
-<script src="//cdn.jsdelivr.net/react/15.0.0/react.min.js"></script>
-<script src="//cdn.jsdelivr.net/react/15.0.0/react-dom.min.js"></script>
-<script src="//cdn.jsdelivr.net/graphiql/0.9.1/graphiql.min.js"></script>
+<script src="//cdn.jsdelivr.net/react/15.4.2/react.min.js"></script>
+<script src="//cdn.jsdelivr.net/react/15.4.2/react-dom.min.js"></script>
+<script src="//cdn.jsdelivr.net/npm/graphiql@0.11.11/graphiql.min.js"></script>
 <script type="text/javascript">
-  // Collect the URL parameters
+  // Parse the search string to get url parameters.
+  var search = window.location.search;
   var parameters = {};
-  window.location.search.substr(1).split('&').forEach(function (entry) {
+  search.substr(1).split('&').forEach(function (entry) {
     var eq = entry.indexOf('=');
     if (eq >= 0) {
       parameters[decodeURIComponent(entry.slice(0, eq))] =
         decodeURIComponent(entry.slice(eq + 1));
     }
   });
-  // Produce a Location query string from a parameter object.
-  function locationQuery(params, location) {
-    return (location ? location: '') + '?' + Object.keys(params).map(function (key) {
-      return encodeURIComponent(key) + '=' +
-        encodeURIComponent(params[key]);
-    }).join('&');
-  }
-  // Derive a fetch URL from the current URL, sans the GraphQL parameters.
-  var graphqlParamNames = {
-    query: true,
-    variables: true,
-    operationName: true
-  };
-  var otherParams = {};
-  for (var k in parameters) {
-    if (parameters.hasOwnProperty(k) && graphqlParamNames[k] !== true) {
-      otherParams[k] = parameters[k];
+
+  // if variables was provided, try to format it.
+  if (parameters.variables) {
+    try {
+      parameters.variables =
+        JSON.stringify(JSON.parse(parameters.variables), null, 2);
+    } catch (e) {
+      // Do nothing, we want to display the invalid JSON as a string, rather
+      // than present an error.
     }
   }
+
+  // When the query and variables string is edited, update the URL bar so
+  // that it can be easily shared
+  function onEditQuery(newQuery) {
+    parameters.query = newQuery;
+    updateURL();
+  }
+  function onEditVariables(newVariables) {
+    parameters.variables = newVariables;
+    updateURL();
+  }
+  function onEditOperationName(newOperationName) {
+    parameters.operationName = newOperationName;
+    updateURL();
+  }
+  function updateURL() {
+    var newSearch = '?' + Object.keys(parameters).filter(function (key) {
+      return Boolean(parameters[key]);
+    }).map(function (key) {
+      return encodeURIComponent(key) + '=' +
+        encodeURIComponent(parameters[key]);
+    }).join('&');
+    history.replaceState(null, null, newSearch);
+  }
+
   // We don't use safe-serialize for location, because it's not client input.
-  var fetchURL = locationQuery(otherParams, '{{ BASE_URL }}{{ endpointURL }}');
+  var fetchURL = '{{ BASE_URL }}{{ endpointURL }}';
 
   // Defines a GraphQL fetcher using the fetch API.
   function graphQLFetcher(graphQLParams) {
@@ -87,34 +113,17 @@
       }
     });
   }
-  // When the query and variables string is edited, update the URL bar so
-  // that it can be easily shared.
-  function onEditQuery(newQuery) {
-    parameters.query = newQuery;
-    updateURL();
-  }
-  function onEditVariables(newVariables) {
-    parameters.variables = newVariables;
-    updateURL();
-  }
-  function onEditOperationName(newOperationName) {
-    parameters.operationName = newOperationName;
-    updateURL();
-  }
-  function updateURL() {
-    history.replaceState(null, null, locationQuery(parameters));
-  }
+
   // Render <GraphiQL /> into the body.
   ReactDOM.render(
     React.createElement(GraphiQL, {
       fetcher: graphQLFetcher,
+      query: parameters.query,
+      variables: parameters.variables,
+      operationName: parameters.operationName,
       onEditQuery: onEditQuery,
       onEditVariables: onEditVariables,
-      onEditOperationName: onEditOperationName,
-      query: null,
-      response: null,
-      variables: null,
-      operationName: null,
+      onEditOperationName: onEditOperationName
     }),
     document.querySelector("#graphiql")
   );