Closed
Bug 715303
Opened 12 years ago
Closed 12 years ago
No focus on select when tapping on selects
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(firefox11 fixed, firefox12 fixed, fennec11+)
VERIFIED
FIXED
People
(Reporter: martijn.martijn, Assigned: wesj)
Details
(Keywords: testcase)
Attachments
(3 files)
696 bytes,
text/html
|
Details | |
513 bytes,
patch
|
mfinkle
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
1.59 KB,
text/html
|
Details |
See testcase, when tapping on the selects, a focus event should fire. This works fine in XUL Fennec and stock browser, but Fennec Native doesn't give focus to the select.
Updated•12 years ago
|
Assignee: nobody → wjohnston
Assignee | ||
Comment 1•12 years ago
|
||
Hmm... looks like we manually fired this in XUL Fennec right before sending a message to show the UI: http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/forms.js#203 I'm only doing this for select elements for now here. The form assistant gets sent taps on all sorts of elements to determine if it needs to help, so we don't want to always call focus.
Attachment #586179 -
Flags: review?(mark.finkle)
Updated•12 years ago
|
Attachment #586179 -
Flags: review?(mark.finkle) → review+
Updated•12 years ago
|
status-firefox11:
--- → affected
status-firefox12:
--- → affected
Reporter | ||
Comment 2•12 years ago
|
||
Sorry, I should have been more clear. Not only, no focus events are fired, but I don't see any of the mouse events either. See testcase, ideally, when tapping on the select, the same events as on the link should fire, namely: mousemove mousedown focus mouseup click And they also should fire in that order.
Assignee | ||
Comment 3•12 years ago
|
||
I'll have to play a bit. We actually don't want clicks to fire on the select elements, as the selected item shouldn't change when you tap on multirow selects in mobile. But maybe there's some trickery we did before to fire click with some other target.
Reporter | ||
Comment 4•12 years ago
|
||
See the patch in bug 685197 for the trickery I used in XUL Fennec, especially the css part.
Assignee | ||
Comment 5•12 years ago
|
||
filed bug 715990 for mouse events https://hg.mozilla.org/integration/mozilla-inbound/rev/309fc0271e4c
Updated•12 years ago
|
Priority: -- → P2
Updated•12 years ago
|
tracking-fennec: --- → 11+
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/309fc0271e4c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 7•12 years ago
|
||
Comment on attachment 586179 [details] [diff] [review] Patch [Approval Request Comment] Caused by the new Native code. This fix gets Fennec back to XUL Fennec compat.
Attachment #586179 -
Flags: approval-mozilla-aurora?
Comment 8•12 years ago
|
||
Comment on attachment 586179 [details] [diff] [review] Patch [Triage Comment] Mobile only - approved for Aurora.
Attachment #586179 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 10•12 years ago
|
||
Verified fixed on: Mozilla/5.0 (Android;Linux armv7l;rv:11.0a2)Gecko/20120130 Firefox/11.0a2 Fennec/11.0a2 Device: Samsung Galaxy S OS: Android 2.2 Mozilla/5.0 (Android;Linux armv7l;rv:12.0a1)Gecko/20120130 Firefox/12.0a1 Fennec/12.0a1 Device: Samsung Galaxy S OS: Android 2.2
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•