table rows of XUL trees no longer containing cell content as accessible name

RESOLVED FIXED in mozilla7

Status

()

Core
Disability Access APIs
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: MarcoZ, Assigned: surkov)

Tracking

(Blocks: 1 bug, {regression})

Trunk
mozilla7
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
STR:
1. Open about:config.
2. Inspect the table.

Expected: The row accessibles should contain an accessible name consisting of all the string values of the cells below them.
Actual: They're empty now.

This is a regression from bug 546812, which was supposed to only fix the accessible name propagation from the tree container to the individual rows, but should not have broken name calculation from the cells up to the row.
(Reporter)

Comment 1

6 years ago
A direct result is that, when arrowing up and down through the table, NVDA no longer reads the currently selected item.
(Assignee)

Comment 2

6 years ago
changing summary to better reflect the bug (otherwise it sounds like tree tables are affected too)
(Assignee)

Updated

6 years ago
Summary: tree table rows no longer containing cell content as accessible name → table rows of XUL trees no longer containing cell content as accessible name
(Assignee)

Comment 3

6 years ago
though actually we want to have rows of tables and tree tables for XUL trees to expose name as cell name concatenation (Marco filed bug 664384 to deprecate this approach in the future).
Surkov, where you planing on working on this soon?  I think this probably causes a bit of pain in nvda, or atleast will when it gets to something other than nightly, and I noticed that I couldn't check that the fix for bug 659018  actually worked because orca nolonger reads anything in the message view in thunderbird.
Blocks: 459353
(Assignee)

Comment 5

6 years ago
Created attachment 542729 [details] [diff] [review]
patch
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #542729 - Flags: review?(marco.zehe)
(Reporter)

Comment 6

6 years ago
Comment on attachment 542729 [details] [diff] [review]
patch

r=me, thanks!
Attachment #542729 - Flags: review?(marco.zehe) → review+
pushed to try with 666504 606085  666337  429642   667396 as http://tbpl.mozilla.org/?tree=Try&rev=dbf9bbf7adc9
landed http://hg.mozilla.org/mozilla-central/rev/3b60b16fce55
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.