New issue
Advanced search Search tips

Issue 887624 link

Starred by 1 user
Status: Fixed
Owner:
joanmari...@gmail.com
Closed: Sep 24
Cc:
aleventhal@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

Incorrect trailing whitespace in accessible name from label contents

Project Member Reported by joanmari...@gmail.com, Sep 20 
Steps to reproduce:
1. Load the attached test case.
2. Use the platform's accessibility inspector to obtain the name of the input with id of "test".

Expected results: The name would be "foo"

Actual results: The name is "foo "
trailing-whitespace.html
104 bytes View Download

Comment 1 by joanmari...@gmail.com, Sep 21

Owner: joanmari...@gmail.com
Status: Assigned (was: Available)

Comment 2 by joanmari...@gmail.com, Sep 21

Summary: Incorrect trailing whitespace in accessible name from label contents (was: Incorrect trailing whitespace in accessible name from label contenets)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 24 

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

commit d3cd27a8e531f3a2c9799fefea77d86b821b81fe
Author: Joanmarie Diggs <jdiggs@igalia.com>
Date: Mon Sep 24 15:05:28 2018

Strip whitespace from label contents for accessible name calculation

The whitespace characters surrounding the text to be displayed for a label
element should not be treated as part of the accessible name of the element
being labeled. Stripping out this whitespace makes us consistent with what
other user agents do.

Bug:  887624 
Change-Id: I1ff691ee7d970435b2b912a2d2b565089b1fc7a3
Reviewed-on: https://chromium-review.googlesource.com/1238774
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593537}
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-checkbox-label-expected-blink.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-checkbox-label-expected-mac.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-list-expected-auralinux.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-list-expected-blink.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-list-expected-mac.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-list-expected-win.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-types-expected-android.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-types-expected-auralinux.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-types-expected-mac.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/content/test/data/accessibility/html/input-types-expected-win.txt
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/third_party/WebKit/LayoutTests/accessibility/name-calc-inputs.html
[modify] https://crrev.com/d3cd27a8e531f3a2c9799fefea77d86b821b81fe/third_party/blink/renderer/modules/accessibility/ax_node_object.cc

Comment 4 by joanmari...@gmail.com, Sep 24

Status: Fixed (was: Assigned)
►

Sign in to add a comment