Closed Bug 591313 Opened 10 years ago Closed 10 years ago

Search engine list not scrollable

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Maemo
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ahoza, Assigned: fabrice)

Details

Attachments

(1 file)

BuildID: Mozilla/5.0 (Android; Linux armv7l; rv:2.0b5pre) Gecko/20100827
Namoroka/4.0b5pre Fennec/2.0b1pre
Device: Nokia N900

Steps to reproduce:

1. Start fennec
2. Go to addons.mozilla.org, click on the favicon to the left of the URL bar.
In the identity panel that appears, click on the "Add Search ..." option.
3. Go to bugzilla.mozilla.org, click on the favicon to the left of the URL bar.
In the identity panel that appears, click on the "Add Search ..." option.
4. Tap in URL bar and open search engine list

Expected results:
All enabled search engines are visible in the list; it list is to large than it should scroll

Actual results:
When list is too long it cannot be scrolled.
Attached patch fixSplinter Review
We were missing a flex on the scrollable richlistbox.
Assignee: nobody → fabrice
Attachment #469850 - Flags: review?(mark.finkle)
Comment on attachment 469850 [details] [diff] [review]
fix

Arrrgh. Bug 590476 regressed this code. We _did_ have a flex in there.
Attachment #469850 - Flags: review?(mark.finkle) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/6e5189f9d6f0
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
verified FIXED On builds:
Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b5pre) Gecko/20100830 Namoroka/4.0b5pre Fennec/2.0a1pre

and

Mozilla/5.0 (Android; Linux armv71; Nokia N900; en-US; rv:2.0b5pre) Gecko/20100830 Namoroka/4.0b5pre Fennec/2.0a1pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.