Sidebar not able to close in mobile browser, if tapped on the sidebar launcher icon

[Nithya]

This issue is reproducable in http://semantic-ui.com/modules/sidebar.html also.

In my page, I had used attach event for setting launcher btn for sidebar as below
$('#sidebar-right').sidebar('attach events', '.sidebar-toggle-btn')

Step1 -> clicking on the sidebar-toggle-btn, shows the sidebar.

Step2 -> When using in Desktop => clicking on the sidebar-toggle-btn, triggers the sidebar toggle event and since sidebar is already visible, it hides the sidebar as expected.

Step 2 again in Mobile => clicking on the sidebar-toggle-btn, triggers the 'clickaway' function as the button is part of the dimmed page, also it moves to showing of the sidebar as well but it also ends up calling sidebar toggle event based on the 'attach event' setting and ends up closing the sidebar.

Couldnt figure out why it happens only in mobile.

I tried in my Mobile browser(chrome) and through browser mobile emulation - both had same results.

Please help me out guys.

Thanks in Advance.

[jlukic]

There might be another explanation. The default transition is set to auto which uses is mobile to determine which animation to use. Mobile was using RegExp.test which would return false for mobile when called twice.

As with exec (or in combination with it), test called multiple times on the same global regular expression instance will advance past the previous match.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/test

[jlukic]

I can confirm this fixes issues with docs sidebar button. Thanks for spotting this issue.

[Nithya]

Hey Jack,
Thanks for the Quick response.
Would applying the same fix from your commit to build-1.7.3 fix the issue or would there be someother changes needed to fix this in 1.7.3 ?

I tried the same changes available in you commit in build-1.7.3 but it didnt solve the problem.