-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bulma.js as recently merged in #31raises an error in JS console #32
Comments
Could you show the code to capture that error? |
Took a while to track the error down, but here is a minimal reprex: library(shiny)
library(shinybulma)
shinyApp(bulmaPage(bulmaNavbar()), function(...) {}) So it is the |
bulmaJS has an API for navbar. It's probably conflicting with the current implementation. By the way I don't manage to access to the bulmaJS doc anymore. I'll check later ;) |
I tried that already, this is not the case. My guess is that Try: shinyApp(
bulmaPage(
bulmaNavbar(
bulmaNavbarBurger()
)
), function(...) {}
) and the error is gone. |
In many template I’v seen, the burger is included in the navbar template by default, only shown when screen size reaches a specific threshold. bulmaNavbarBurger should be internal to the template and not exported to the end user. As you mentioned, this would fix the issue. |
bulma.js
(updated in #31) is raising an error as can be seen in the JS console:The text was updated successfully, but these errors were encountered: