Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

deep linking to tabs seems not to work in MSIE 11 #10771

Closed
hubert-ried opened this issue Nov 15, 2017 · 2 comments
Closed

deep linking to tabs seems not to work in MSIE 11 #10771

hubert-ried opened this issue Nov 15, 2017 · 2 comments

Comments

@hubert-ried
Copy link

hubert-ried commented Nov 15, 2017

How to reproduce this bug:

In f6 it should be possible to set an anchor link to a tabpanel. Clicking on it should open the referenced tab. This works fine in chrome and ff, but not in msie 11 (see jsfiddle-link, click on the "go to panel"-links)

What should happen:

Linked tab should open

What happened instead:

nothing

Browser(s) and Device(s) tested on:

Internet Explorer 11 (11.0.9600.17843), Windows 7 Professional Service Pack 1

Foundation Version(s) you are using:

6.4.3

Test case link:

https://jsfiddle.net/1zptbLbm/

ADD:
I found a workaround, that seem to work at least in regular tabs: when I select the target tab onclick via "selectTab", everything is fine, smth like:

$(function() {
    $('.deeplinks').on('click',function(ev){
        $("#example-tabs").foundation('selectTab',ev.currentTarget.hash.substr(1),true);
    });
    $(document).foundation();
});

Unfortunately we use a responsive accordion tab, where "selectTab" is not implemented:
https://jsfiddle.net/3u172fdb/

@DanielRuf DanielRuf self-assigned this Feb 23, 2018
@DanielRuf
Copy link
Contributor

https://developer.mozilla.org/en-US/docs/Web/Events/popstate#Browser_compatibility

[3] IE & Edge do not fire the popstate event when the URL's hash value changes, see the bug report.

https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/3740423/

@DanielRuf
Copy link
Contributor

https://codepen.io/DanielRuf/pen/YeMwrJ

This includes a fix which I will provide as PR very soon.

For responsive accordion tabs component see #10961

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants