1059758 - Search results disappear and reappear on every keystroke - very distracting

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Search, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

When typing in the rocketbar search the search results (and the tooltip that shows up on first use) disappear and reappear on every single keystroke. This is highly distracting. When I already have a URL I'm trying to go to the results churning away like that just slows me down and annoys me. It would be good to put in a delay so that the results only update after say 500ms of not typing. Alternatively, the results update should be done atomically in a single layout/repaint cycle so that they just update rather than disappearing and reappearing. I'll attach a video shortly.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Actually it does seem like there's a bit of a delay in today's latest code compared to yesterday's build (which is much better, thanks!) so maybe you guys already addressed this to some extent. Still, here's the video as promised:

http://people.mozilla.org/~kgupta/tmp/rocketbar.mp4 (13MB video, too big for bugzilla attachment)

Comment 2

[Kevin Grandon :kgrandon]

We're supposed to have a delay here already, I'll look into it today and tune if necessary.

Comment 3

[Kevin Grandon :kgrandon]

We should have a 600ms delay here: https://github.com/mozilla-b2g/gaia/blob/master/apps/search/js/search.js#L12

Could the problem be that we are clearing the results after you press a character? If we are interleaving keystrokes with long delays I could see this being a problem with the results flashing.

E.g.,

w (+100ms)
w (+100ms)
w (+700ms)
<- results display ->
. <- results clear ->(+100ms)


Francis - what do you think here, should we not clear the previous results until we meet the 600ms threshold?

Comment 4

[Kevin Grandon :kgrandon]

Attachment: Pull request - keep results until next set is ready

Kats - not sure if you have cycles or care to give this a run through, but I'm proposing that we keep the existing set of results until we have something to display. It keeps the screen in a more steady state while your typing.

Feel free to chime in if you'd like to, but if you don't have time that's fine too. Thanks for filing this!

Comment 5

[Kartikaya Gupta (email:[e-mail])]

I tried it out and it definitely feels like an improvement. Thanks!

Comment 6

[Francis Djabri [:djabber]]

> 
> Francis - what do you think here, should we not clear the previous results
> until we meet the 600ms threshold?

As discussed, let's try it with 500ms

Comment 7

[Kevin Grandon :kgrandon]

Comment on attachment 8481441 [details] [review]
Pull request - keep results until next set is ready

This patch is fairly simple and since both kats and djabber have had a look at it I feel comfortable with r=me.

Comment 8

[Kevin Grandon :kgrandon]

Master: https://github.com/mozilla-b2g/gaia/commit/47d8f63dd7d2bbbbe1a7962c0caa32468767e007

Comment 9

[Chris Lord [:cwiiis]]

Comment on attachment 8481441 [details] [review]
Pull request - keep results until next set is ready

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): This bug
[User impact] if declined: Terrible experience when using search
[Testing completed]: Been on master for some time
[Risk to taking this patch] (and alternatives if risky): Low risk
[String changes made]: None

Comment 10

[Kevin Grandon :kgrandon]

Comment on attachment 8481441 [details] [review]
Pull request - keep results until next set is ready

It seems like this change was already uplifted to v2.1. It does seem like we could make this better though. We may want to do this work in bug 1027381.

https://github.com/mozilla-b2g/gaia/blob/v2.1/apps/search/js/search.js#L145

Comment 11

[Annabelle]

Attachment: verified_2.2.3gp

This bug has been verified as pass on latest build of Flame v2.2 by STR in comment 0.
Actual result:When typing in the rocketbar search ,the search results will not disappear and reappear on every single keystroke.The result will display  in some milliseconds after there is not taping in  rocketbar.
See attachment:verified_2.2.3gp
 
Device: Flame 2.2 build(pass)
Build ID               20150701162502
Gaia Revision          bd386f346eb1591fddbc84bf034b22700e7e2a58
Gaia Date              2015-06-30 15:53:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/f16c1125b9d6
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150701.195343
Firmware Date          Wed Jul  1 19:53:55 EDT 2015
Bootloader             L1TC000118D0