-
Notifications
You must be signed in to change notification settings - Fork 0
Incorrect workflow #1
Copy link
Copy link
Open
Description
ttarass
opened on Dec 2, 2014
Issue body actions
- You have 2 AsyncTasks that are executed one after another synchroneously. What's the point in it? you could put all that code into 1 AsyncTask and have the same effect.
- Your code loads all article data AND all images and only THEN displays it to the user. That's incorrect. Imagine bad internet connection where text loads relatively fast and then we wait forever for the images. Please rework your code to show article texts IMMEDIATELY after they're available AND load images in background. Images should be displayed on availability. This is a common mobile development practice which is imperative for a successful projects
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels