FormHelper doesnt move focus properly on checkboxes

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Chwiej Pawel, Assigned: vingtetun)

Tracking

({testcase})

Trunk
Other
Linux
testcase

Details

(Whiteboard: formfill, URL)

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
User-Agent:       Opera/9.80 (Windows NT 5.1; U; pl) Presto/2.6.30 Version/10.60
Build Identifier: Mozilla/5.0 (X11; Linux armv7l; rv:2.0b4pre) Gecko/20100815  Namoroka/4.0b4pre Fennec/2.0a1pre

Maemo 5
Mozilla-central GTK fennec_2.0~a1~20100816005603_armel.deb


Reproducible: Always

Steps to Reproduce:
1. Go to http://mail.google.com/mail/signup (or use the tc)
2. Tap on the first input field (First name) and write something
3. Tap on first checkbox (Stay signed in)
4. Tap on the "next" formhelper button

Actual Results:  
Focus is moved to the second input field (Last name)


Expected Results:  
"Enable Web History" checbox should be focused
(Reporter)

Comment 1

7 years ago
Created attachment 466263 [details]
TC

Updated

7 years ago
Status: UNCONFIRMED → NEW
tracking-fennec: --- → ?
Ever confirmed: true
Keywords: testcase
Whiteboard: formfill
tracking-fennec: ? → 2.0b2+
Yeah, we're not updating the inner position of the form helper on a user interaction :s
Created attachment 474671 [details] [diff] [review]
Patch

The patch check the element on a focus change and if the elements is not the currently focused one and it is handle by the form helper we update the UI to reflect that.
Assignee: nobody → 21
Attachment #474671 - Flags: review?(mark.finkle)
Attachment #474671 - Flags: review?(mark.finkle) → review+
http://hg.mozilla.org/mobile-browser/rev/790c8e493009
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
verified FIXED on builds:
Mozilla/5.0 (maemol Linux armv7l; rv:2.0b7pre) Gecko/20101029 Firefox/4.0b8pre Fennec/4.0b2

Mozilla/5.0 (android Linux armv7l; rv:2.0b7pre) Gecko/20101029 Firefox/4.0b8pre Fennec/4.0b2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.