Skip to content
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

External links in bs4SidebarMenuItem() not opening properly #290

Closed
veer0318 opened this issue Jun 7, 2022 · 1 comment
Closed

External links in bs4SidebarMenuItem() not opening properly #290

veer0318 opened this issue Jun 7, 2022 · 1 comment
Labels
fixed-in-dev Fixed in development version

Comments

@veer0318
Copy link

veer0318 commented Jun 7, 2022

Hi,

I found a small yet annoying bug: I'm using bs4SidebarMenuItem() with href = 'https://www.google.com' (no tabName set) and newTab = T. This doesn't work: the link won't open. I have traced the issue here:
https://github.com/RinteRface/bs4Dash/blob/master/R/dashboardSidebar.R#L407

Because you set
data-toggle = "tab",
irrespective of whether it is an internal or external URL, external URLs are not loaded because of javascript code interfering.

This fix works and seems to not cause any regressions (AFAIK):
data-toggle= if (is.null(href)) "tab",

Could you put this fix in the source code?

Best,
Harmen

@DivadNojnarg DivadNojnarg added the fixed-in-dev Fixed in development version label Jul 25, 2022
@DivadNojnarg
Copy link
Member

Good catch. Added to dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-dev Fixed in development version
Projects
None yet
Development

No branches or pull requests

2 participants