New issue
Advanced search Search tips

Issue 836956 link

Starred by 4 users
Status: Assigned
Owner:
lgrey@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

[macviews] omnibox focus ring sometimes gets stuck in the wrong theme

Project Member Reported by thakis@chromium.org, Apr 25 2018 
What steps will reproduce the problem?
1. Take turns in opening and closing tabs and toggling Appearance in SysPrefs->General while the omnibox has focus

What is the expected result?

Omnibox focus ring color keeps up with what I do in sysprefs


What happens instead of that?

Sometimes the omnibox focus color is the opposite of what's in sysprefs


Please provide any additional information below. Attach a screenshot if
possible.

UserAgentString: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.106 Safari/537.36
Screen Shot 2018-04-25 at 4.24.24 PM.png
37.8 KB View Download

Comment 1 by meh...@chromium.org, Apr 25 2018

Components: UI>Browser Internals>Views>Desktop
Labels: Proj-MacViews

Comment 2 by ellyjo...@chromium.org, Apr 26 2018

Labels: MacViews-Browser Target-69 M-69
Owner: ellyjo...@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by robliao@chromium.org, Jul 12

Labels: -M-69

Comment 4 by robliao@chromium.org, Jul 12

Labels: M-69

Comment 5 by robliao@chromium.org, Jul 21

Labels: Group-Focus_Input_Selection_Activation_KeyState

Comment 6 by ellyjo...@chromium.org, Aug 2

Labels: -M-69 -Target-69 Target-70 M-70
MacViews triage: targeting this for M70.

Comment 7 by ellyjo...@chromium.org, Aug 28

Labels: -Target-70 -M-70 Target-71 M-71
Owner: lgrey@chromium.org
Repro steps:

1) Set tint to grey
2) Open chrome, focus omnibox - see grey ring
3) Change system tint to blue
4) Switch to chrome and focus omnibox again - ring still grey

Maybe related to NSDynamicSystemColorStore as mentioned in content/renderer/theme_helper_mac.mm?

Comment 8 by rbasuvula@chromium.org, Nov 21

Labels: Hotlist-DesktopUIChecked Hotlist-DesktopUIValid
***Mass UI Triage***
As per dev comments.
►

Sign in to add a comment