Kaynağa Gözat

Only emit `executed` on markdown and code cells.

Afshin Darian 6 yıl önce
ebeveyn
işleme
9545cefccc
1 değiştirilmiş dosya ile 3 ekleme ve 5 silme
  1. 3 5
      packages/notebook/src/actions.tsx

+ 3 - 5
packages/notebook/src/actions.tsx

@@ -1368,6 +1368,7 @@ namespace Private {
     case 'markdown':
       (child as MarkdownCell).rendered = true;
       child.inputHidden = false;
+      executed.emit({ parent, child });
       break;
     case 'code':
       if (session) {
@@ -1382,13 +1383,12 @@ namespace Private {
             }
           }
 
-          const run = reply ? reply.content.status === 'ok' : true;
-
           if (reply ? reply.content.status === 'ok' : true) {
             executed.emit({ parent, child });
+            return true;
           }
 
-          return run;
+          return false;
         }).catch(reason => {
           if (reason.message !== 'Canceled') {
             throw reason;
@@ -1402,8 +1402,6 @@ namespace Private {
       break;
     }
 
-    executed.emit({ parent, child });
-
     return Promise.resolve(true);
   }