Android Content Menu overlays Find in Page/Form Filler but still allows typing

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: tchung, Assigned: mbrubeck)

Tracking

({polish})

Trunk
ARM
Android
polish

Details

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
Created attachment 489548 [details]
Overlay screenshot

When the find in page menu is open, and clicking the android context menu will overlay the find in page.  That's fine, but it seems strange that i can continue to type characters in the keyboard but not see what im' typing because of the overlay.

* See screenshot (cant' see the Find in Page, so not sure this helps much)

Repro:
1) install android nightly: Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101110 Firefox/4.0b8pre Fennec/4.0b3pre	
2) launch any website with text
3) click site identifier > Find in Page.   Soft keyboard and Find in page textbox appears.  cursor should be in focus
4) now click the android content menu button.  options pop up above keyboard
5) start typing in the keyboard.  Verify the content menu does not dissapear when typing, but if you hide it, then you'll see your find text.
6) repeat the above with a website that launches form filler.  same result

Expected:
- once i start typing, the content menu should go away so i can see what im typing in Find Page/Form Filler

Actual:
- blind typing
This popup should eat "keypress" events like context menus do
I think the VKB should hide, it does not make sense to have it there since the menu if for going to an other place
For comparison, when in this situation in Browser, the menu just doesn't come up.  The menu button does nothing.
(Reporter)

Comment 4

7 years ago
flagging for blocking
tracking-fennec: --- → ?
Assignee: nobody → mbrubeck
tracking-fennec: ? → 2.0+
(Assignee)

Comment 5

7 years ago
Created attachment 494520 [details] [diff] [review]
patch

This patch just hides the app menu when you press a key, which seemed like the most useful of the alternatives I tried.
Attachment #494520 - Flags: review?(mark.finkle)
(Assignee)

Updated

7 years ago
Keywords: polish
Comment on attachment 494520 [details] [diff] [review]
patch

OK. This seems reasonable and is most like the normal Android behavior.
Attachment #494520 - Flags: review?(mark.finkle) → review+
Whiteboard: [fennec-checkin-postb3]
pushed:
http://hg.mozilla.org/mobile-browser/rev/4f250d6b6415
Whiteboard: [fennec-checkin-postb3]
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 8

7 years ago
VERIFIED FIXED on:

Mozilla /5.0 (Android;Linux armv7l;rv:7.0a1) Gecko/20110602 Firefox/7.0a1 Fennec/7.0a1 

Mozilla /5.0 (Android;Linux armv7l;rv:6.0a2) Gecko/20110602 Firefox/6.0a2 Fennec/6.0a2 

Mozilla /5.0 (Android;Linux armv7l;rv:5.0) Gecko/20110527 Firefox/5.0 Fennec/5.0

Device: HTC Desire Z (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.