exception when `isEmptyAnchor` called with SVG <use> tag about bootstrap.native HOT 6 CLOSED

A possible fix is to rewrite these:

t.href.slice(-1)
t.parentNode.href.slice(-1)

into these:

t.getAttribute('href')).slice(-1)
t.parentNode.getAttribute('href')).slice(-1)

since the return value of getAttribute("href") is guaranteed to be a string, based on DOM Level 2 Spec and SVG is compatible with DOM Level 2.

I made a patch in a forked repo but tend to not send a PR for now, only for your reference: zbryikt@737dc2e

I found href.slice in both src/util/isEmptyAnchor.js and src/components/dropdown-native.js so I patched both of them in that commit.

from bootstrap.native.

Change implemented and solved issues. New version online now.

from bootstrap.native.

Please provide a test pen/fiddle to look into it.

from bootstrap.native.

It seems that v4 is now in a separated branch (not sure if it's still maintained since there is no separated release for v4 currently) and it doesn't seem to still available in the latest release files so I'm not sure what's the latest officially released version for v4, so in case that I use a wrong version of code, I copied all code from latest snapshot of dist/bootstrap-native.min.js in bs-4 branch directly and make a pen here:

https://codepen.io/loadingio/pen/YzRpJvz

To reproduce, simply toggle the dropdown on and then click the red circle, which is a SVG image tag with data url in itshref attribute.

Have also tried v5 and it seems to work correctly but we don't use v5 so not sure if we've done it right - since this issue mainly discuss v4, I will skip that part here. :)

(migration to v5 will be a big task and when we are ready there will be probably v6 so we will still use bootstrap v4 for some time. )

from bootstrap.native.

@zbryikt just that you know, I've updated the local repo on my machine with the proposed changes but I'm having problems with some dependencies vulnerabilities, we might have to wait a couple of days to get them sorted out.

from bootstrap.native.

No worries! It's an open-source project so please don't feel any pressure, especially this bs-4 branch is more like a legacy version. We can use the forked custom built version first. Thank you for this awesome lib! :)

from bootstrap.native.