Closed Bug 925722 Opened 7 years ago Closed 7 years ago
Search suggestions require an extra tap to hit
At some point during our start page redesign, it looks like we changed the behaviour of our search suggestions such that now it requires two taps to trigger instead of just one. We should fix that. Expected: I type a few letters, and can tap on a search suggest tile and instantly trigger a search Actual: I need to tap twice now: once to dismiss the keyboard, and once to trigger the search.
Didn't even notice! Yeah, reproducible.
I can't repro on my Galaxy Nexus, 4.2.2, FF 27 (Nightly).
This seems fixed for me as well (10/15). Ian, are you still able to reproduce?
Still repros for me using a 10/15 nightly. First tap hides the keyboard. Second does the suggestion.
Yep, still able to reproduce this as well. Nexus 4, Android 4.3, Nightly 10/16
Assignee: nobody → lucasr.at.mozilla
tracking-fennec: ? → 26+
Comment on attachment 823428 [details] [diff] [review] Handle touch events in HomeSearchListView with onTouchEvent (r=sriram) Apparently, touch event interception + requestFocus() don't go so well on ListViews.
Attachment #823428 - Flags: review?(sriram)
Attachment #823428 - Flags: review?(sriram) → review+
Comment on attachment 823428 [details] [diff] [review] Handle touch events in HomeSearchListView with onTouchEvent (r=sriram) [Approval Request Comment] Bug caused by (feature/regressing bug #): new about:home (bug 862793) User impact if declined: extra tap needed to trigger a search suggestions while the virtual keyboard is on screen. Testing completed (on m-c, etc.): local testing for now, let's bake this in Nightly for a few days and then uplift it. Risk to taking this patch (and alternatives if risky): low, just changes the way to capture touch events to dismiss the virtual keyboard while showing awesomescreen search results. String or IDL/UUID changes made by this patch: n/a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 28
Verified fixed on: Build: Firefox for Android 27.0a2 (2013-12-06) Device: Samsung Galaxy Nexus OS: Android 4.2.1
You need to log in before you can comment on or make changes to this bug.