plotlyOutput()'s width and height now default to NULL (and default CSS sizing is provided using a non-inline style) (#2192)

cpsievert · web-flow · commit 20620965d2ce · 2022-10-19T16:14:44.000-05:00
diff --git a/R/shiny.R b/R/shiny.R
@@ -23,7 +23,7 @@
 #' @name plotly-shiny
 #'
 #' @export
-plotlyOutput <- function(outputId, width = "100%", height = "400px", 
+plotlyOutput <- function(outputId, width = NULL, height = NULL, 
                          inline = FALSE, reportTheme = TRUE) {
   args <- list(
     outputId = outputId, 
diff --git a/inst/htmlwidgets/lib/plotlyjs/plotly-htmlwidgets.css b/inst/htmlwidgets/lib/plotlyjs/plotly-htmlwidgets.css
@@ -1,3 +1,8 @@
+.plotly.html-widget {
+  width: 100%;
+  height: 400px;
+}
+
 /*
 just here so that plotly works
 correctly with ioslides.
