From 3d2bd78889dea78781b9e92d5525ccf2bd2c4724 Mon Sep 17 00:00:00 2001
From: miukimiu <elizabet.oliveira@elastic.co>
Date: Wed, 9 Oct 2019 18:19:59 +0100
Subject: [PATCH] Tooltip styles

---
 .../public/components/_tooltip_selector.scss   |  5 ++++-
 .../components/add_tooltip_field_popover.js    | 18 +++++++++++-------
 .../maps/public/components/tooltip_selector.js |  4 +++-
 .../es_search_source/update_source_editor.js   |  6 ++++--
 4 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/x-pack/legacy/plugins/maps/public/components/_tooltip_selector.scss b/x-pack/legacy/plugins/maps/public/components/_tooltip_selector.scss
index 9abf7d619df04..74c7860f3fa99 100644
--- a/x-pack/legacy/plugins/maps/public/components/_tooltip_selector.scss
+++ b/x-pack/legacy/plugins/maps/public/components/_tooltip_selector.scss
@@ -1,7 +1,7 @@
 .mapTooltipSelector__propertyRow {
   display: flex;
 
-  padding-bottom: $euiSizeS;
+  padding: $euiSizeXS 0;
   border-bottom: 1px solid $euiColorLightShade;
   background-color: $euiColorEmptyShade;
 
@@ -29,12 +29,14 @@
 
 .mapTooltipSelector__propertyRow-isDraggingOver {
   // Don't allow interaction events while layer is being re-ordered
+  // sass-lint:disable-block no-important
   pointer-events: none !important;
 }
 
 .mapTooltipSelector__propertyContent {
   overflow: hidden;
   flex-grow: 1;
+  min-height: 24px;
 }
 
 .mapTooltipSelector__propertyIcons {
@@ -50,6 +52,7 @@
   0% {
     opacity: 0;
   }
+
   100% {
     opacity: 1;
   }
diff --git a/x-pack/legacy/plugins/maps/public/components/add_tooltip_field_popover.js b/x-pack/legacy/plugins/maps/public/components/add_tooltip_field_popover.js
index 09d3d9e72801a..53a9fd33c2612 100644
--- a/x-pack/legacy/plugins/maps/public/components/add_tooltip_field_popover.js
+++ b/x-pack/legacy/plugins/maps/public/components/add_tooltip_field_popover.js
@@ -13,6 +13,7 @@ import {
   EuiSelectable,
   EuiButton,
   EuiSpacer,
+  EuiTextAlign
 } from '@elastic/eui';
 import { FormattedMessage } from '@kbn/i18n/react';
 import { i18n } from '@kbn/i18n';
@@ -142,13 +143,16 @@ export class AddTooltipFieldPopover extends Component {
 
         <EuiSpacer size="xs" />
         <EuiPopoverFooter>
-          <EuiButton
-            fill
-            isDisabled={this.state.checkedFields.length === 0}
-            onClick={this._onAdd}
-          >
-            {addLabel}
-          </EuiButton>
+          <EuiTextAlign textAlign="right">
+            <EuiButton
+              fill
+              isDisabled={this.state.checkedFields.length === 0}
+              onClick={this._onAdd}
+              size="s"
+            >
+              {addLabel}
+            </EuiButton>
+          </EuiTextAlign>
         </EuiPopoverFooter>
       </Fragment>
     );
diff --git a/x-pack/legacy/plugins/maps/public/components/tooltip_selector.js b/x-pack/legacy/plugins/maps/public/components/tooltip_selector.js
index 06b1fe30f9936..f5a4b94072a4d 100644
--- a/x-pack/legacy/plugins/maps/public/components/tooltip_selector.js
+++ b/x-pack/legacy/plugins/maps/public/components/tooltip_selector.js
@@ -98,7 +98,7 @@ export class TooltipSelector extends Component {
                       'mapTooltipSelector__propertyRow-isDraggingOver': snapshot.isDraggingOver
                     })}
                   >
-                    <EuiText className="mapTooltipSelector__propertyContent">
+                    <EuiText className="mapTooltipSelector__propertyContent" size="s">
                       {this._getPropertyLabel(propertyName)}
                     </EuiText>
                     <div className="mapTooltipSelector__propertyIcons">
@@ -155,6 +155,8 @@ export class TooltipSelector extends Component {
 
         {this._renderProperties()}
 
+        <EuiSpacer size="s" />
+
         <EuiTextAlign textAlign="center">
           <AddTooltipFieldPopover
             onAdd={this._onAdd}
diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/update_source_editor.js b/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/update_source_editor.js
index 606b0721a73dc..6627c4f335f08 100644
--- a/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/update_source_editor.js
+++ b/x-pack/legacy/plugins/maps/public/layers/sources/es_search_source/update_source_editor.js
@@ -196,13 +196,14 @@ export class UpdateSourceEditor extends Component {
     let topHitsCheckbox;
     if (this.state.dateFields && this.state.dateFields.length) {
       topHitsCheckbox = (
-        <EuiFormRow>
+        <EuiFormRow display="rowCompressed">
           <EuiSwitch
             label={i18n.translate('xpack.maps.source.esSearch.useTopHitsLabel', {
               defaultMessage: `Show most recent documents by entity`,
             })}
             checked={this.props.useTopHits}
             onChange={this.onUseTopHitsChange}
+            compressed
           />
         </EuiFormRow>
       );
@@ -218,13 +219,14 @@ export class UpdateSourceEditor extends Component {
           />
         </EuiFormRow>
 
-        <EuiFormRow>
+        <EuiFormRow display="rowCompressed">
           <EuiSwitch
             label={i18n.translate('xpack.maps.source.esSearch.extentFilterLabel', {
               defaultMessage: `Dynamically filter for data in the visible map area`,
             })}
             checked={this.props.filterByMapBounds}
             onChange={this._onFilterByMapBoundsChange}
+            compressed
           />
         </EuiFormRow>