New issue
Advanced search Search tips

Issue 688016 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Info message above keyboard overlay has orange/red border around it

Project Member Reported by derat@chromium.org, Feb 2 2017

Issue description

Chrome Version: 56.0.2924.87
OS Version: 9000.81.0

The "Hold Control, Alt, Shift, ..." box displayed above the Ctrl+Alt+/ keyboard overlay has a weird one-pixel orange/red border around it when the overlay is first displayed. It goes away if you press and release a modifier.

I'm unable to take a screenshot of this (since the screenshot accelerator makes the box go away). I suspect that it's a focus ring.

I'm not sure who, if anyone, owns this now. Fixing it is hopefully just a matter of tweaking some CSS.
 
Labels: -Pri-2 Pri-3
Owner: afakhry@chromium.org
Cc: xiy...@chromium.org tbuck...@chromium.org
Status: Assigned (was: Untriaged)
This is the default focus ring. We manually request to focus that element specifically for chrome vox to start reading the instructions, otherwise it will start reading keys in random order.

I can change the style of the focus ring. Do we want to do that?
IMG_20170202_101050.jpg
4.7 MB View Download

Comment 3 by derat@chromium.org, Feb 2 2017

Your phone has a better camera than mine. :-)

Yes, I think it makes sense to hide the focus ring in this case. I don't think that any interaction with the message (or any other part of the overlay) is possible, and the ring looks strange as well (e.g. it doesn't follow the box's rounded corners).
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ccfadbad03c399f42543a9ea45d1748a85278216

commit ccfadbad03c399f42543a9ea45d1748a85278216
Author: afakhry <afakhry@chromium.org>
Date: Mon Feb 06 23:06:41 2017

Remove the ugly focus outline of the keyboard overlay instructions

The instructions div of the keyboard overlay is programmaticllay focused
when the widget is shown to enable chrome vox to read it the first thing, however
the default style of a focused element is to show an ugly orange outline around it.

This CL removes that outline.

BUG= 688016 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2679563003
Cr-Commit-Position: refs/heads/master@{#448433}

[modify] https://crrev.com/ccfadbad03c399f42543a9ea45d1748a85278216/chrome/browser/resources/chromeos/keyboard_overlay.css

Status: Fixed (was: Assigned)

Comment 6 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 7 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 8 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment