Closed
Bug 803321
Opened 12 years ago
Closed 12 years ago
Gaia homescreen app: lazily load everything.me
Categories
(Firefox OS Graveyard :: Gaia::Homescreen, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philikon, Assigned: philikon)
References
Details
(Whiteboard: [fast:>200ms][slim:>7M])
Attachments
(1 file)
Everything.me takes a while to load. This creates a particularly bad UX when the homescreen gets unloaded due to lack of memory and the user presses the home button. We should lazily load everything.me when the user visits the first page of the grid.
Assignee | ||
Comment 1•12 years ago
|
||
PR sent: https://github.com/mozilla-b2g/gaia/pull/5889
This lazily loads Everything.me when we pan to the left most page. The UX can definitely be improved here, it's not perfect. I left the default stubs ("Loading..." and something about being offline) in place for now. I'm not a good HTML/CSS guy, so if somebody else could handle the polish in a follow-up, that'd be great. Some sort of standard throbber [1] would be nice here.
Judging from my profiling, this shaves at least ~200ms off of the Homescreen app's start time.
[1] http://telefonicaid.github.com/Gaia-UI-Building-Blocks/#widgets/progress_activity/
Whiteboard: [fast:>200ms]
Comment 2•12 years ago
|
||
(In reply to Philipp von Weitershausen [:philikon] from comment #1)
> Judging from my profiling, this shaves at least ~200ms off of the Homescreen
> app's start time.
I'd actually expect it to save a lot more experience-wise in addition due to making it less likely that the homescreen will be evicted from memory - are you seeing that in your build with this patch? If so, that would be cool!
Comment 3•12 years ago
|
||
Pointer to Github pull-request
Updated•12 years ago
|
Attachment #673283 -
Flags: review?(crdlc)
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Robert Kaiser (:kairo@mozilla.com) from comment #2)
> (In reply to Philipp von Weitershausen [:philikon] from comment #1)
> > Judging from my profiling, this shaves at least ~200ms off of the Homescreen
> > app's start time.
>
> I'd actually expect it to save a lot more experience-wise in addition due to
> making it less likely that the homescreen will be evicted from memory - are
> you seeing that in your build with this patch? If so, that would be cool!
Yes, yes I am. In fact, unless I pan to the everything.me page, the Homescreen app is now at least 7 megs slimmer.
Whiteboard: [fast:>200ms] → [fast:>200ms][slim:>7M]
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 673283 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/5889
Vivien wants to do the review
Attachment #673283 -
Flags: review?(crdlc) → review?(21)
Blocks: 796172
Comment 6•12 years ago
|
||
Comment on attachment 673283 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/5889
This block the blocker of loading quickly the homescreen. This also reduce the memory usage if e.me is not used and will make less OOM for minor changes. a=me.
Attachment #673283 -
Flags: review?(21)
Attachment #673283 -
Flags: review+
Attachment #673283 -
Flags: approval-gaia-master+
Assignee | ||
Comment 7•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Updated•12 years ago
|
Component: Gaia → Gaia::Homescreen
Flags: approval-gaia-master+
Blocks: 814399
You need to log in
before you can comment on or make changes to this bug.
Description
•