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

Issue with iOS? #146

Closed
trevorholman opened this issue Mar 4, 2014 · 7 comments
Closed

Issue with iOS? #146

trevorholman opened this issue Mar 4, 2014 · 7 comments

Comments

@trevorholman
Copy link

I have this working fine on everything but Apple devices. Is there a known issue or something I'm missing?

@gitsuraj
Copy link

gitsuraj commented Apr 7, 2014

Same issue works fine with everything but iPhone

@topher6345
Copy link

In particular,when I activate the sidr menu, the animation seems to happen twice, and the transistion is jerky.

Also the menu items, once they appear, are not the Google/Apple recommended 40 pixels to be user-friendly for touch devices.

@glesperance
Copy link

+1

@ethanbeyer
Copy link

I'm currently unable to get this to work on iOS - when I click on the toggle link, nothing happens. If I click on the link within a desktop browser, it works perfectly fine. Not sure what to do with it!

@englishextra
Copy link

You need to include touchwipe.

For instance:

                yepnope.injectJs("http://ajax.googleapis.com/ajax/libs/jquery/2.1.0/jquery.min.js", function () {
                    yepnope.injectJs("http://cdn.jsdelivr.net/jquery.sidr/1.1.1/jquery.sidr.js", function () {
                        "undefined"!==typeof window.jQuery&&function(a){a(document).ready(function(){a("#leftnavbtn").sidr({name:"leftnav",source:"#leftnav"});a(window).click(function(){a.sidr("close","leftnav")})})}(jQuery);
                        Modernizr.touch&&yepnope.injectJs("http://cdn.jsdelivr.net/jquery.touchwipe/1.1.1/jquery.touchwipe.min.js", function () {
                            (function(a){a(window).touchwipe({wipeLeft:function(){a.sidr("close","leftnav")},wipeRight:function(){a.sidr("open","leftnav")},preventDefaultEvents:!1})})(jQuery);
                        },{charset:"utf-8"},5E3);
                    },{charset:"utf-8"},5E3);
                },{charset:"utf-8"},5E3);

see here http://englishextra.github.io/libs/16pixels/

@alex-bukach
Copy link

Any progress with this issue? I'm unable to make it work on iOS as well.

artberri added a commit that referenced this issue Jan 9, 2016
…ixes #179 and fixes #266 and fixes #246 and fixes #245 and fixes #211 and fixes #178 and fixes #118 and fixes #98  and fixes #214 and fixes #117 and fixes #130 and fixes #144 and fixes #146 and fixes #165 and fixes #189 and fixes #193 and fixes #203 and fixes #213 and fixes #218 and fixes #240 and fixes #249
@artberri
Copy link
Owner

artberri commented Jan 9, 2016

Preparing a new version with a fix for this https://github.com/artberri/sidr/tree/reborn

@artberri artberri closed this as completed Jan 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants