Remove close buttons from Saved Object relationship and Index Managem…

…ent flyouts. (elastic#20043)
maryia-lapata · Jun 25, 2018 · 27d3e0e · 27d3e0e
1 parent bb11327
commit 27d3e0e
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 152 deletions.
diff --git a/...objects_table/components/relationships/__tests__/__snapshots__/relationships.test.js.snap b/...objects_table/components/relationships/__tests__/__snapshots__/relationships.test.js.snap
@@ -100,32 +100,6 @@ exports[`Relationships should render dashboards normally 1`] = `
       </React.Fragment>
     </EuiDescriptionList>
   </EuiFlyoutBody>
-  <EuiFlyoutFooter>
-    <EuiFlexGroup
-      alignItems="stretch"
-      component="div"
-      direction="row"
-      gutterSize="l"
-      justifyContent="spaceBetween"
-      responsive={true}
-      wrap={false}
-    >
-      <EuiFlexItem
-        component="div"
-        grow={false}
-      >
-        <EuiButtonEmpty
-          color="primary"
-          iconSide="left"
-          onClick={[MockFunction]}
-          size="s"
-          type="button"
-        >
-          Close
-        </EuiButtonEmpty>
-      </EuiFlexItem>
-    </EuiFlexGroup>
-  </EuiFlyoutFooter>
 </EuiFlyout>
 `;
 
@@ -167,32 +141,6 @@ exports[`Relationships should render errors 1`] = `
       foo
     </EuiCallOut>
   </EuiFlyoutBody>
-  <EuiFlyoutFooter>
-    <EuiFlexGroup
-      alignItems="stretch"
-      component="div"
-      direction="row"
-      gutterSize="l"
-      justifyContent="spaceBetween"
-      responsive={true}
-      wrap={false}
-    >
-      <EuiFlexItem
-        component="div"
-        grow={false}
-      >
-        <EuiButtonEmpty
-          color="primary"
-          iconSide="left"
-          onClick={[MockFunction]}
-          size="s"
-          type="button"
-        >
-          Close
-        </EuiButtonEmpty>
-      </EuiFlexItem>
-    </EuiFlexGroup>
-  </EuiFlyoutFooter>
 </EuiFlyout>
 `;
 
@@ -352,32 +300,6 @@ exports[`Relationships should render index patterns normally 1`] = `
       </React.Fragment>
     </EuiDescriptionList>
   </EuiFlyoutBody>
-  <EuiFlyoutFooter>
-    <EuiFlexGroup
-      alignItems="stretch"
-      component="div"
-      direction="row"
-      gutterSize="l"
-      justifyContent="spaceBetween"
-      responsive={true}
-      wrap={false}
-    >
-      <EuiFlexItem
-        component="div"
-        grow={false}
-      >
-        <EuiButtonEmpty
-          color="primary"
-          iconSide="left"
-          onClick={[MockFunction]}
-          size="s"
-          type="button"
-        >
-          Close
-        </EuiButtonEmpty>
-      </EuiFlexItem>
-    </EuiFlexGroup>
-  </EuiFlyoutFooter>
 </EuiFlyout>
 `;
 
@@ -535,32 +457,6 @@ exports[`Relationships should render searches normally 1`] = `
       </React.Fragment>
     </EuiDescriptionList>
   </EuiFlyoutBody>
-  <EuiFlyoutFooter>
-    <EuiFlexGroup
-      alignItems="stretch"
-      component="div"
-      direction="row"
-      gutterSize="l"
-      justifyContent="spaceBetween"
-      responsive={true}
-      wrap={false}
-    >
-      <EuiFlexItem
-        component="div"
-        grow={false}
-      >
-        <EuiButtonEmpty
-          color="primary"
-          iconSide="left"
-          onClick={[MockFunction]}
-          size="s"
-          type="button"
-        >
-          Close
-        </EuiButtonEmpty>
-      </EuiFlexItem>
-    </EuiFlexGroup>
-  </EuiFlyoutFooter>
 </EuiFlyout>
 `;
 
@@ -664,31 +560,5 @@ exports[`Relationships should render visualizations normally 1`] = `
       </React.Fragment>
     </EuiDescriptionList>
   </EuiFlyoutBody>
-  <EuiFlyoutFooter>
-    <EuiFlexGroup
-      alignItems="stretch"
-      component="div"
-      direction="row"
-      gutterSize="l"
-      justifyContent="spaceBetween"
-      responsive={true}
-      wrap={false}
-    >
-      <EuiFlexItem
-        component="div"
-        grow={false}
-      >
-        <EuiButtonEmpty
-          color="primary"
-          iconSide="left"
-          onClick={[MockFunction]}
-          size="s"
-          type="button"
-        >
-          Close
-        </EuiButtonEmpty>
-      </EuiFlexItem>
-    </EuiFlexGroup>
-  </EuiFlyoutFooter>
 </EuiFlyout>
 `;
diff --git a/...ement/sections/objects/components/objects_table/components/relationships/relationships.js b/...ement/sections/objects/components/objects_table/components/relationships/relationships.js
@@ -24,11 +24,7 @@ import {
   EuiTitle,
   EuiFlyout,
   EuiFlyoutBody,
-  EuiFlyoutFooter,
   EuiFlyoutHeader,
-  EuiFlexGroup,
-  EuiFlexItem,
-  EuiButtonEmpty,
   EuiDescriptionList,
   EuiDescriptionListTitle,
   EuiLink,
@@ -240,16 +236,6 @@ export class Relationships extends Component {
         </EuiFlyoutHeader>
 
         <EuiFlyoutBody>{this.renderRelationships()}</EuiFlyoutBody>
-
-        <EuiFlyoutFooter>
-          <EuiFlexGroup justifyContent="spaceBetween">
-            <EuiFlexItem grow={false}>
-              <EuiButtonEmpty onClick={close} size="s">
-                Close
-              </EuiButtonEmpty>
-            </EuiFlexItem>
-          </EuiFlexGroup>
-        </EuiFlyoutFooter>
       </EuiFlyout>
     );
   }

diff --git a/...ugins/index_management/public/sections/index_list/components/detail_panel/detail_panel.js b/...ugins/index_management/public/sections/index_list/components/detail_panel/detail_panel.js
@@ -11,7 +11,6 @@ import { Summary } from './summary';
 import { EditSettingsJson } from './edit_settings_json';
 
 import {
-  EuiButtonEmpty,
   EuiFlexGroup,
   EuiFlexItem,
   EuiFlyout,
@@ -84,13 +83,7 @@ export class DetailPanel extends Component {
         <EuiFlyoutBody>{component}</EuiFlyoutBody>
 
         <EuiFlyoutFooter>
-          <EuiFlexGroup justifyContent="spaceBetween">
-            <EuiFlexItem grow={false}>
-              <EuiButtonEmpty iconType="cross" onClick={closeDetailPanel}>
-                Close
-              </EuiButtonEmpty>
-            </EuiFlexItem>
-
+          <EuiFlexGroup justifyContent="flexEnd">
             <EuiFlexItem grow={false}>
               <Route
                 key="menu"