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

Issue 894602 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit 15 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

A11y test error because links have no underlines

Project Member Reported by jlklein@chromium.org, Oct 11

Issue description

See example test failure here:

https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8932942268393419584/+/steps/browser_tests__with_patch_/0/stdout

The error states that "Links must be distinguished from surrounding text in a way that does not rely on color".

Context: "https://dequeuniversity.com/rules/axe/3.0/link-in-text-block?application=axeAPI",

Shiba, can you give us some guidance on this? I'm landing an exception to unblock this test right now, but we should figure out if this is the right thing to do.
 
It sounds from the context link like font style differences (bold, underline, italics) or a contrast difference of 3:1 with surrounding text would be ok.
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 12

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

commit 4cf4ff7aa5b049728149b2a55e6a021301fa6657
Author: Jeremy Klein <jlklein@google.com>
Date: Fri Oct 12 01:29:37 2018

Fix a11y test error for multidevice settings.

Adding a temporary exception for one link-in-text-block error. This
error comes from the fact that we (intentionally) removed the underline
from the learn more links in our settings. I've filed crbug.com/894602
to work with UX on a new strategy and remove this exception.

Bug: 894602
Change-Id: I0da37adfee02699415a011939dbfc6d4e1d93f34
Reviewed-on: https://chromium-review.googlesource.com/c/1278131
Commit-Queue: Jeremy Klein <jlklein@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599059}
[modify] https://crrev.com/4cf4ff7aa5b049728149b2a55e6a021301fa6657/chrome/test/data/webui/settings/a11y/multidevice_features_a11y_test.js

Cc: -elizabethchiu@chromium.org
Owner: elizabethchiu@chromium.org
I thought removing the underlines was WAD. Elizabeth could you confirm?
Yeah, it *is* working as shown in the mocks and spec (no underlines). However, the lack of underlines for a link is going against accessibility guidelines and is causing this test to fail. The exact error is:

"Links must be distinguished from surrounding text in a way that does not rely on color".
Status: Assigned (was: Unconfirmed)
This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.

Sign in to add a comment