Revert "Change entry-point for dependencies to be bs_theme_dependenci…

…es() instead of page() constructors" This reverts commit 147462a from #810
rstudio · Nov 2, 2023 · a928fad · a928fad
1 parent b5ab607
commit a928fad
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/R/bs-dependencies.R b/R/bs-dependencies.R
@@ -146,7 +146,6 @@ bs_theme_dependencies <- function(
         meta = list(viewport = "width=device-width, initial-scale=1, shrink-to-fit=no")
       )
     ),
-    if (version >= 5) component_dependencies(),
     htmlDependencies(out_file)
   ))
 }

diff --git a/R/page.R b/R/page.R
@@ -25,7 +25,8 @@ page <- function(..., title = NULL, theme = bs_theme(), lang = NULL) {
       tags$body(...),
       title = title,
       theme = theme,
-      lang = lang
+      lang = lang,
+      component_dependencies()
     ),
     theme = theme
   )
@@ -37,7 +38,13 @@ page <- function(..., title = NULL, theme = bs_theme(), lang = NULL) {
 #' @export
 page_fluid <- function(..., title = NULL, theme = bs_theme(), lang = NULL) {
   as_page(
-    shiny::fluidPage(..., title = title, theme = theme, lang = lang),
+    shiny::fluidPage(
+      ...,
+      title = title,
+      theme = theme,
+      lang = lang,
+      component_dependencies()
+    ),
     theme = theme
   )
 }
@@ -49,7 +56,13 @@ page_fluid <- function(..., title = NULL, theme = bs_theme(), lang = NULL) {
 #' @export
 page_fixed <- function(..., title = NULL, theme = bs_theme(), lang = NULL) {
   as_page(
-    shiny::fixedPage(..., title = title, theme = theme, lang = lang),
+    shiny::fixedPage(
+      ...,
+      title = title,
+      theme = theme,
+      lang = lang,
+      component_dependencies()
+    ),
     theme = theme
   )
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -146,7 +146,6 @@ bs_theme_dependencies <- function( @@
             meta = list(viewport = "width=device-width, initial-scale=1, shrink-to-fit=no")
           )
         ),
-        if (version >= 5) component_dependencies(),
         htmlDependencies(out_file)
       ))
     }
@@ Expand Down @@