Issue with plotlyOutput width argument

[vnijs]

Same install with 4.6.0 the width arguments works fine. After upgrading to 4.7.0 the default width is very narrow and changing the width setting has no effect in the browser. After reinstalling 4.6.0 the width argument works fine again. Any ideas?

library(shiny)
library(plotly)

## Define UI for application 
ui <- fluidPage(
  titlePanel("My Shiny app with plotly"),
  sidebarLayout(
    sidebarPanel(
      sliderInput("my_slider", label = "Number of data points", 
                  min = 100, max = 500, value = 200)
    ),
    mainPanel(
      plotlyOutput("show_scatter", width = "100%")
    )
  )
)

# Define server logic 
server <- function(input, output) {
  output$show_scatter <- renderPlotly({
    y <- rnorm(input$my_slider)
    x <- y  + rnorm(input$my_slider)
    df <- data.frame(x = x, y = y)
    p <- ggplot(df, aes(x = x, y = y)) + 
      geom_point()
    ggplotly(p)
  })
}

# Run the application with plotly
shinyApp(ui = ui, server = server)

[vnijs]

Adding screenshots for 4.6 and 4.7.

In 4.6.0:

In 4.7.0:

[vnijs]

Just noticed something interesting ... as I change the size of the viewer panel in Rstudio, the size of the plot changes in the browser (after refresh).

[cpsievert]

I'm not sure specifying height/width in plotlyOutput() has ever really worked the way you'd expect it to in all situations/contexts.

For now, you can specify height/width in ggplotly() or plot_ly(), but I'll think about a better long-term solution

[vnijs]

OK. But autosizing worked when no width was set in 4.6.0 and no longer does that at all in 4.7.0. In fact, the size of the image in a Shiny app shown in Chrome seems to be determined by the size of the Viewer panel in Rstudio. Adding another set of screenshots below as illustration.

4.7.0

4.6.0

[cpsievert]

Ok, yea, that's definitely a problem. Does the same thing happen if you launch the app outside of RStudio?

[cpsievert]

Also, BTW, an official fix for this may take a bit, but for now, if you want the old behavior back:

l <- ggplotly(qplot(1:10))
l$x$layout$width <- NULL
l$x$layout$height <- NULL
l$width <- NULL
l$height <- NULL
l

[vnijs]

The issue doesn't seems to occur outside of Rstudio. The fix does seems to work from Rstudio.

[vnijs]

Works! Thanks @cpsievert

[peranti]

@cpsievert The fix works for dynamically adjusting the plotly output wrt width. However, the height of the plot is not dynamically adjusting to the browser window.

[daattali]

It seems width+height in plotlyOutput() still don't work (in a browser), but setting it in ggplotly() does. If they aren't intended to work in plotlyOutput(), would it be a good idea to remove them from the signature, or at least mention in the documentation that they don't work?

[cpsievert]

Could you be more specific by "doesn't work"? Both height/width seem to work for me.

[daattali]

the height and width arguments seem to be completely ignored when used in plotlyOutput in a shiny app

> packageVersion("shiny")
[1] ‘1.0.5’
> packageVersion("htmlwidgets")
[1] ‘0.9’
> packageVersion("plotly")
[1] ‘4.7.0’

Actually, after doing a specific small code example I think that the container is set to the proper dimensions, but the plot isn't. Example:

library(shiny)
library(plotly)
library(ggplot2)

ui <- fluidPage(
  plotlyOutput("p1", height=100, width=100),
  plotlyOutput("p2")
)

server <- function(input, output, session) {
  output$p1 <- renderPlotly({
    ggplotly(ggplot(mtcars, aes(mpg,wt))+geom_point())
  })
  output$p2 <- renderPlotly({
    ggplotly(ggplot(mtcars, aes(mpg,wt))+geom_point(), width=100,height=100)
  })
}

shinyApp(ui, server)

Result:

Notice that the small plot (which is p2) looks like it's starting 100px down, meaning the p1 actually does think it's only 100px tall, but the plot itself is regular sized. Hope that makes sense

[cpsievert]

Changes have been made here since 4.7.0 -- 7a8ee9c

Can you please try with a more recent version?

[daattali]

Ah yes the dev version works! The bug report title is for v 4.7.0 so I thought that was the latest

[dfrail24]

I was seeing this issue with the newest dev version - reverting back to 4.7.1 version on CRAN did fix the issue though.

[cosmin-novac]

Still seeing this issue in 4.9.0 when using plotlyOutput inside renderUI.

Setting the plotlyOutput width to "auto" results in a plot that is too small width-wise, and setting the width to NULL removes the height style attribute alltogether. Setting the width to 100% doesn't work at all, the app just crashes.