567613 - Make entire urlbar edit area touchable

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: patch

Only a small area of the urlbar edit area responds to clicks/taps, and the boundaries of this area are invisible to users (see screenshot).

This patch makes the entire white edit box touchable.

(Note: This patch expects the showToolbar behavior added by bug 567044, which is currently on mobile-browser but not on mobile-1.1.)

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: Screenshot: touchable area (shaded)

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Bug 564075 caused this regression, I think, with:
http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser-ui.js#767

the "&& gFocusManager.focusedElement == this._edit.inputField" part

Bug 557619 has also affected how we handle tapping in the urlbar area.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 446939 [details] [diff] [review]
patch

I'd rather not introduce a new way (and code path) of opening the awesomebar. Let's get the previous code working again.

Comment 4

[Matt Brubeck (:mbrubeck)]

Attachment: patch v2

Updated based on comment 2.  This fixes the "onmouseup" problem noted in bug 564075 by listening for click instead of mouseup, so the action won't be triggered by a drag that didn't start in the edit area.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 447012 [details] [diff] [review]
patch v2

>       // URL textbox events
>-      case "mouseup":
>-        if (aEvent.detail < 2 && aEvent.button == 0 && gFocusManager.focusedElement == this._edit.inputField) {
>+      case "click":
>+        if (aEvent.detail < 2 && aEvent.button == 0) {
>           this.doCommand("cmd_openLocation");
>         }

This patch works well on mobile-trunk (mozilla-central) but still fails on mobile-1.1 (mozilla-192). Fails ==  regresses bug 557619.

We can drop the aEvent.detail and aEvent.button checks too. They were only used to make sure a "click" had happened.

I tested without the checks, and everything worked fine. I'll remove them for check in.

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed to m-b:
http://hg.mozilla.org/mobile-browser/rev/3988af326c73

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

This bug (not all the url bar is touchable) _does_ exist in m-1.1 too, but we would need a different patch to fix it, if we deem it blocking Fennec 1.1

We might be able to use the initial patch in this bug (or similar) if we need a quick fix.

Comment 9

[Cristian Nicolae (:xti)]

Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110922
Firefox/9.0a1 Fennec/9.0a1
Device: Samsung Galaxy S
OS: Android 2.2