Search suggestions do not work in mobile because of onkeyup

VERIFIED FIXED in 2012-05-31

Status

P3
normal
VERIFIED FIXED
7 years ago
6 years ago

People

(Reporter: cvan, Assigned: cvan)

Tracking

2012-05-31
Points:
---

Details

(Whiteboard: [woomp], URL)

(Assignee)

Description

7 years ago
Because `onkeyup` and `onkeypress` don't get fired in mobile, we aren't detecting when people type something into the search box. As a result the search suggestions box doesn't appear at all!
Blocks: 752013
No longer blocks: 735054
Priority: -- → P3
(Assignee)

Comment 1

7 years ago
14:32 < davidwalsh|brb> cvan: During the focus event, I'd add a listener which would use setInterval to evaluate the value of the input every, say second, and use that value to provide suggestions
14:32 < davidwalsh|brb> When the field was blurred, I'd remove the event to release the setInterval stress
...
14:40 < digitarald> +1 on interval-while-focussed always works best
...
14:41 < digitarald> somebody committed my code here, its basically this class with option periodical: true 
https://github.com/angelsk/mootools-autocompleter/blob/f81253a35d63336c955b94b916b4a9f3ab7c7d25/Source/Observer.js
(Assignee)

Updated

7 years ago
Blocks: 756282
No longer blocks: 752013
(Assignee)

Updated

7 years ago
Component: Public Pages → Consumer Pages
Product: addons.mozilla.org → Marketplace
QA Contact: web-ui → consumer-pages
Version: unspecified → 1.0
(Assignee)

Updated

7 years ago
Assignee: nobody → cvan
Whiteboard: [woomp]
Target Milestone: --- → 2012-05-31
(Assignee)

Comment 2

7 years ago
https://github.com/mozilla/zamboni/commit/79f307d
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 3

6 years ago
Verified as fixed : http://screencast.com/t/q1StLqBJg
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.