Closed Bug 640216 Opened 9 years ago Closed 9 years ago

Hide navigation arrows of the form assistant if there is previous/next are both disabled

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: vingtetun, Assigned: vingtetun)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
The attached patch fix that and also add some changes to the CSS to be consistent between menulist, context menu and <select/> styles
Attachment #518094 - Flags: review?(mark.finkle)
Attachment #518094 - Flags: review?(mark.finkle) → review+
Comment on attachment 518094 [details] [diff] [review]
Patch

This fixes at least one crummy situation where the next/prev buttons stay active cause the single form widget stays focused and the disabled buttons stay visible.
Attachment #518094 - Flags: approval2.0+
http://hg.mozilla.org/mobile-browser/rev/e5ee75799b51
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Could you please provide some steps to reproduce so that I could validate this bug?
verified:
Mozilla/5.0 (Android; Linux armv71; rv2.0b13pre) Gecko/20110317 Firefox/4.0b13pre Fennec/4.0b6pre
Device: Droid 2 
OS: Android 2.2

Mozilla/5.0 (Maemo; Linux armv71; rv:2.0b13pre) Gecko/20110317 Firefox/4.0b13pre Fennec/4.0b6pre
Device: Maemo N900

Steps to Reproduce:
1. goto http://people.mozilla.com/~nhirata/html_tp/textarea.html
2. click in the text area
3. verify that the form assistant's arrows do not appear.
Status: RESOLVED → VERIFIED
bugspam
Assignee: nobody → 21
You need to log in before you can comment on or make changes to this bug.