Closed
Bug 964833
Opened 11 years ago
Closed 11 years ago
Entering the edit mode in the Dialer call log is slow
Categories
(Firefox OS Graveyard :: Gaia::Dialer, defect, P2)
Firefox OS Graveyard
Gaia::Dialer
Tracking
(Not tracked)
RESOLVED
FIXED
1.4 S1 (14feb)
People
(Reporter: vingtetun, Assigned: rik)
References
Details
(Keywords: perf, Whiteboard: [c=effect p= s=2014.02.14 u=])
Attachments
(1 file)
The call log edit mode is slow like a slug. We are doing a few reflows and we try to do one animation per row. It's not visible with a few items but on my phone with a full call log i can never see any of the animations and it takes forever. My feeling for now is that we should first remove the animation to save a lot of times, and then someone needs to try to reland some animations that works with 200 rows. So this patch is about removing all animations when entering edit mode for the moment. Then a followup can be opened once this patch lands to add back usable animations.
Comment 1•11 years ago
|
||
Hey Vivien, how slow is slow like a slog? Do you have any information on how much reflow is occuring? Thanks!
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to Mason Chang [:mchang] from comment #1) > Hey Vivien, how slow is slow like a slog? Do you have any information on how > much reflow is occuring? Thanks! I don't have a measure handy. But if you do |make reference-workload-medium| in the $GAIA folder and then open the call log, you should see how fast it is. Now if you compare it with the email app which is not doing any animations this is way much faster.
Flags: needinfo?(21)
Assignee | ||
Comment 3•11 years ago
|
||
I'm removing all the animations because we couldn't see them in the first place.
Attachment #8369541 -
Flags: review?(etienne)
Comment 4•11 years ago
|
||
Comment on attachment 8369541 [details] [review] https://github.com/mozilla-b2g/gaia/pull/15918 r=me The transition looked nice but even a reference-workload-light was enough to get 0 frames.
Attachment #8369541 -
Flags: review?(etienne) → review+
Assignee | ||
Comment 5•11 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/dfe33a3b6c32f242d9629220e5a90dae5b53bd50 This is a slight improvement, maybe a profile could give us more insights if needed. Let's close this for now, please open another bug if you still find it too slow.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
(In reply to Anthony Ricaud (:rik) from comment #5) > This is a slight improvement, maybe a profile could give us more insights if > needed. Let's close this for now, please open another bug if you still find > it too slow. Yes please, open a follow up and make it block bug 837633.
Updated•11 years ago
|
Whiteboard: [c=effect p= s= u=] → [c=effect p= s=2014.02.14 u=]
Target Milestone: --- → 1.4 S1 (14feb)
You need to log in
before you can comment on or make changes to this bug.
Description
•