[7.x] Upgrade EUI to v18.2.0 (#54786) (#54869)

* Upgrade EUI to v18.2.0 (#54786) * 18.2.0 * ts update * Updated `euiColorVis0` * Update `euiColorVis1` * Updating `euiColorVis2` * Updated `euiColorVis3` * Updated the rest * Updated hard-coded viz palette hexes to latest * src snapshot updates * x-pack test updates * mock jest fn * Updated two vegalite visualization screenshots Co-authored-by: Caroline Horn <[email protected]> Co-authored-by: Chandler Prall <[email protected]> * snapshot Co-authored-by: Caroline Horn <[email protected]> Co-authored-by: Chandler Prall <[email protected]>
elastic · Jan 15, 2020 · e0d6407 · e0d6407
1 parent 7556436
commit e0d6407
Show file tree

Hide file tree

Showing 69 changed files with 712 additions and 591 deletions.
diff --git a/package.json b/package.json
@@ -115,7 +115,7 @@
     "@elastic/charts": "^16.1.0",
     "@elastic/datemath": "5.0.2",
     "@elastic/ems-client": "1.0.5",
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "@elastic/filesaver": "1.1.2",
     "@elastic/good": "8.1.1-kibana2",
     "@elastic/numeral": "2.3.3",

diff --git a/packages/kbn-ui-shared-deps/package.json b/packages/kbn-ui-shared-deps/package.json
@@ -9,7 +9,7 @@
     "kbn:watch": "node scripts/build --watch"
   },
   "devDependencies": {
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "@elastic/charts": "^16.1.0",
     "@kbn/dev-utils": "1.0.0",
     "@yarnpkg/lockfile": "^1.1.0",

diff --git a/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_graph.hjson b/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_graph.hjson
@@ -39,7 +39,7 @@
     range: {
       category: {scheme: "elastic"}
     }
-    mark: {color: "#00B3A4"}
+    mark: {color: "#54B399"}
   }
   autosize: {type: "fit", contains: "padding"}
 }
diff --git a/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_image_256.png b/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_image_256.png
diff --git a/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_image_512.png b/src/legacy/core_plugins/vis_type_vega/public/__tests__/vegalite_image_512.png
diff --git a/src/legacy/core_plugins/vis_type_vega/public/data_model/__tests__/vega_parser.js b/src/legacy/core_plugins/vis_type_vega/public/data_model/__tests__/vega_parser.js
@@ -53,7 +53,7 @@ describe(`VegaParser._setDefaultColors`, () => {
     test({}, true, {
       config: {
         range: { category: { scheme: 'elastic' } },
-        mark: { color: '#5BBAA0' },
+        mark: { color: '#54B399' },
       },
     })
   );
@@ -63,15 +63,15 @@ describe(`VegaParser._setDefaultColors`, () => {
     test({}, false, {
       config: {
         range: { category: { scheme: 'elastic' } },
-        arc: { fill: '#5BBAA0' },
-        area: { fill: '#5BBAA0' },
-        line: { stroke: '#5BBAA0' },
-        path: { stroke: '#5BBAA0' },
-        rect: { fill: '#5BBAA0' },
-        rule: { stroke: '#5BBAA0' },
-        shape: { stroke: '#5BBAA0' },
-        symbol: { fill: '#5BBAA0' },
-        trail: { fill: '#5BBAA0' },
+        arc: { fill: '#54B399' },
+        area: { fill: '#54B399' },
+        line: { stroke: '#54B399' },
+        path: { stroke: '#54B399' },
+        rect: { fill: '#54B399' },
+        rule: { stroke: '#54B399' },
+        shape: { stroke: '#54B399' },
+        symbol: { fill: '#54B399' },
+        trail: { fill: '#54B399' },
       },
     })
   );

diff --git a/src/legacy/core_plugins/vis_type_vega/public/data_model/vega_parser.js b/src/legacy/core_plugins/vis_type_vega/public/data_model/vega_parser.js
@@ -577,7 +577,7 @@ export class VegaParser {
     this._setDefaultValue({ scheme: 'elastic' }, 'config', 'range', 'category');
 
     if (this.isVegaLite) {
-      // Vega-Lite: set default color, works for fill and strike --  config: { mark:  { color: '#00B3A4' }}
+      // Vega-Lite: set default color, works for fill and strike --  config: { mark:  { color: '#54B399' }}
       this._setDefaultValue(defaultColor, 'config', 'mark', 'color');
     } else {
       // Vega - global mark has very strange behavior, must customize each mark type individually

diff --git a/src/legacy/ui/public/styles/bootstrap/_custom_variables.less b/src/legacy/ui/public/styles/bootstrap/_custom_variables.less
@@ -345,7 +345,7 @@
 //** Background color of the whole progress component
 @progress-bg:                           shade(@gray-lighter, 13%);
 //** Default progress bar color
-@progress-bar-bg:                       #00B3A4;
+@progress-bar-bg:                       #54B399;
 
 //== List group
 //

diff --git a/src/legacy/ui/public/vis/editors/default/controls/precision.tsx b/src/legacy/ui/public/vis/editors/default/controls/precision.tsx
@@ -40,7 +40,7 @@ function PrecisionParamEditor({ agg, value, setValue }: AggParamEditorProps<numb
       <EuiRange
         min={1}
         max={config.get('visualization:tileMap:maxPrecision')}
-        value={value}
+        value={value || ''}
         onChange={(ev: React.ChangeEvent<HTMLInputElement> | React.MouseEvent<HTMLButtonElement>) =>
           setValue(Number(ev.currentTarget.value))
         }

diff --git a/src/plugins/kibana_react/public/field_icon/__snapshots__/field_icon.test.tsx.snap b/src/plugins/kibana_react/public/field_icon/__snapshots__/field_icon.test.tsx.snap
diff --git a/test/interpreter_functional/plugins/kbn_tp_run_pipeline/package.json b/test/interpreter_functional/plugins/kbn_tp_run_pipeline/package.json
@@ -7,7 +7,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "react": "^16.12.0",
     "react-dom": "^16.12.0"
   }

diff --git a/test/plugin_functional/plugins/kbn_tp_custom_visualizations/package.json b/test/plugin_functional/plugins/kbn_tp_custom_visualizations/package.json
@@ -7,7 +7,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "react": "^16.12.0"
   }
 }
diff --git a/test/plugin_functional/plugins/kbn_tp_embeddable_explorer/package.json b/test/plugin_functional/plugins/kbn_tp_embeddable_explorer/package.json
@@ -8,7 +8,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "react": "^16.12.0"
   },
   "scripts": {

diff --git a/test/plugin_functional/plugins/kbn_tp_sample_panel_action/package.json b/test/plugin_functional/plugins/kbn_tp_sample_panel_action/package.json
@@ -8,7 +8,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@elastic/eui": "18.0.0",
+    "@elastic/eui": "18.2.0",
     "react": "^16.12.0"
   },
   "scripts": {

diff --git a/...ublic/components/app/ServiceOverview/__test__/__snapshots__/ServiceOverview.test.tsx.snap b/...ublic/components/app/ServiceOverview/__test__/__snapshots__/ServiceOverview.test.tsx.snap
diff --git a/...terfallContainer/Waterfall/waterfall_helpers/__snapshots__/waterfall_helpers.test.ts.snap b/...terfallContainer/Waterfall/waterfall_helpers/__snapshots__/waterfall_helpers.test.ts.snap