Merge pull request #415 from Kitware/proxy-panel

fix(ProxyEditorWidget): Add autoApply option
Kitware · Jan 4, 2018 · b11c216 · b11c216
2 parents ed43866 + a9158c8
commit b11c216
Showing 1 changed file with 17 additions and 5 deletions.
diff --git a/src/React/Widgets/ProxyEditorWidget/index.js b/src/React/Widgets/ProxyEditorWidget/index.js
@@ -32,7 +32,14 @@ export default class ProxyEditorWidget extends React.Component {
 
   updateChangeSet(change) {
     const changeSet = Object.assign({}, this.state.changeSet, change);
-    this.setState({ changeSet });
+    if (this.props.autoApply) {
+      if (this.props.onApply) {
+        this.props.onApply(changeSet);
+      }
+      this.setState({ changeSet: {} });
+    } else {
+      this.setState({ changeSet });
+    }
   }
 
   applyChanges() {
@@ -58,10 +65,13 @@ export default class ProxyEditorWidget extends React.Component {
             onChange={this.updateFilter}
             className={style.filter}
           />
-          <i
-            className={changeCount ? style.validateButtonOn : style.validateButton}
-            onClick={this.applyChanges}
-          />
+          {this.props.autoApply ?
+            null :
+            <i
+              className={changeCount ? style.validateButtonOn : style.validateButton}
+              onClick={this.applyChanges}
+            />
+          }
         </div>
         <div className={style.contentContainer}>
           {this.props.children}
@@ -87,9 +97,11 @@ ProxyEditorWidget.propTypes = {
   onApply: PropTypes.func,
   sections: PropTypes.array.isRequired,
   onCollapseChange: PropTypes.func,
+  autoApply: PropTypes.bool,
 };
 
 ProxyEditorWidget.defaultProps = {
   advanced: false,
+  autoApply: false,
 };