Merge pull request #6990 from BigFunger/add-data-processor-dropdown

[add data] removes Add Processor button
elastic · Apr 20, 2016 · 36b332f · 36b332f
2 parents 84755e5 + 939a6f8
commit 36b332f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/...blic/settings/sections/indices/add_data_steps/pipeline_setup/directives/pipeline_setup.js b/...blic/settings/sections/indices/add_data_steps/pipeline_setup/directives/pipeline_setup.js
@@ -69,6 +69,13 @@ app.directive('pipelineSetup', function () {
         pipeline.updateParents();
       });
 
+      $scope.$watch('processorType', (newVal) => {
+        if (!newVal) return;
+
+        pipeline.add(newVal.Type);
+        $scope.processorType = '';
+      });
+
       $scope.$watch('pipeline.dirty', simulatePipeline);
     }
   };

diff --git a/.../public/settings/sections/indices/add_data_steps/pipeline_setup/views/pipeline_setup.html b/.../public/settings/sections/indices/add_data_steps/pipeline_setup/views/pipeline_setup.html
@@ -19,15 +19,11 @@
 </div>
 
 <div class="form-group">
-  <label>Processor Type:</label>
   <select
     class="form-control"
     ng-options="processorType.title for processorType in processorTypes"
-    ng-model="processorType">
+    ng-model="processorType"
+    ng-disabled="pipeline.hasCompileError">
+    <option value="">Select a Processor...</option>
   </select>
 </div>
-<button
-  ng-click="pipeline.add(processorType.Type)"
-  ng-disabled="!processorType || pipeline.hasCompileError">
-  Add Processor
-</button>