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.

Sign up for GitHub

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

Jump to bottom

bug: Memory leaks between views #10416

Closed
adamgalmor opened this issue Feb 13, 2017 · 5 comments
Closed

bug: Memory leaks between views #10416

adamgalmor opened this issue Feb 13, 2017 · 5 comments
Assignees
@manucorporat

Comments

@adamgalmor
Copy link

adamgalmor commented Feb 13, 2017 •
edited
Loading

Type: bug

Ionic Version: 2.x

Platform: all

I noticed a memory leak when I'm moving between the views of my app.
So I've created a clean Ionic 2 temple tab project ("ionic start --v2 myApp tabs") without any additional code and noticed it to leaks when I switch between tabs:

image

I saw others posted related topics but without a solution:

We are at the final stage before releasing our enterprise application and we will appreciate your quick response.

P.S
This is my relate ionic forum post URL:
https://forum.ionicframework.com/t/memory-leaks-between-views/79322
@jgw96
Copy link
Contributor

jgw96 commented Feb 13, 2017

Thanks for opening an issue with us, we will take a look into this.

@jgw96 jgw96 added the v2 label Feb 13, 2017
@adamgalmor
Copy link
Author

adamgalmor commented Feb 14, 2017 •
edited
Loading

FYI
This is an allocation timelines snapshot of my app (on my Android device) after pushing and poping a view from the navController:

image

I also noticed the aforementioned memory leak between views inst happening on my iOS device.

@manucorporat manucorporat self-assigned this Feb 23, 2017
@manucorporat
Copy link
Contributor

Working in this right now!

@manucorporat
Copy link
Contributor

I have a fix for this memory leak!

manucorporat added a commit to manucorporat/ionic that referenced this issue Feb 26, 2017
@manucorporat
fix(all): memory leaks
a994adb 
fixes ionic-team#10459
fixes ionic-team#10416
fixes ionic-team#10286
@manucorporat manucorporat mentioned this issue Feb 26, 2017
Closed
@ionitron-bot
Copy link

ionitron-bot bot commented Sep 3, 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 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@manucorporat manucorporat

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Post split panel manucorporat/ionic
3 participants
@manucorporat @adamgalmor @jgw96