New issue
Advanced search Search tips

Issue 646100 link

Starred by 2 users

Issue metadata

Status: Verified
Owner: ----
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

ChromeVox should support the "mixed" checkbox state

Project Member Reported by dmazz...@chromium.org, Sep 12 2016

Issue description

ChromeVox needs to support checkboxes in the "mixed" state.

<div role="checkbox" aria-checked="mixed" tabindex=0>
  ARIA checkbox
</div>

Internally we already have a state "mixed" that handles this, we should check that in addition to checked.

For braille, we currently use "( )" for unchecked and "(X)" for checked, I propose "(-)" for mixed, as that conveniently corresponds with half of the dots of an X character in braille.


 
Status: Available (was: Untriaged)
Components: UI>Accessibility
Labels: OS-Chrome
Labels: braille Phase3
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 25 2016

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

commit 6007cdb65f04ec552d34c04b744cd50f16defc01
Author: dtseng <dtseng@chromium.org>
Date: Tue Oct 25 21:45:23 2016

Complete checked state announcements and clarify state usage

Firstly, the checked state has some unexpected bheaviors as follows:
- aria checked is not reflected by the state bitmap; for example, if aria-checked is set to true, the state bitmap doesn't actually contain the checked state. ChromeVox should not have to make this workaround, but for the purposes of output, we should do the right thing regardless of this behavior
- some states really should only be relevant  when they are true (e.g. collapsed, expanded, visited); encode this in the state data
- other states are only relevant for specific roles (e.g. pressed).

BUG= 646100 , 628862 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/common/chrome_extension_externs.js
[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
[modify] https://crrev.com/6007cdb65f04ec552d34c04b744cd50f16defc01/chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings.grd

Status: Verified (was: Fixed)
as per #5

Sign in to add a comment