diff --git a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/_index.scss b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/_index.scss
index b219f59476ce9..fd074edf032fa 100644
--- a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/_index.scss
+++ b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/_index.scss
@@ -1,3 +1,4 @@
@import './layer_panel';
@import './filter_editor/filter_editor';
-@import './join_editor/resources/join';
\ No newline at end of file
+@import './join_editor/resources/join';
+@import './style_settings/style_settings';
\ No newline at end of file
diff --git a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/_style_settings.scss b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/_style_settings.scss
new file mode 100644
index 0000000000000..249b6dfca5c76
--- /dev/null
+++ b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/_style_settings.scss
@@ -0,0 +1,3 @@
+.mapStyleSettings__fixedBox {
+ width: $euiSize * 7.5;
+}
\ No newline at end of file
diff --git a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/style_settings.js b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/style_settings.js
index 2857065f04c70..69cf51fb29c0d 100644
--- a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/style_settings.js
+++ b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/style_settings/style_settings.js
@@ -21,7 +21,7 @@ export function StyleSettings({ layer, updateStyleDescriptor }) {
return (
-
+
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/_color_stops.scss b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/_color_stops.scss
index 001ca0685d0e9..519e97f4b30cd 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/_color_stops.scss
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/_color_stops.scss
@@ -1,6 +1,5 @@
.mapColorStop {
position: relative;
- padding-right: $euiSizeXL + $euiSizeS;
& + & {
margin-top: $euiSizeS;
@@ -17,23 +16,3 @@
}
}
-.mapColorStop__icons {
- flex-shrink: 0;
- display: none;
- position: absolute;
- right: 0;
- top: 50%;
- margin-right: -$euiSizeS;
- margin-top: -$euiSizeM;
-}
-
-@keyframes mapColorStopBecomeVisible {
-
- 0% {
- opacity: 0;
- }
-
- 100% {
- opacity: 1;
- }
-}
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_map_select.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_map_select.js
index e8d5754ef4206..436a92b619909 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_map_select.js
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_map_select.js
@@ -105,6 +105,7 @@ export class ColorMapSelect extends Component {
onChange={this._onColorMapSelect}
valueOfSelected={valueOfSelected}
hasDividers={true}
+ compressed
/>
{this._renderColorStopsInput()}
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_stops.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_stops.js
index 47c2d037e0c79..3e9b9e2aafc47 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_stops.js
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/color_stops.js
@@ -10,7 +10,19 @@ import { removeRow, isColorInvalid } from './color_stops_utils';
import { i18n } from '@kbn/i18n';
import { EuiButtonIcon, EuiColorPicker, EuiFlexGroup, EuiFlexItem, EuiFormRow } from '@elastic/eui';
-function getColorStopRow({ index, errors, stopInput, colorInput, deleteButton, onAdd }) {
+function getColorStopRow({ index, errors, stopInput, onColorChange, color, deleteButton, onAdd }) {
+ const colorPickerButtons = (
+
+ {deleteButton}
+
+
+ );
return (
-
-
- {stopInput}
- {colorInput}
-
-
- {deleteButton}
-
+
+ {stopInput}
+
+
+
-
-
+
+
);
}
@@ -80,17 +89,6 @@ export const ColorStops = ({
};
}
- function getColorInput(onColorChange, color) {
- return {
- colorError: isColorInvalid(color)
- ? i18n.translate('xpack.maps.styles.colorStops.hexWarningLabel', {
- defaultMessage: 'Color must provide a valid hex value',
- })
- : undefined,
- colorInput: ,
- };
- }
-
const rows = colorStops.map((colorStop, index) => {
const onColorChange = color => {
const newColorStops = _.cloneDeep(colorStops);
@@ -102,7 +100,15 @@ export const ColorStops = ({
};
const { stopError, stopInput } = getStopInput(colorStop.stop, index);
- const { colorError, colorInput } = getColorInput(onColorChange, colorStop.color);
+
+ const color = colorStop.color;
+
+ const colorError = isColorInvalid(color)
+ ? i18n.translate('xpack.maps.styles.colorStops.hexWarningLabel', {
+ defaultMessage: 'Color must provide a valid hex value',
+ })
+ : undefined;
+
const errors = [];
if (stopError) {
errors.push(stopError);
@@ -131,7 +137,7 @@ export const ColorStops = ({
deleteButton = getDeleteButton(onRemove);
}
- return getColorStopRow({ index, errors, stopInput, colorInput, deleteButton, onAdd });
+ return getColorStopRow({ index, errors, stopInput, onColorChange, color, deleteButton, onAdd });
});
return {rows}
;
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/dynamic_color_form.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/dynamic_color_form.js
index af5e5b37f5467..3dc356c31cf30 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/dynamic_color_form.js
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/color/dynamic_color_form.js
@@ -90,8 +90,10 @@ export function DynamicColorForm({
return (
-
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
-
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
-
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/__snapshots__/icon_select.test.js.snap b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/__snapshots__/icon_select.test.js.snap
index 706dc0763b7ca..8fa69e1a5b467 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/__snapshots__/icon_select.test.js.snap
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/__snapshots__/icon_select.test.js.snap
@@ -34,7 +34,7 @@ exports[`Should render icon select 1`] = `
}
closePopover={[Function]}
- display="inlineBlock"
+ display="block"
hasArrow={true}
isOpen={false}
ownFocus={true}
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/dynamic_icon_form.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/dynamic_icon_form.js
index afa11daf45217..9065102dc8bd7 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/dynamic_icon_form.js
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/dynamic_icon_form.js
@@ -53,8 +53,10 @@ export function DynamicIconForm({
return (
-
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+
{this._renderIconSelectable()}
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/static_icon_form.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/static_icon_form.js
index b20d8f2eba162..9b00b2fe38d7b 100644
--- a/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/static_icon_form.js
+++ b/x-pack/legacy/plugins/maps/public/layers/styles/vector/components/symbol/static_icon_form.js
@@ -20,8 +20,10 @@ export function StaticIconForm({
};
return (
-
- {staticDynamicSelect}
+
+
+ {staticDynamicSelect}
+