Activate only the appropriate pairing options

#213
mwouts · Jun 17, 2019 · abafe01 · abafe01
1 parent ffbdc24
commit abafe01
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 9 deletions.
diff --git a/jupytext/nbextension/index.js b/jupytext/nbextension/index.js
@@ -83,8 +83,10 @@ define([
         checkAutosave();
     }
 
-    function jupytext_pair(formats, text) {
-        return $('<li/>').append($('<a/>')
+    function jupytext_pair(formats, text, active) {
+        return $('<li/>')
+            .addClass(active ? null : 'disabled')
+            .append($('<a/>')
             .attr('id', 'jupytext_pair_' + formats.replace(':', '_').replace(',', '_'))
             .text(text)
             .attr('title',
@@ -137,14 +139,22 @@ define([
             JupytextActions.append($('<li/>').addClass('divider'));
             JupytextActions.append(toggle_autosave);
             JupytextActions.append($('<li/>').addClass('divider'));
-            JupytextActions.append(jupytext_pair('ipynb,auto:light', 'Pair Notebook with light Script'));
-            JupytextActions.append(jupytext_pair('ipynb,auto:percent', 'Pair Notebook with percent Script'));
-            JupytextActions.append(jupytext_pair('ipynb,auto:hydrogen', 'Pair Notebook with Hydrogen Script'));
-            JupytextActions.append(jupytext_pair('ipynb,md', 'Pair Notebook with Markdown'));
-            JupytextActions.append(jupytext_pair('ipynb,Rmd', 'Pair Notebook with R Markdown'));
-            JupytextActions.append(jupytext_pair('custom', 'Custom pairing'));
+
+            var notebook_extension = Jupyter.notebook.notebook_path.split('.').pop();
+            var active = (notebook_extension === 'ipynb' || (notebook_extension != 'md' && notebook_extension != 'Rmd'));
+            JupytextActions.append(jupytext_pair('ipynb,auto:light', 'Pair Notebook with light Script', active));
+            JupytextActions.append(jupytext_pair('ipynb,auto:percent', 'Pair Notebook with percent Script', active));
+            JupytextActions.append(jupytext_pair('ipynb,auto:hydrogen', 'Pair Notebook with Hydrogen Script', active));
+
+            active = (notebook_extension === 'ipynb' || notebook_extension === 'md');
+            JupytextActions.append(jupytext_pair('ipynb,md', 'Pair Notebook with Markdown', active));
+
+            active = (notebook_extension === 'ipynb' || notebook_extension === 'Rmd');
+            JupytextActions.append(jupytext_pair('ipynb,Rmd', 'Pair Notebook with R Markdown', active));
+
+            JupytextActions.append(jupytext_pair('custom', 'Custom pairing', true));
             JupytextActions.append($('<li/>').addClass('divider'));
-            JupytextActions.append(jupytext_pair('none', 'Unpair notebook'));
+            JupytextActions.append(jupytext_pair('none', 'Unpair notebook', true));
 
             $('#jupytext_sub_menu').after('<li class="divider"/>');
 

diff --git a/packages/labextension/src/index.ts b/packages/labextension/src/index.ts
@@ -102,6 +102,18 @@ const extension: JupyterLabPlugin<void> = {
 
           return jupytext_formats == formats;
         },
+        isEnabled: () => {
+          if (formats == "custom" || formats == "none")
+            return true;
+          const notebook_extension: string = notebook_tracker.currentWidget.context.path.split('.').pop();
+          if (notebook_extension == "ipynb")
+            return true;
+          if (notebook_extension == "md")
+            return formats == "ipynb,md";
+          if (notebook_extension == "Rmd")
+            return formats == "ipynb,Rmd";
+          return formats != "ipynb,md" && formats != "ipynb,Rmd";
+        },
         execute: () => {
           console.log("Jupytext: executing command=" + command);
           if (formats == "custom") {