Merge pull request #13705 from quarto-dev/fix/issue-13694

Author: cderv

news/changelog-1.9.md

@@ -9,6 +9,7 @@ All changes included in 1.9:
 - ([#13616](https://github.com/quarto-dev/quarto-cli/issues/13616)): Fix fatal error when rendering manuscript projects with custom blocks like ConditionalBlock.
 - ([#13625](https://github.com/quarto-dev/quarto-cli/issues/13625)): Fix Windows file locking error (os error 32) when rendering with `--output-dir` flag. Context cleanup now happens before removing the temporary `.quarto` directory, ensuring file handles are properly closed.
 - ([#13633](https://github.com/quarto-dev/quarto-cli/issues/13633)): Fix detection and auto-installation of babel language packages from newer error format that doesn't explicitly mention `.ldf` filename.
+- ([#13694](https://github.com/quarto-dev/quarto-cli/issues/13694)): Fix `notebook-view.url` being ignored - external notebook links now properly use specified URLs instead of local preview files.
 
 ## Dependencies
 

src/format/html/format-html-notebook-preview.ts

@@ -252,7 +252,7 @@ export const notebookPreviewer = (
         // to form links to this notebook
         const nbPreview = {
           title: resolvedTitle,
-          href: relative(inputDir, renderedNotebook[kHtmlPreview].hrefPath),
+          href: descriptor?.url || relative(inputDir, renderedNotebook[kHtmlPreview].hrefPath),
           supporting,
           resources,
           order: work.order,

tests/docs/smoke-all/2023/01/05/notebook-preview-complex.qmd

@@ -29,7 +29,7 @@ _quarto:
   tests:
     html:
       ensureHtmlElements:
-        - ['a.quarto-notebook-link']
+        - ['a.quarto-notebook-link[href*="kaggle.com"]']
         - ['div.quarto-alternate-notebooks a']
       ensureFileRegexMatches:
         - ['Source: TEST NB']