switch keyboard action button from go to search when action would be a search in the awesome bar

VERIFIED FIXED

Status

()

defect
VERIFIED FIXED
8 years ago
3 years ago

People

(Reporter: blassey, Assigned: blassey)

Tracking

unspecified
ARM
Android
Points:
---
Bug Flags:
in-litmus +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound][QA+])

Attachments

(1 attachment)

No description provided.
Posted patch patchSplinter Review
Assignee: nobody → blassey.bugs
Attachment #581896 - Flags: review?(mark.finkle)
Comment on attachment 581896 [details] [diff] [review]
patch

Does this cause any flicker of the keyboard?

+            imm.restartInput(mText);

Also wondering how Swype will handle this, but testing will tell. Looks good.
Attachment #581896 - Flags: review?(mark.finkle) → review+
(In reply to Mark Finkle (:mfinkle) from comment #2)
> Comment on attachment 581896 [details] [diff] [review]
> patch
> 
> Does this cause any flicker of the keyboard?
no flicker for me, testing on a Galaxy S

pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/e9014ab86f5d
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/e9014ab86f5d
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Samsung Nexus S (Android 2.3.6)
20111216052315
http://hg.mozilla.org/mozilla-central/rev/80ac06ad280d
Status: RESOLVED → VERIFIED
Flags: in-litmus?(nhirata.bugzilla)
Whiteboard: [inbound] → [inbound][QA+]
litmus test case : 49198
Flags: in-litmus?(nhirata.bugzilla) → in-litmus+
You need to log in before you can comment on or make changes to this bug.