Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Hide Virtual Keyboard when bookmarks list is opened

VERIFIED FIXED in Firefox 20



Firefox for Android
5 years ago
a year ago


(Reporter:, Assigned: wesj)


16 Branch
Firefox 20

Firefox Tracking Flags

(firefox19- affected, firefox20 verified, firefox21 verified, relnote-firefox 20+, fennec20+)



(2 attachments, 1 obsolete attachment)



5 years ago
Created attachment 678128 [details]

User Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:16.0) Gecko/20100101 Firefox/16.0
Build ID: 20121024073032

Steps to reproduce:

- open firefox
- rotate device to landscape mode
- tap url bar to open awesomescreen
- tap bookmarks

Actual results:

- Virtual Keyboard is shown, so I cannot see my bookmarks. Have to hide Virtual Keyboard myself.

Expected results:

- Virtual Keyboard should be hidden so that I can see my bookmarks

Comment 1

5 years ago
Device: HTC Sensation
Android Version: 4.0.3
Ideally the URL bar should lose focus. It seems to do so when you switch between Top Sites and Bookmarks back and forth, so I wonder why the first time the focus is kept.

CC :ibarlow, :mfinkle for input here
Ever confirmed: true
OS: Linux → Android
Hardware: x86 → ARM

Comment 3

5 years ago
This used to work. Its a regression.
Keywords: regressionwindow-wanted
tracking-fennec: --- → ?
Keywords: regression
Yeah, some (recent?) change broke focus handling in Awesome Screen. It would probably help to know when this happened.

Comment 6

5 years ago
I noticed it when I upgraded form FF15 to 16 via google play.
Assignee: nobody → wjohnston
tracking-fennec: ? → 18+
Possible dupe of 770170?
Looks like it
Last Resolved: 5 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 770170
Can someone carry over the tracking 18+ to bug 770170.
Keywords: regressionwindow-wanted

Comment 10

5 years ago
Created attachment 682180 [details] [diff] [review]

This fixes two bugs (switching screens and scrolling the list).
Attachment #682180 - Flags: review?(bnicholson)
Comment on attachment 682180 [details] [diff] [review]

>-                if (event.getAction() == MotionEvent.ACTION_DOWN)
>+                if ((event.getAction() & MotionEvent.ACTION_MASK) == MotionEvent.ACTION_DOWN)

It'd probably be cleaner to use MotionEvent.getActionMasked() to apply the action mask.
Attachment #682180 - Flags: review?(bnicholson) → review+
Resolution: DUPLICATE → ---
Duplicate of this bug: 770170
tracking-fennec: 18+ → 20+

Comment 13

5 years ago

Comment 14

5 years ago
I'm hoping this was the cause of robocop failures. backed out:
Keywords: regression


5 years ago
Duplicate of this bug: 813018
Was it?
Also, did you intentionally omit the change suggested in comment 11?


5 years ago
tracking-firefox19: --- → ?
tracking-firefox19: ? → +

Comment 18

5 years ago
Created attachment 691097 [details] [diff] [review]

Fix the test failures. I apparently added tests to make sure we didn't fix this at one point.

I also put in a fix for finding the bookmarks button in the menu on ICS+ devices (where it no longer has a text label so clickText("Bookmark") doesn't work anymore. Maybe we should move that somewhere more general?
Attachment #691097 - Flags: review?(gbrown)

Comment 19

5 years ago
Looks mostly good on try. Waiting for a retrigger.

Comment 20

5 years ago
Comment on attachment 691097 [details] [diff] [review]

I may have added that vkb code when the tests started failing -- oops! Glad we are on the same page now. 

Try run looks solid.
Attachment #691097 - Flags: review?(gbrown) → review+

Comment 22

5 years ago
Pushed again. Fingers crossed. I'll request Aurora approval once they're green.

Comment 23

5 years ago
Comment on attachment 682180 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 759041
User impact if declined: impossible to hide the keyboard and see the whole awesomescreen
Testing completed (on m-c, etc.): Landed on mc 12-11
Risk to taking this patch (and alternatives if risky): This is pretty low risk. Hooking up some function I accidentally unhooked.
String or UUID changes made by this patch:  None.
Attachment #682180 - Flags: approval-mozilla-aurora?

Comment 24

5 years ago
Comment on attachment 691097 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Same
User impact if declined: Non. Fixes tests
Testing completed (on m-c, etc.): Landed on mc 12-11. Tests now pass
Risk to taking this patch (and alternatives if risky): Low risk. Test only.
String or UUID changes made by this patch: none.
Attachment #691097 - Flags: approval-mozilla-aurora?
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20


5 years ago
Attachment #682180 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+


5 years ago
Attachment #691097 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 26

5 years ago
(reminder to set fixed on the channel)
status-firefox19: --- → fixed
status-firefox20: --- → fixed
(In reply to Wesley Johnston (:wesj) from comment #26)

This broke robocop on aurora:

Aurora backout:
status-firefox19: fixed → affected
Wesley - can you take another look at landing this on Aurora?

Comment 30

5 years ago
Missed one test change when unbitrotting last time. Pushed again:
Still broke robocop. Also, remember that Aurora isn't inbound. You are expected to watch your pushes and back them out if they break things.

0 INFO SimpleTest START
1 INFO TEST-START | testBookmark
2 INFO TEST-PASS | testBookmark | checking that bookmarks list exists - bookmarks list exists
3 INFO TEST-PASS | testBookmark | bookmarks list has 4 children (the default bookmarks) - 4 should equal 4
Exception caught during test!
	at org.mozilla.fennec_aurora.tests.testBookmark.runAwesomeScreenTest(
	at org.mozilla.fennec_aurora.tests.testBookmark.testBookmark(
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(
	at android.test.InstrumentationTestCase.runMethod(
	at android.test.InstrumentationTestCase.runTest(
	at android.test.ActivityInstrumentationTestCase2.runTest(
	at org.mozilla.fennec_aurora.tests.BaseTest.runTest(
	at junit.framework.TestCase.runBare(
	at junit.framework.TestResult$1.protect(
	at junit.framework.TestResult.runProtected(
	at android.test.AndroidTestRunner.runTest(
	at android.test.AndroidTestRunner.runTest(
	at android.test.InstrumentationTestRunner.onStart(
4 INFO TEST-UNEXPECTED-FAIL | testBookmark | Exception caught - java.lang.NullPointerException
5 INFO TEST-END | testBookmark | finished in 42833ms
6 INFO TEST-START | Shutdown
7 INFO Passed: 2
8 INFO Failed: 1
9 INFO Todo: 0
Attachment #682180 - Attachment is obsolete: true
Attachment #682180 - Flags: approval-mozilla-aurora+
Attachment #691097 - Flags: approval-mozilla-aurora+
I have a fix for the testBookmark problem coming soon in bug 769919.

Comment 33

5 years ago
This slipped my mind. I fixed up geoff's patch and pushed it with these to try (based on top of beta):
Virtual keyboard is hidden when accessing the Bookmarks tab either by tapping on the Tab title or by swipe on Aurora 20.0a2 2013-01-10 on the HTC Desire Z (Android 2.3.3) marking as verified on FF 20 but leaving th bug open for the Robocop test fixes
status-firefox20: fixed → verified
Given comment 6, we won't track for release. Feel free to nominate for uplift if you believe the fix is low risk.
tracking-firefox19: + → -
This issue doesn't occur anymore on the latest Nightly. Closing bug as verified fixed on:

Firefox for Android
Version: 21.0a1 (2013-01-29)
Device: Galaxy R
OS: Android 2.3.4
status-firefox21: --- → verified


5 years ago
relnote-firefox: --- → ?
relnote-firefox: ? → 20+
You need to log in before you can comment on or make changes to this bug.