566304 - Cannot pan autocomplete list to left after selecting Next and Previous buttons

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[ahoza]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3
Build Identifier: 

"<" button is not working unless some information is deleted from previously pre-filled field

Reproducible: Always

Steps to Reproduce:
1. Go to bugzilla
2. With the search field, search about 15 times with different terms to get a scrollable list of autocomplete suggestions
3. Tap on the bugzilla search field again
4. Pan the list, select the right-most element in the list (or one from the middle)
5. Tap Next
6. Tap Previous
7. Pan the list to the left

Actual Results:  
List does not pan on step 7.
If user deletes information in Find field, "<" and ">" buttons in autocomplete list are working.

Expected Results:  
On both steps 4 and 7, the list should pan

Comment 1

[ahoza]

Attachment: Screen capture

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Thanks for the bug report and the screen capture!

> Expected Results:  
> On both steps 4 and 7, the list should pan

The list is not supposed to pan during step 7 because the that is a different workflow than step 4, but also the arrow should not be display in that case and that is misleading.

The attached patch correct this behavior (basically the underflow event didn't fired if the arrowscrollbox is hidden http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/scrollbox.xml#437)

Comment 3

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

The patch is low risk and just used the "collapsed" property instead of "hidden"

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed to m-c:
http://hg.mozilla.org/mobile-browser/rev/96ad02fc510d

low risk patch fixes a bug in a primary feature
pushed to m-1.1:
http://hg.mozilla.org/releases/mobile-1.1/rev/92c5158df3ed

Comment 5

[Aakash Desai [:aakashd]]

verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.5pre) Gecko/20100519 Namoroka/3.6.5pre Fennec/1.1b2pre

and

Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:1.9.3a5pre) Gecko/20100519 Namoroka/3.7a5pre Fennec/2.0a1pre