bindFillRole() should attach its HTMLDependency() to fill items

[cpsievert]

With this change, something like this now works:

div(class = "html-fill-container", bindFillRole(div(), item = TRUE))

The importance of which has increased since the emergence of div(bslib::as_fillable_container()), which only adds the HTML classes (not the dependencies).

And since bslib::page_fillable() uses as_fillable_container(), this currently doesn't work:

library(shiny)
library(plotly)
library(bslib)

ui <- page_fillable(
  plotlyOutput("p")
)

server <- function(input, output) {
  output$p <- renderPlotly({
    plotly::plot_ly()
  })
}

shinyApp(ui, server)

[gadenbuie]

This looks great to me. I was assuming that bindFillRole() always added the fill dependency, not just to containers, so I think making that explicitly the case is a positive move.