add httpSerializer to Discover

elastic · Nov 25, 2019 · 4b2c750 · 4b2c750
1 parent 7abb1b1
commit 4b2c750
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/src/legacy/core_plugins/kibana/public/discover/_discover.scss b/src/legacy/core_plugins/kibana/public/discover/_discover.scss
@@ -96,12 +96,6 @@ discover-app {
 
 .dscTable {
   overflow: auto;
-
-  // SASSTODO: add a monospace modifier to the doc-table component
-  .kbnDocTable__row {
-    font-family: $euiCodeFontFamily;
-    font-size: $euiFontSizeXS;
-  }
 }
 
 /**

diff --git a/src/legacy/core_plugins/kibana/public/discover/angular/discover.js b/src/legacy/core_plugins/kibana/public/discover/angular/discover.js
@@ -184,6 +184,7 @@ function discoverController(
   $scope,
   $timeout,
   $window,
+  $httpParamSerializer,
   AppState,
   Private,
   Promise,
@@ -1104,9 +1105,7 @@ function discoverController(
       indexPattern: $scope.indexPattern.id,
     });
 
-    // TODO should be $httpParaSerializer like in table_row.js
-    // const hash = $httpParamSerializer({
-    const hash = JSON.stringify({
+    const hash = $httpParamSerializer({
       _a: rison.encode({
         columns: $state.columns,
         filters: ($scope.filters || []).map(esFilters.disableFilter),

diff --git a/src/legacy/core_plugins/kibana/public/discover/angular/doc.ts b/src/legacy/core_plugins/kibana/public/discover/angular/doc.ts
@@ -17,7 +17,6 @@
  * under the License.
  */
 import { getServices, IndexPatterns } from '../kibana_services';
-// @ts-ignore
 import { getRootBreadcrumbs } from '../breadcrumbs';
 import html from './doc.html';
 import { Doc } from '../doc/doc';