Closed
Bug 594918
Opened 14 years ago
Closed 14 years ago
dismiss vkb as a user starts to pan the awesomelist
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec2.0b2+)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0b2+ | --- |
People
(Reporter: madhava, Assigned: vingtetun)
References
Details
(Whiteboard: [VKB])
Attachments
(1 file, 1 obsolete file)
1.73 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
In current builds, when the user taps on the titlebar to bring up the awesomescreen, the url field is focused and the vkb comes up. Right now, you have to tap on something (which blurs the input field) to dismiss the keyboard. Instead, the keyboard should go away as soon as the user starts to pan or flick the awesomelist. By the time he or she is panning through the list, he or she is clearly interested in seeing more results. Tapping on the field again should bring the keyboard back up.
Reporter | ||
Updated•14 years ago
|
blocking2.0: --- → ?
tracking-fennec: --- → ?
Comment 1•14 years ago
|
||
If we blur the text field, we would lose any context like selected text or caret position. I assume that is OK.
Reporter | ||
Comment 2•14 years ago
|
||
Is there a way to not lose those things? A good example to follow is the search mode of the Android Market - they seem to preserve carat position. In fact, the field remains (at least visually) focused.
Updated•14 years ago
|
blocking2.0: ? → ---
Comment 3•14 years ago
|
||
As far as I know this is the _only_ situation we want to hide the vkbd when panning a list. Correct?
tracking-fennec: ? → 2.0b2+
Updated•14 years ago
|
Summary: dismiss vkb as a user starts to pan the awesomelist → [VKB]dismiss vkb as a user starts to pan the awesomelist
Comment 4•14 years ago
|
||
I think that's right. It's certainly not the case that when the user has focused a text field or list in content that panning should dismiss the VKD (see bug 592330 comment 5)
Updated•14 years ago
|
Assignee: nobody → 21
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #3) > As far as I know this is the _only_ situation we want to hide the vkbd when > panning a list. Correct? I think we also want to dismiss it while panning any of the awesome screen once bug 596614 will land. With the patch from there we will be able to have the VKB without changing the current awesomescreen, so it will be possible to have a VKB in the bookmarks awesome panel for example. What do you guys think?
Comment 6•14 years ago
|
||
Vivien - did bug 596614 fix this bug too?
Assignee | ||
Comment 7•14 years ago
|
||
(In reply to comment #6) > Vivien - did bug 596614 fix this bug too? No it won't but this one is easy to fix. I think we juts need to add a PanBegin event listener on the list and blur the focused element when it starts
Assignee | ||
Comment 8•14 years ago
|
||
Attachment #482204 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 9•14 years ago
|
||
Comment on attachment 482204 [details] [diff] [review] Patch (In reply to comment #8) > Created attachment 482204 [details] [diff] [review] > Patch hmm, this one has some bad side effects, unrequesting review
Attachment #482204 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 10•14 years ago
|
||
Comment on attachment 482204 [details] [diff] [review] Patch Re-requesting review since the side effects came from one of my other patch
Attachment #482204 -
Flags: review?(mark.finkle)
Comment 11•14 years ago
|
||
Comment on attachment 482204 [details] [diff] [review] Patch A comment saying that setting this._edit.readOnly = true, will close the awesomebar might be helpful here too.
Attachment #482204 -
Flags: review?(mark.finkle) → review+
Updated•14 years ago
|
Summary: [VKB]dismiss vkb as a user starts to pan the awesomelist → dismiss vkb as a user starts to pan the awesomelist
Whiteboard: [VKB]
Comment 12•14 years ago
|
||
ready to land?
Assignee | ||
Comment 13•14 years ago
|
||
This one is rebased on top of bug 609619
Attachment #482204 -
Attachment is obsolete: true
Attachment #483132 -
Flags: review?(mark.finkle)
Updated•14 years ago
|
Attachment #483132 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 14•14 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/a204590a2af8
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 15•14 years ago
|
||
verified FIXED on builds (nexus one and vibrant): Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101015 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.
Description
•