address design feedback

elastic · Sep 15, 2020 · 7f87c1a · 7f87c1a
1 parent 1aa0ef4
commit 7f87c1a
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/...mponents/pipeline_processors_editor/components/pipeline_processors_editor_item_status.tsx b/...mponents/pipeline_processors_editor/components/pipeline_processors_editor_item_status.tsx
@@ -18,7 +18,7 @@ interface ProcessorStatusIcon {
 
 const processorStatusToIconMap: Record<ProcessorStatus, ProcessorStatusIcon> = {
   success: {
-    icon: 'check',
+    icon: 'checkInCircleFilled',
     iconColor: 'success',
     label: i18n.translate('xpack.ingestPipelines.pipelineEditorItem.successStatusAriaLabel', {
       defaultMessage: 'Success',
@@ -54,7 +54,7 @@ const processorStatusToIconMap: Record<ProcessorStatus, ProcessorStatusIcon> = {
   },
   inactive: {
     icon: 'dot',
-    iconColor: 'subdued',
+    iconColor: '#D3DAE6', // $euiColorLightShade
     label: i18n.translate('xpack.ingestPipelines.pipelineEditorItem.inactiveStatusAriaLabel', {
       defaultMessage: 'Not run',
     }),
@@ -65,7 +65,7 @@ const processorStatusToIconMap: Record<ProcessorStatus, ProcessorStatusIcon> = {
 // This is not expected and likely means we need to modify the code to support a new status
 const unknownStatus = {
   icon: 'dot',
-  iconColor: 'subdued',
+  iconColor: '#D3DAE6', // $euiColorLightShade
   label: i18n.translate('xpack.ingestPipelines.pipelineEditorItem.unknownStatusAriaLabel', {
     defaultMessage: 'Unknown',
   }),

diff --git a/...ion/components/pipeline_processors_editor/components/test_pipeline/test_output_button.tsx b/...ion/components/pipeline_processors_editor/components/test_pipeline/test_output_button.tsx
@@ -12,7 +12,7 @@ const i18nTexts = {
   buttonLabel: i18n.translate(
     'xpack.ingestPipelines.pipelineEditor.testPipeline.outputButtonLabel',
     {
-      defaultMessage: 'Output',
+      defaultMessage: 'View output',
     }
   ),
 };

diff --git a/.../components/pipeline_processors_editor/components/test_pipeline/test_pipeline_actions.tsx b/.../components/pipeline_processors_editor/components/test_pipeline/test_pipeline_actions.tsx
@@ -18,7 +18,7 @@ const i18nTexts = {
   testPipelineActionsLabel: i18n.translate(
     'xpack.ingestPipelines.pipelineEditor.testPipeline.testPipelineActionsLabel',
     {
-      defaultMessage: 'Test:',
+      defaultMessage: 'Test pipeline:',
     }
   ),
 };

diff --git a/...ne_processors_editor/components/test_pipeline/test_pipeline_flyout_tabs/tab_documents.tsx b/...ne_processors_editor/components/test_pipeline/test_pipeline_flyout_tabs/tab_documents.tsx
@@ -82,6 +82,7 @@ export const DocumentsTab: React.FunctionComponent<Props> = ({
         size="s"
         isLoading={isRunningTest}
         disabled={isSubmitButtonDisabled}
+        iconType="play"
       >
         {isRunningTest ? (
           <FormattedMessage