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

Issue 687227 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

"Increase Contrast" mode doesn't affect the omnibox or toolbar buttons

Project Member Reported by pinkerton@chromium.org, Jan 31 2017

Issue description

Chrome Version: M56beta
OS: macOS 10.12.2

- In System Preferences, enable "Accessibility > Display > Increase Contrast"

expected:
- Omnibox should have a black stroke around it so it's easier to see
- back/forward/reload buttons should have a black stroke around them so they are easier to see
- Extension icons should probably have the same.

actual:
- Nothing changes

Safari does this correctly and darkens the stroke around everything on their toolbar. 
 

Comment 1 by rpop@chromium.org, Jan 31 2017

Cc: emilyschechter@chromium.org bettes@chromium.org
Components: UI>Browser>Omnibox
Note that  bug 565996  is about the larger question of addressing this across our UI in the MacViews world, so this bug could be scoped to whether we want to address this before that.
Components: UI>Accessibility
Owner: ellyjo...@chromium.org
Status: Assigned (was: Untriaged)
#3: since we think MacViews top chrome is a long way off, this bug was about fixing the problem in the Cocoa chrome, which I think will not be too difficult.

Also, yoink :)
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 23 2017

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

commit b85e2704bd0928862b747cc19e10b95f35efef04
Author: ellyjones <ellyjones@chromium.org>
Date: Thu Feb 23 17:23:04 2017

cocoa: draw BackgroundGradientView separators darker in Increase Contrast mode

BUG= 687227 

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

[modify] https://crrev.com/b85e2704bd0928862b747cc19e10b95f35efef04/chrome/browser/ui/cocoa/background_gradient_view.mm

Project Member

Comment 7 by bugdroid1@chromium.org, Feb 23 2017

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

commit 2c766c7081a39d69bdde6f008f1c5e0dc6d4388c
Author: ellyjones <ellyjones@chromium.org>
Date: Thu Feb 23 17:53:52 2017

omnibox: use dark border stroke in Increase Contrast mode

BUG= 687227 

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

[modify] https://crrev.com/2c766c7081a39d69bdde6f008f1c5e0dc6d4388c/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.mm

Status: Fixed (was: Assigned)
This is finally Fixed :D tomorrow's canary should have these changes.

Sign in to add a comment