Issue 103961 - Inserting rows in a table results in a11y events announcing that the table has been removed
Summary: Inserting rows in a table results in a11y events announcing that the table ha...
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: ui (show other issues)
Version: OOO310m14
Hardware: All Solaris
: P3 Trivial (vote)
Target Milestone: ---
Assignee: eric.savary
QA Contact: issues@sw
URL:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2009-08-02 21:10 UTC by joaniediggs
Modified: 2013-08-07 14:44 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description joaniediggs 2009-08-02 21:10:35 UTC
Steps to reproduce:

1. Launch OOo Writer and Accerciser

2. In Accerciser, in the tree of accessibles on the left, highlght/select the
Accessible named "Document view" which is of role "document frame".

3. In Accerciser's Event Monitor, select the "Selected accessible" radio button
as the source. Check "object" for the type of event to monitor (I unchecked
bounds-changed, but left the rest of the object events checked.)

4. In Writer, go to the Table menu, choose Insert, accept the defaults, and
press the OK button.

5. Press Tab until the last cell in the table is reached. Then press Tab again,
which will cause a new row to be inserted.

Results:

After step 4, the following event is shown by Accerciser:

  object:children-changed:add(0, 0, [table | Table1-1])
	source: [document frame | Document view]
	application: [application | soffice]

This is expected. So far so good.

After step 5, the following event is shown by Accerciser:

  object:children-changed:remove(0, 0, [table | Table1-1])
	source: [document frame | Document view]
	application: [application | soffice]

In other words, when the user adds a new row to the current table, Writer is
emitting events to ATs announcing that the current table, which is still very
much present, no longer exists. This is problematic for screen readers like Orca
because they cannot/will not present any information to the user for an object
which no longer exists. (i.e. The current table is rendered inaccessible because
of this bug.)

It would be less problematic (I *think*) if a new object:children-changed:add
event were emitted for a replacement table. However, under those circumstances,
the AT still has to work out that the current object was replaced with a new
object. That can be hard to do. Therefore, given that the original table is
(from an AT and an end-user point of view) still present, but a bit larger, what
is expected is the following:

1. The aforementioned object:children-changed:removed event would not be emitted.

2. Object:children-changed:add events would be emitted by the table to announce
the addition of the new cells.

Note that this is reproducible in the latest 3.1.1-dev (build 9415) and 3.2-dev
(build 9412) builds available for external download.
Comment 1 malte_timmermann 2009-12-03 10:26:56 UTC
Sounds to be something for OOo 3.3...
Comment 2 Oliver-Rainer Wittmann 2009-12-03 13:51:01 UTC
OD->ES: Can you please confirm this issue and assign it back to me.

OD->MT: Do not hurry and do not bypass the quality assurance team.
Comment 3 eric.savary 2009-12-16 17:25:57 UTC
CONFIRMED.

As described I get object:children-changed:*remove*
Comment 4 Oliver-Rainer Wittmann 2010-02-16 16:05:44 UTC
fix in progress in cws sw33a11y01
Comment 5 Oliver-Rainer Wittmann 2010-02-18 09:15:31 UTC
fixed in cws sw33a11y01 - changed files:
/sw/source/core/inc/cellfrm.hxx,
/sw/source/core/layout/tabfrm.cxx,
/sw/source/core/access/acctable.cxx,
/sw/source/core/layout/wsfrm.cxx,
change set 6a3ca94f51cd
Comment 6 Oliver-Rainer Wittmann 2010-03-11 13:56:23 UTC
OD->ES: Checked in internal installation set of cws sw33a11y01 - please verify.
Important note: Corresponding object:children-changed events are now emitted at
the accessible object representing the table.
Comment 7 eric.savary 2010-04-22 14:18:48 UTC
Verified in CWS sw33a11y01
Comment 8 malte_timmermann 2010-06-23 12:17:57 UTC
Closing accessibility issues which have been fixed, verified and integrated...