Tapping urlbar to show keyboard should not de-select text

VERIFIED FIXED

Status

Firefox for Android Graveyard
General
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: mbrubeck, Assigned: vingtetun)

Tracking

Details

(Whiteboard: [VKB])

Attachments

(1 obsolete attachment)

(Reporter)

Description

8 years ago
1. Tap the address bar.  Text is selected, but on-screen keyboard is hidden.
2. Tap the address bar again.

Actual results:  On-screen keyboard appears, but text is de-selected.

Expected results:  Can we change this so that the text is still selected when the on-screen keyboard appears?  (Ideally we could detect if a hardware keyboard is present, and behave differently in that case.)
Before fixing those, we should be sure we want to keep the "double tap" to open the virtual keyboard feature.
The text should be selected when the user brings up the keyboard (i.e. when he or she can type).  Users overwhelming type new urls or search terms vs. editing existing urls.
Created attachment 473555 [details] [diff] [review]
Patch v0.1

The patch keeps the text selected when doing a second touch and correct some others focus bugs on the urlbar:
 * The one mentioned by mbrubeck about the fact that you need 3 taps in landscape mode to show the VKB
 * The fact that the typed text disappears when you try to tap somewhere on the urlbar

Everything went fine on Desktop but I need to check on device (once my build is finished)
Assignee: nobody → 21
Sadly it does not behave as I want on device :(
tracking-fennec: --- → ?
Summary: Tapping urlbar to show keyboard should not de-select text → [VKB]Tapping urlbar to show keyboard should not de-select text
tracking-fennec: ? → 2.0b2+
This will be fixed by the patch in bug 596614
Summary: [VKB]Tapping urlbar to show keyboard should not de-select text → Tapping urlbar to show keyboard should not de-select text
Whiteboard: [VKB]
Was this fixed by bug 596614?
> Was this fixed by bug 596614?

Matt, can you confirm the awesome bar selection works the way you want now?
Comment on attachment 473555 [details] [diff] [review]
Patch v0.1

obsoleting patch since this was fixed by a different bug
Attachment #473555 - Attachment is obsolete: true
fixed by bug 596614
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
verified FIXED on builds:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101014 Namoroka/4.0b8pre Fennec/4.0b2pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.