rename directives jupyter-download:script -> jupyter-download-script, similar for nb and notebook

Author: basnijholt

doc/source/index.rst

@@ -372,16 +372,16 @@ Downloading the code as a script
 --------------------------------
 
 Jupyter Sphinx includes 2 roles that can be used to download the code embedded in a document:
-``:jupyter-download:script:`` (for a raw script file) and ``:jupyter-download:notebook:`` or ``:jupyter-download:nb:`` (for
+``:jupyter-download-script:`` (for a raw script file) and ``:jupyter-download-notebook:`` or ``:jupyter-download-nb:`` (for
 a Jupyter notebook).
 
 These roles are equivalent to the standard sphinx `download role <https://www.sphinx-doc.org/en/master/usage/restructuredtext/roles.html#role-download>`__, **except** the extension of the file should not be given.
 For example, to download all the code from this document as a script we
 would use::
 
-    :jupyter-download:script:`click to download <index>`
+    :jupyter-download-script:`click to download <index>`
 
-Which produces a link like this: :jupyter-download:script:`click to download <index>`. The target that the role is
+Which produces a link like this: :jupyter-download-nb:`click to download <index>`. The target that the role is
 applied to (``index`` in this case) is the name of the document for which you wish to download
 the code. If a document contains ``jupyter-kernel`` directives with ``:id:`` specified, then
 the name provided to ``:id:`` can be used to get the code for the cells belonging to the

jupyter_sphinx/__init__.py

@@ -271,9 +271,13 @@ def setup(app):
     app.add_directive("jupyter-input", CellInput)
     app.add_directive("jupyter-output", CellOutput)
     app.add_directive("thebe-button", ThebeButton)
-    app.add_role("jupyter-download:notebook", JupyterDownloadRole())
-    app.add_role("jupyter-download:nb", JupyterDownloadRole())
-    app.add_role("jupyter-download:script", JupyterDownloadRole())
+    for sep in [":", "-"]:
+        # Since Sphinx 4.0.0 using ":" inside of a role/directive does not work.
+        # Therefore, we add "-" as separator to get e.g., jupyter-download-notebook
+        # We leave the ":" syntax for backward compatibility reasons.
+        app.add_role(f"jupyter-download{sep}notebook", JupyterDownloadRole())
+        app.add_role(f"jupyter-download{sep}nb", JupyterDownloadRole())
+        app.add_role(f"jupyter-download{sep}script", JupyterDownloadRole())
     app.add_transform(CombineCellInputOutput)
     app.add_transform(ExecuteJupyterCells)
 

jupyter_sphinx/ast.py

@@ -587,7 +587,8 @@ def apply_styling(node, thebe_config):
 
 class JupyterDownloadRole(ReferenceRole):
     def run(self):
-        _, filetype = self.name.split(":")
+        sep = ":" if ":" in self.name else "-"
+        _, filetype = self.name.rsplit(sep, maxsplit=1)
 
         assert filetype in ("notebook", "nb", "script")
         ext = ".ipynb" if filetype in ("notebook", "nb") else ".py"

tests/test_execute.py

@@ -610,7 +610,7 @@ def test_download_role(text, reftarget, caption, tmp_path):
         "reporter.get_source_and_line": lambda l: ("source", l),
     }
     mock_inliner.configure_mock(**config)
-    ret, msg = role("jupyter-download:notebook", text, text, 0, mock_inliner)
+    ret, msg = role("jupyter-download-notebook", text, text, 0, mock_inliner)
 
     if os.name == "nt":
         # Get equivalent abs path for Windows