fix #2271: use the normal progress bar by default when knitting takes…

… place in RStudio's background jobs pane
yihui · Aug 23, 2023 · ca71727 · ca71727
1 parent 50e28da
commit ca71727
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -12,6 +12,8 @@
 
 - Figure captions are no longer escaped for reStructuredText output, and the alt text can also be specified via the `fig.alt` chunk option now (thanks, @trevorld, #2023).
 
+- Use the correct type of progress bar when rendering Quarto documents in RStudio, which takes place in RStudio's background jobs pane (thanks, @hadley, #2271).
+
 # CHANGES IN knitr VERSION 1.43
 
 ## NEW FEATURES

diff --git a/R/utils.R b/R/utils.R
@@ -1140,6 +1140,8 @@ txt_pb = function(total, labels) {
     if (identical(con, '')) con = stdout()
     if (!inherits(con, 'connection')) return(TRUE)
     if (isatty(con)) return(FALSE)
+    # RStudio's background jobs
+    if (Sys.getenv('RSTUDIO_CHILD_PROCESS_PANE') == 'job') return(FALSE)
     # when RStudio is available, return FALSE
     is.null(tryCatch(rstudioapi::versionInfo(), error = function(e) NULL))
   })()