BUG: Do not subscribe to blend mode changed with 2D images

InsightSoftwareConsortium · Jan 15, 2020 · 7356aa6 · 7356aa6
1 parent ec066cd
commit 7356aa6
Showing 1 changed file with 24 additions and 24 deletions.
diff --git a/js/lib/viewer.js b/js/lib/viewer.js
@@ -585,32 +585,32 @@ const ViewerView = widgets.DOMWidgetView.extend({
       }
       this.model.itkVtkViewer.subscribeToggleCroppingPlanes(onToggleCroppingPlanes)
 
-      const onBlendModeChanged = (blend) => {
-        let pythonMode = null
-        switch(blend) {
-        case 0:
-          pythonMode = 'composite'
-          break
-        case 1:
-          pythonMode = 'max'
-          break
-        case 2:
-          pythonMode = 'min'
-          break
-        case 3:
-          pythonMode = 'average'
-          break
-        default:
-          throw new Error('Unknown blend mode')
-        }
-        if (pythonMode !== this.model.get('blend')) {
-          this.model.set('blend', pythonMode)
-          this.model.save_changes()
+      if (!this.model.use2D) {
+        const onBlendModeChanged = (blend) => {
+          let pythonMode = null
+          switch(blend) {
+          case 0:
+            pythonMode = 'composite'
+            break
+          case 1:
+            pythonMode = 'max'
+            break
+          case 2:
+            pythonMode = 'min'
+            break
+          case 3:
+            pythonMode = 'average'
+            break
+          default:
+            throw new Error('Unknown blend mode')
+          }
+          if (pythonMode !== this.model.get('blend')) {
+            this.model.set('blend', pythonMode)
+            this.model.save_changes()
+          }
         }
-      }
-      this.model.itkVtkViewer.subscribeBlendModeChanged(onBlendModeChanged)
+        this.model.itkVtkViewer.subscribeBlendModeChanged(onBlendModeChanged)
 
-      if (!this.model.use2D) {
         const onViewModeChanged = (mode) => {
           let pythonMode = null;
           switch (mode) {