Skip to content

Collection-repeat is not compatible with jQuery? #2723

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

Closed
xmiao opened this issue Dec 15, 2014 · 4 comments
Closed

Collection-repeat is not compatible with jQuery? #2723

xmiao opened this issue Dec 15, 2014 · 4 comments

Comments

@xmiao
Copy link

xmiao commented Dec 15, 2014

The code below takes from the ionic demo, which is about the 1000 contacts shown in the collection-repeat.
https://dl.dropboxusercontent.com/u/67612992/contacts.tgz

If there is no jQuery loaded, the performance is amazing. But with the jQuery, the scrolling stuck here and there.

index.html:
    <meta name="viewport" content="initial-scale=1, maximum-scale=1, user-scalable=no, width=device-width">
    <script src="./jquery-2.0.3.min.js"></script> <-- adding this line, the scrolling speed slowed down dramatically.
    <link rel="stylesheet" href="/lib/ionic/css/ionic.min.css">
    <script src="/lib/ionic/js/ionic.bundle.min.js"></script>
    <link rel="stylesheet" href="./contacts.css">
    <script src="./contacts.js"></script>
@mhartington
Copy link
Contributor

Try downgrading jquery to something below 2.0

angular/angular.js#6428

@xmiao
Copy link
Author

xmiao commented Dec 18, 2014

Good to know.
I also tried jQuery 1.11 before submitting this ticket, which had the same performance issue. It seems like some size calculation problem on my android phone (samsung galaxy note 3).

I can do more test when I get a chance. Currently, I removed the dependency of jQuery.

@drewrygh drewrygh added the needs: reply the issue needs a response from the user label Dec 19, 2014
@Ionitron
Copy link
Collaborator

Me close

@adamdbradley adamdbradley removed the needs: reply the issue needs a response from the user label Dec 31, 2014
@ionitron-bot
Copy link

ionitron-bot bot commented Sep 6, 2018

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 6, 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