New issue
Advanced search Search tips

Issue 856555 link

Starred by 4 users

Issue metadata

Status: Duplicate
Merged: issue 835550
Owner:
Closed: Jul 12
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Mac views] Blue focus is seen at 'X' icon of a tab while moving tabs on chrome

Project Member Reported by jmukthavaram@chromium.org, Jun 26 2018

Issue description

Chrome Version: Version 69.0.3473.0
OS: MacOS 10.13.4

What steps will reproduce the problem?
(1) Enable 'Use Views browser windows instead of Cocoa.' in about://flags
(2) Relaunch chrome
(3) Open multiple tabs (Gmail, youtube, wikipedia...) on same Window
(4) Now try to move tab from one place to other place and observe blue focus on 'X' icon of a tab

What is the expected result?
No blue focus should be seen at 'X'  icon of  a tab while moving tabs on chrome

What happens instead?
Blue focus is seen at 'X'icon of a tab while moving tabs on chrome

Screencast attached for reference.

Thanks..!
 
Blue highlight is seen at close button of a tab.mp4
3.2 MB View Download
Labels: -Pri-2 MacViews-Browser M-69 Project-Views Target-69 FoundIn-69 Pri-1
Note:
1. Issue seen only on M69 build & no issue seen on M68.
2. Issue also seen on guest mode & incognito mode.

Summary: [Mac views] Blue focus is seen at 'X' icon of a tab while moving tabs on chrome (was: Blue focus is seen at 'X' icon of a tab while moving tabs on chrome)
Labels: Hotlist-Polish
Status: Available (was: Untriaged)
Owner: tommycli@chromium.org
Status: Started (was: Available)
I'll take a look.
This is only on Mac because only Mac has FocusManager::keyboard_accessible_ == true.

https://cs.chromium.org/chromium/src/ui/views/focus/focus_manager.cc?q=FocusManager&sq=package:chromium&dr=CSs&l=566
This CL fixes the bug by making the tabs themselves focusable:
https://chromium-review.googlesource.com/c/chromium/src/+/1121349

After the above, we should do a followup CL that:
 1. Draws some visual indication that the tab itself is focused
 2. Make the tab itself respond to keyboard Space to activate the tab.
Labels: -MacViews-Release
Labels: -M-69 Group-Focus_Input_Selection_Activation_KeyState
Labels: M-69
Mergedinto: 835550
Status: Duplicate (was: Started)

Sign in to add a comment