696300 - Can't focus on text fields when switching to a new tab

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Doug Turner (:dougt)]

Currently, you can not focus on text fields unless the page focus the element for you.  This makes testing very hard.

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

Comment 2

[Doug Turner (:dougt)]

http://hg.mozilla.org/projects/birch/rev/beed650abc13

Comment 3

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

Comment on attachment 568582 [details] [diff] [review]
patch v.1

I don't think this is exactly what we want

Comment 4

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

Attachment: patch 2

This patch backs out the event handler changes and moves the browser.focus calls to better spots (when the page loads and when we switch tabs).

The event handler problems are due to the kinetic panning code itself. We need to do a better job of stopping event propagation. This patch removes the event code that prevents tapping on editboxes from displaying the keyboard. It also now allows tapping on the background to hide the leyboard if it's already open.

The _problem_ with this patch is we will select text when we run out of scrollable space. This is a problem and will need to be fixed, but it's less of an issue than the current situation.

Comment 5

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

Opened bug 696330 for the remaining issues

Comment 6

[Camelia Urian]

Build ID: Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111115 Firefox/11.0a1 Fennec/11.0a1 - Native Fennec build
Device: HTC Desire Z - Android 2.3

Verified on several sites: bugzilla.mozilla.org, google.com, yahoo.com, facebook.com, issue did not reproduce.