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

Slides v2 beta10 does not work on android #7192

Closed
ValterSantosMatos opened this issue Jul 4, 2016 · 4 comments
Closed

Slides v2 beta10 does not work on android #7192

ValterSantosMatos opened this issue Jul 4, 2016 · 4 comments

Comments

@ValterSantosMatos
Copy link

Slides v2 beta10 does not work on android (unable to swipe)

Expect to see 3 swiper-pagination-bullets on the bottom and be able to swipe between the screens.
Worked in beta 9, works if emulate IOS device.

Steps to reproduce:
Just check the Plunker.

Which Ionic Version? 2.10

Plunker that shows an example of your issue

http://plnkr.co/edit/gW6ckkR5cZ4G8e3EbiRh?p=preview

@mangonecristian
Copy link

mangonecristian commented Jul 5, 2016

Same here. It works after a window resize.

Workaround:
./node_modules/ionic-angular/components/slides/slides.js

Change this:

Slides.prototype.update = function () {
    var _this = this;
    setTimeout(function () {
        _this.slider.update();
        // Don't allow pager to show with > 10 slides
        if (_this.length() > 10) {
            _this.showPager = false;
        }
    });
};

To this:

Slides.prototype.update = function () {
    var _this = this;
    setTimeout(function () {
        _this.slider.update();
        // Don't allow pager to show with > 10 slides
        if (_this.length() > 10) {
            _this.showPager = false;
        }
    },250);
};

@schankam
Copy link

schankam commented Jul 6, 2016

Also got the bug on Android (working fine on iOS). Looks like it does not work when using ion-slides inside an ion-content...

This was working fine before beta 9, noticed it only when I was on beta 10.

@jgw96
Copy link
Contributor

jgw96 commented Jul 6, 2016

Hello all, thanks for opening an issue with us! I will be closing this issue as a duplicate of #7089. Thanks for using Ionic!

@jgw96 jgw96 closed this as completed Jul 6, 2016
@barocsi
Copy link

barocsi commented Jul 19, 2016

It's even closed in #7089 wher it should not be. It also present in simple chrome browsers.
And also, why is it aligned vertically centered?

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants