RinteRface / shinybulma

🌐 Bulma.io for Shiny
https://rinterface.github.io/shinybulma/
Other
111 stars 15 forks source link

Several css classes not available due to old bulma version #23

Closed thothal closed 3 years ago

thothal commented 3 years ago

The bulma framework introduced several new classes in newer version. Currently shinybulma uses version 0.7.2. Thus, features such as: is-family-* or spacing helpers are not available.

Can we change the CSS to a more recent version? Any reasons not to do yo? Ideally we would use the newest version to include all the features, but maybe that is a too drastic change and at least 0.7.4 to allow for the is-family-* classes?

Your thoughts?

JohnCoene commented 3 years ago

I just pushed an updated version of the CSS

thothal commented 3 years ago

Hm, I saw that you updated the css files of the themes which are in the repo (thanks), while the main bulma css (which is loaded from a cdn) still uses an old version (0.7.2. cf to the linked line in code in the issue). That is, things which are covered in the themes work now, while others (which are solely in the master bulma won't).

Compare this fiddle with the output of:

library(shinybulma)
library(shiny)

ui <- bulmaPage(bulmaSection(p("This text is code style", 
                               class = "is-family-code has-background-danger p-6 has-text-white")))

shinyApp(ui, function(...) {})

While the new is-family-code is properly displayed (I guess it also part of the themes css), the p-6 (the padding modifyer) does not work (I guess it is only in the master bulma css)

Fiddle Screenschot

Bulma_new

Shinybulma Screenshot

Bulma_old

Remedy

Update also the master bulma dependency in deps.R

JohnCoene commented 3 years ago

Sorry, I omitted that: should be good now; the snippet you shared worked fine on my end.