Upgrade to EUI 3.4.0 (#21870) (#21964)

* Upgrade to EUI 3.4.0 * Update snapshots
elastic · Aug 15, 2018 · f48d6c8 · f48d6c8
1 parent 5098b5f
commit f48d6c8
Show file tree

Hide file tree

Showing 18 changed files with 131 additions and 49 deletions.
diff --git a/package.json b/package.json
@@ -75,7 +75,7 @@
     "url": "https://github.com/elastic/kibana.git"
   },
   "dependencies": {
-    "@elastic/eui": "3.2.1",
+    "@elastic/eui": "3.4.0",
     "@elastic/filesaver": "1.1.2",
     "@elastic/numeral": "2.3.2",
     "@elastic/ui-ace": "0.2.3",

diff --git a/...e_plugins/input_control_vis/public/components/vis/__snapshots__/list_control.test.js.snap b/...e_plugins/input_control_vis/public/components/vis/__snapshots__/list_control.test.js.snap
@@ -25,6 +25,7 @@ exports[`renders ListControl 1`] = `
 >
   <EuiComboBox
     data-test-subj="listControlSelect0"
+    fullWidth={false}
     isClearable={true}
     isLoading={false}
     onChange={[Function]}

diff --git a/src/core_plugins/kibana/public/dashboard/top_nav/__snapshots__/add_panel.test.js.snap b/src/core_plugins/kibana/public/dashboard/top_nav/__snapshots__/add_panel.test.js.snap
@@ -6,6 +6,7 @@ exports[`render 1`] = `
   closeButtonAriaLabel="Closes this dialog"
   data-test-subj="dashboardAddPanel"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[Function]}
   ownFocus={true}
   size="s"

diff --git a/...kibana/public/home/components/tutorial/__snapshots__/saved_objects_installer.test.js.snap b/...kibana/public/home/components/tutorial/__snapshots__/saved_objects_installer.test.js.snap
@@ -92,12 +92,20 @@ exports[`bulkCreate should display error message when bulkCreate request fails 1
             <span
               className="euiScreenReaderOnly"
             >
-              Step
+              Incomplete
+               Step 
             </span>
           </EuiScreenReaderOnly>
-          <div
-            className="euiStep__circle euiStep__circle--incomplete"
-          />
+          <EuiStepNumber
+            className="euiStep__circle"
+            isHollow={true}
+            number={1}
+            status="incomplete"
+          >
+            <div
+              className="euiStepNumber euiStepNumber--incomplete euiStepNumber-isHollow euiStep__circle"
+            />
+          </EuiStepNumber>
           <EuiTitle
             className="euiStep__title"
             size="s"
@@ -308,51 +316,60 @@ exports[`bulkCreate should display success message when bulkCreate is successful
             <span
               className="euiScreenReaderOnly"
             >
-              Step
+               Step 
             </span>
           </EuiScreenReaderOnly>
-          <div
-            className="euiStep__circle euiStep__circle--complete"
+          <EuiStepNumber
+            className="euiStep__circle"
+            isHollow={false}
+            number={1}
+            status="complete"
           >
-            <EuiIcon
-              className="euiStep__circleIcon"
-              color="ghost"
-              size="m"
-              type="check"
+            <div
+              className="euiStepNumber euiStepNumber--complete euiStep__circle"
             >
-              <check
-                className="euiIcon euiIcon--medium euiIcon--ghost euiStep__circleIcon"
-                focusable="false"
-                height="16"
-                style={null}
-                viewBox="0 0 16 16"
-                width="16"
-                xmlns="http://www.w3.org/2000/svg"
-                xmlnsXlink="http://www.w3.org/1999/xlink"
+              <EuiIcon
+                className="euiStepNumber__icon"
+                size="m"
+                title="complete"
+                type="check"
               >
-                <svg
-                  className="euiIcon euiIcon--medium euiIcon--ghost euiStep__circleIcon"
+                <check
+                  className="euiIcon euiIcon--medium euiStepNumber__icon"
                   focusable="false"
                   height="16"
-                  style={null}
+                  style={
+                    Object {
+                      "fill": undefined,
+                    }
+                  }
+                  title="complete"
                   viewBox="0 0 16 16"
                   width="16"
                   xmlns="http://www.w3.org/2000/svg"
-                  xmlnsXlink="http://www.w3.org/1999/xlink"
                 >
-                  <defs>
+                  <svg
+                    className="euiIcon euiIcon--medium euiStepNumber__icon"
+                    focusable="false"
+                    height="16"
+                    style={
+                      Object {
+                        "fill": undefined,
+                      }
+                    }
+                    title="complete"
+                    viewBox="0 0 16 16"
+                    width="16"
+                    xmlns="http://www.w3.org/2000/svg"
+                  >
                     <path
                       d="M6.5 12a.502.502 0 0 1-.354-.146l-4-4a.502.502 0 0 1 .708-.708L6.5 10.793l6.646-6.647a.502.502 0 0 1 .708.708l-7 7A.502.502 0 0 1 6.5 12"
-                      id="check-a"
                     />
-                  </defs>
-                  <use
-                    xlinkHref="#check-a"
-                  />
-                </svg>
-              </check>
-            </EuiIcon>
-          </div>
+                  </svg>
+                </check>
+              </EuiIcon>
+            </div>
+          </EuiStepNumber>
           <EuiTitle
             className="euiStep__title"
             size="s"

diff --git a/...ts/components/objects_table/components/flyout/__tests__/__snapshots__/flyout.test.js.snap b/...ts/components/objects_table/components/flyout/__tests__/__snapshots__/flyout.test.js.snap
@@ -4,6 +4,7 @@ exports[`Flyout conflicts should allow conflict resolution 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"
@@ -164,6 +165,7 @@ exports[`Flyout should render import step 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"

diff --git a/...objects_table/components/relationships/__tests__/__snapshots__/relationships.test.js.snap b/...objects_table/components/relationships/__tests__/__snapshots__/relationships.test.js.snap
@@ -4,6 +4,7 @@ exports[`Relationships should render dashboards normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"
@@ -108,6 +109,7 @@ exports[`Relationships should render errors 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"
@@ -150,6 +152,7 @@ exports[`Relationships should render index patterns normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"
@@ -310,6 +313,7 @@ exports[`Relationships should render searches normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"
@@ -468,6 +472,7 @@ exports[`Relationships should render visualizations normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[MockFunction]}
   ownFocus={false}
   size="m"

diff --git a/...mponents/field_format_editor/editors/url/__snapshots__/label_template_flyout.test.js.snap b/...mponents/field_format_editor/editors/url/__snapshots__/label_template_flyout.test.js.snap
@@ -6,6 +6,7 @@ exports[`LabelTemplateFlyout should render normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[Function]}
   ownFocus={false}
   size="m"

diff --git a/...components/field_format_editor/editors/url/__snapshots__/url_template_flyout.test.js.snap b/...components/field_format_editor/editors/url/__snapshots__/url_template_flyout.test.js.snap
@@ -6,6 +6,7 @@ exports[`UrlTemplateFlyout should render normally 1`] = `
 <EuiFlyout
   closeButtonAriaLabel="Closes this dialog"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[Function]}
   ownFocus={false}
   size="m"

diff --git a/src/ui/public/field_editor/components/scripting_help/__snapshots__/help_flyout.test.js.snap b/src/ui/public/field_editor/components/scripting_help/__snapshots__/help_flyout.test.js.snap
@@ -5,6 +5,7 @@ exports[`ScriptingHelpFlyout should render normally 1`] = `
   closeButtonAriaLabel="Closes this dialog"
   data-test-subj="scriptedFieldsHelpFlyout"
   hideCloseButton={false}
+  maxWidth={false}
   onClose={[Function]}
   ownFocus={false}
   size="m"

diff --git a/x-pack/package.json b/x-pack/package.json
@@ -79,7 +79,7 @@
     "yargs": "4.7.1"
   },
   "dependencies": {
-    "@elastic/eui": "3.2.1",
+    "@elastic/eui": "3.4.0",
     "@elastic/node-crypto": "0.1.2",
     "@elastic/node-phantom-simple": "2.2.4",
     "@elastic/numeral": "2.3.2",

diff --git a/...pm/public/components/app/ErrorGroupOverview/List/__test__/__snapshots__/List.test.js.snap b/...pm/public/components/app/ErrorGroupOverview/List/__test__/__snapshots__/List.test.js.snap
@@ -70,6 +70,7 @@ exports[`ErrorGroupOverview -> List should render empty state 1`] = `
         <tr>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_groupId_0"
             scope="col"
             width="96px"
           >
@@ -85,6 +86,7 @@ exports[`ErrorGroupOverview -> List should render empty state 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_message_1"
             scope="col"
             width="50%"
           >
@@ -100,6 +102,7 @@ exports[`ErrorGroupOverview -> List should render empty state 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_handled_2"
             scope="col"
           >
             <div
@@ -112,11 +115,13 @@ exports[`ErrorGroupOverview -> List should render empty state 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_occurrenceCount_3"
             scope="col"
           >
             <button
               aria-label="Sort Occurrences ascending"
               className="euiTableHeaderButton"
+              data-test-subj="tableHeaderSortButton"
               onClick={[Function]}
               type="button"
             >
@@ -133,11 +138,13 @@ exports[`ErrorGroupOverview -> List should render empty state 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_latestOccurrenceAt_4"
             scope="col"
           >
             <button
               aria-label="Sort Latest occurrence ascending"
               className="euiTableHeaderButton euiTableHeaderButton-isSorted"
+              data-test-subj="tableHeaderSortButton"
               onClick={[Function]}
               type="button"
             >
@@ -367,6 +374,7 @@ exports[`ErrorGroupOverview -> List should render with data 1`] = `
         <tr>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_groupId_0"
             scope="col"
             width="96px"
           >
@@ -382,6 +390,7 @@ exports[`ErrorGroupOverview -> List should render with data 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_message_1"
             scope="col"
             width="50%"
           >
@@ -397,6 +406,7 @@ exports[`ErrorGroupOverview -> List should render with data 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_handled_2"
             scope="col"
           >
             <div
@@ -409,11 +419,13 @@ exports[`ErrorGroupOverview -> List should render with data 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_occurrenceCount_3"
             scope="col"
           >
             <button
               aria-label="Sort Occurrences ascending"
               className="euiTableHeaderButton"
+              data-test-subj="tableHeaderSortButton"
               onClick={[Function]}
               type="button"
             >
@@ -430,11 +442,13 @@ exports[`ErrorGroupOverview -> List should render with data 1`] = `
           </th>
           <th
             className="euiTableHeaderCell"
+            data-test-subj="tableHeaderCell_latestOccurrenceAt_4"
             scope="col"
           >
             <button
               aria-label="Sort Latest occurrence ascending"
               className="euiTableHeaderButton euiTableHeaderButton-isSorted"
+              data-test-subj="tableHeaderSortButton"
               onClick={[Function]}
               type="button"
             >