649000 - Drop down options not present for input.mozilla.org mobile version

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Ioana Chiorean]

Attachment: input

Build id : Mozilla/5.0 (Android;Linux armv7l;rv:2.2a1pre)Gecko/20110411
Firefox/4.2a1pre Fennec /4.1a1pre
Device: HTC Desire and Google Nexus S
OS: Android 2.3 and Android 2.2

Steps to reproduce:
1. Go to www.input.mozilla.org
2. Go to Drop down button from left upper part of site. (default Firefox and 4.0)

Actual result:
- option are available by tapping the arrow of the drop down

Expected result: 
- the drop down is not functional, please see attache screen

Comment 1

[Aakash Desai [:aakashd]]

This is working fine for me on the stock browser.

Comment 2

[Martijn Wargers (dead)]

Attachment: testcase

The problem seems to be that <select>'s with opacity:0 don't seem to be triggered by the formhelper.

A quick look at mxr seems to give this to be the likely culprit:
http://mxr.mozilla.org/mozilla-central/source/mobile/chrome/content/forms.js#580

Comment 3

[Martijn Wargers (dead)]

Attachment: testcase

Comment 4

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

(In reply to comment #2)
> Created attachment 526760 [details]
> testcase
> 
> The problem seems to be that <select>'s with opacity:0 don't seem to be
> triggered by the formhelper.
> 
> A quick look at mxr seems to give this to be the likely culprit:
> http://mxr.mozilla.org/mozilla-central/source/mobile/chrome/content/forms.js#580

You sounds right.

Comment 5

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

Attachment: Patch

I need to test the patch to ensure everything is fine but I can't do it right now, I'm recompiling and this take forever.

Comment 6

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

(In reply to comment #5)
> Created attachment 527143 [details] [diff] [review]
> Patch
> 
> I need to test the patch to ensure everything is fine but I can't do it right
> now, I'm recompiling and this take forever.

The patch works fine.

Comment 7

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

Comment on attachment 527143 [details] [diff] [review]
Patch

Can we add a test case for this too?

Comment 9

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

pushed:
http://hg.mozilla.org/mozilla-central/rev/4c978d7f77e8

Comment 10

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Mozilla/5.0 (Android; Linux armv71; rv6.0a1) Gecko/20110506 Firefox/6.0a1 Fennec/6.0a1
Device: Droid 2 
OS: Android 2.2

Mozilla/5.0 (Android; Linux armv71; rv6.0a1) Gecko/20110506 Firefox/6.0a1 Fennec/6.0a1
Device: Thunderbolt
OS: Android 2.2

reported bug 655384 for the trivial down arrow not showing.