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

Refreshing / Loading records when viewing a Metric sheet locks up UI #30

Open
Glavin001 opened this issue Aug 21, 2014 · 8 comments
Open

Refreshing / Loading records when viewing a Metric sheet locks up UI #30

Glavin001 opened this issue Aug 21, 2014 · 8 comments
Assignees
@Glavin001
Labels
bug high priority

Comments

@Glavin001
Copy link
Member

/cc @toddmurphy
@Glavin001 Glavin001 added the bug label Aug 21, 2014
@Glavin001 Glavin001 self-assigned this Aug 21, 2014
@Glavin001
Copy link
Member Author

  • Show "Loading..." for fields initially until load has completed
  • Check if value is nil or "" (empty string) and if so, set to "No description for Asset", etc

@Glavin001
Copy link
Member Author

This should help with loading Asset performance: Streamlyne/Cocoa-SDK#15
but only on the second read.. there are still many other optimizations to do!

@Glavin001
Copy link
Member Author

When loading:

image

@Glavin001
Copy link
Member Author

Another necessary issue: Streamlyne/Cocoa-SDK#17

@Glavin001
Copy link
Member Author

And Streamlyne/Cocoa-SDK#18

@Glavin001
Copy link
Member Author

This is the true issue and solution!
Using Background Fetching With Core Data: http://www.sebastianrehnby.com/blog/2013/05/20/using-background-fetching-with-core-data/

Using Time Profiler I see
image

@Glavin001
Copy link
Member Author

Another, UI Freeze with Magical Record: http://stackoverflow.com/a/13060226

@Glavin001
Copy link
Member Author

Common save patterns for use with MagicalRecord: https://gist.github.com/tonyarnold/4694673

@Glavin001 Glavin001 mentioned this issue Aug 22, 2014
Closed
1 task
Glavin001 added a commit that referenced this issue Aug 22, 2014
@Glavin001
See #30. Show “Loading…” for fields until initial load has completed.
63f70b7 
Removed all console logging to speed up app performance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Glavin001 Glavin001

Labels
bug high priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Glavin001