New issue
Advanced search Search tips

Issue 856297 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Chrome OS keyboard shortcut viewer tabbed pane focused tab appearance

Project Member Reported by msw@chromium.org, Jun 25 2018

Issue description

Chrome OS keyboard shortcut viewer tabbed pane focused tab appearance
(1) Run Chrome OS (on linux desktop) on ToT / 69.0.3469.0 
(2) Launch the keyboard shortcut viewer (Ctrl-Alt-/)
(3) Observe the selected tab appearance and hit the Tab key.
Expected: The selected tab should show focus in a nice way.
Actual: The selected tab show focus with an ugly filled rect.

It might be nice to outline the shape of the selected tab itself?
I imagine PM/UX might have some input for the desired appearance.

See the attached screenshots.
Tao, can you take a look? Thanks.
 
tabbed_pane_unfocused.png
64.0 KB View Download
tabbed_pane_focused.png
63.6 KB View Download

Comment 1 by wutao@chromium.org, Jun 25 2018

Cc: ovanieva@chromium.org
We noticed this before with conflict of MDTab focus ring.
Maybe we do not show focus ring when it is highlighted mode?

+ovanieva@

Comment 2 by wutao@chromium.org, Jun 25 2018

Cc: sgabr...@chromium.org
Labels: -Pri-3 M-89 Pri-1
+sgabriel@
Or make the ring a little larger than the highlighted background?

Comment 3 by msw@chromium.org, Jun 25 2018

FWIW, I think it'd be nice to show a focus ring that follows the contours of the tab highlight round rect.
Focus for this is defined as a darker shade of blue, following the spec here.
https://drive.google.com/open?id=16UgdCDlKqKksvzFnL2NvIkaw9AKyXAVi&authuser=sgabriel@google.com

I think we should do that. I believe this issue is happening when switching the harmony flag on.

Comment 5 by wutao@chromium.org, Jun 25 2018

Thanks sgabriel@, we can change the color of highlighted rect.
But I do not see "focus ring" in the spec when the tab is focused. Do we show that "focus ring" at all?
It replaces the focus ring.

Comment 7 by wutao@chromium.org, Jun 25 2018

Thank you sgabriel@.

Comment 8 by wutao@chromium.org, Jun 25 2018

Mergedinto: 822382
Status: Duplicate (was: Assigned)

Sign in to add a comment