Add "clear all" button to omnibox |
||||||||||||||||
Issue descriptionTouch users on desktop have a hard time removing the text from the omnibox. Add a button to clear text from the omnibox. 1. Appears at the right edge of the omnibox (as the leftmost element if there are multiple decorators) 2. Only appears when there is text in the omnibox. 3. tool-tip text TBD (suggestion: "Clear search text")
,
May 3 2017
drive-by: the text in the omnibox is commonly a URL. A tool-tip of "Clear search text" seems inappropriate. Perhaps "Clear text"?
,
May 4 2017
@1: Please do this only with OSK, not all the time.
,
May 5 2017
+shimi for help with tooltip string. I think "Clear text" is a reasonable first proposal. This button will be an 'x', right? Bettes will need to tell us which version to use to fit in with Harmony.
,
May 11 2017
"Clear text" or "Clear address bar" for the tooltip. BTW, another use case could be for the screen reader user.
,
May 11 2017
re: the premise of this bug, why do touch users have a hard time removing text? One tap selects all and brings up the OSK. Then you can begin typing.
,
May 30 2017
so here is my proposal, reflected in WIP (but mostly finished) CL here: https://chromium-review.googlesource.com/517915 Show the close button when a) the virtual keyboard is visible, AND b) input is in progress in the omnibox. Currently we hide other location bar decorations (star, zoom icon, etc.) when input is in progress. This will be the reverse: only *show* the clear button when input is in progress. This means you won't see it on first tap, when the text all gets highlighted anyway and typing will clear it, and you won't see it in addition to any other decorations (except sometimes the keyword hint view that says "Press to search..."). What asset are we using for this icon? Screenshot attached using the same X icon that we use in a few other places like the find bar.
,
May 30 2017
Thanks evan. That looks good to me. +hwi to review.
,
May 30 2017
And here's what it looks like with the keyword hint, which I think is a bit off. It makes it look like the [x] applies to the search bubble somehow.
,
May 30 2017
,
Jun 5 2017
Hwi, please assign back to me after review, thanks.
,
Jun 5 2017
- See the attached for spec - X from tab strip looks better to me. Let me know if this needs a separate set of svgs. - Label idea: "Clear input" (we use this for clank a11y label)
,
Jun 5 2017
,
Jun 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cadfcdcc9c336034eb55128765768d393d443a70 commit cadfcdcc9c336034eb55128765768d393d443a70 Author: Evan Stade <estade@chromium.org> Date: Thu Jun 08 01:59:07 2017 Show a clear all button in the omnibox when editing with virtual keyboard. Bug: 718037 Change-Id: I9b9761919b9211fa4b2605c3317bf6de5ad668eb Reviewed-on: https://chromium-review.googlesource.com/517915 Commit-Queue: Evan Stade <estade@chromium.org> Reviewed-by: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#477858} [modify] https://crrev.com/cadfcdcc9c336034eb55128765768d393d443a70/chrome/app/generated_resources.grd [modify] https://crrev.com/cadfcdcc9c336034eb55128765768d393d443a70/chrome/browser/ui/views/location_bar/keyword_hint_view.cc [modify] https://crrev.com/cadfcdcc9c336034eb55128765768d393d443a70/chrome/browser/ui/views/location_bar/keyword_hint_view.h [modify] https://crrev.com/cadfcdcc9c336034eb55128765768d393d443a70/chrome/browser/ui/views/location_bar/location_bar_view.cc [modify] https://crrev.com/cadfcdcc9c336034eb55128765768d393d443a70/chrome/browser/ui/views/location_bar/location_bar_view.h
,
Jun 8 2017
,
Jun 13 2017
Tested the issue on Windows-10 using chrome latest Dev M61-61.0.3128.0 by following steps mentioned in the original comment. Steps Followed: Scenario-1 : Install chrome and change windows to tablet mode and enter the text in omnibox with keyboard. Scenario-2 : Install chrome and change windows to tablet mode and connect external keyboard to laptop and enter the text in omnibox. Scenario-3 : Install chrome and change windows to tablet mode and enter the text in omnibox with on screen keyboard. Scenario-4 : Install chrome & Virtual Keyboard in windows tablet mode and enter the text in omnibox Please find the screen shot for reference. @estade: Not able to find the clear all icon in omnibox. Could you please help us to provide the test steps of the issue which would help us to verify the issue. Thanks in Advance.
,
Jun 13 2017
+bsep, pkasting, does keyboard_util:: work on Windows? rbasuvula, does this work for you on chrome os?
,
Jun 13 2017
+gene who said he was looking into #17
,
Jun 13 2017
I'm not sure. I can't find a reason it wouldn't work just by looking at it. But I seem to recall virtual keyboard detection is weird (related to bug 491516?). Rob might know more.
,
Jun 15 2017
,
Jun 28 2017
If this is Pri-2, it probably should have a milestone. Where does it fit on the roadmap?
,
Jun 28 2017
,
Jul 5 2017
,
Aug 17 2017
Verified that this works in Windows (tablet mode) in 62.0.3188.0 |
||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||
Comment 1 by girard@chromium.org
, May 3 2017Components: UI>Browser>Omnibox
Labels: OS-Chrome OS-Windows