New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 718037 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Chrome
Pri: 2
Type: Bug

Blocked on:
issue 733710



Sign in to add a comment

Add "clear all" button to omnibox

Project Member Reported by girard@chromium.org, May 3 2017

Issue description

Touch 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")
 
Cc: omrilio@chromium.org rpop@chromium.org bettes@chromium.org est...@chromium.org
Components: UI>Browser>Omnibox
Labels: OS-Chrome OS-Windows
TBD: We could present this all the time, or only when the user is interacting with touch (via an on-screen-keyboard)
drive-by: the text in the omnibox is commonly a URL.  A tool-tip of "Clear search text" seems inappropriate.  Perhaps "Clear text"?
@1: Please do this only with OSK, not all the time.

Comment 4 by rpop@chromium.org, May 5 2017

Cc: srahim@chromium.org
+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.

Comment 5 by srahim@chromium.org, May 11 2017

"Clear text" or "Clear address bar" for the tooltip. 

BTW, another use case could be for the screen reader user.

Comment 6 by est...@chromium.org, 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.

Comment 7 by est...@chromium.org, May 30 2017

Owner: est...@chromium.org
Status: Started (was: Available)
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.
L8zK1pNROgn.png
12.7 KB View Download

Comment 8 by rpop@chromium.org, May 30 2017

Cc: hwi@chromium.org
Thanks evan. That looks good to me. +hwi to review.

Comment 9 by est...@chromium.org, 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.
WmJMP89B4fC.png
34.7 KB View Download
Owner: hwi@chromium.org
Status: Assigned (was: Started)
Hwi, please assign back to me after review, thanks.

Comment 12 by hwi@chromium.org, 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)
02 suggest-append-on-touch-spec.png
31.3 KB View Download

Comment 13 by hwi@chromium.org, Jun 5 2017

Owner: est...@chromium.org
Status: Fixed (was: Assigned)
Cc: rbasuvula@chromium.org
Labels: Needs-Feedback
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.



718037.PNG
318 KB View Download
Cc: pkasting@chromium.org bsep@chromium.org
Status: Assigned (was: Fixed)
+bsep, pkasting, does keyboard_util:: work on Windows?

rbasuvula, does this work for you on chrome os?
Cc: gene@chromium.org
+gene who said he was looking into #17

Comment 19 by bsep@chromium.org, Jun 13 2017

Cc: robliao@chromium.org
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.
Blockedon: 733710
Labels: -Needs-Feedback
If this is Pri-2, it probably should have a milestone.  Where does it fit on the roadmap?
Labels: M-62
Cc: -gene@chromium.org girard@chromium.org
Labels: -M-62 M-61
Status: Fixed (was: Assigned)
Verified that this works in Windows (tablet mode) in 62.0.3188.0

Sign in to add a comment