After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 690004 - Add ATSPI_STATE_READ_ONLY
Add ATSPI_STATE_READ_ONLY
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: at-spi2-core
unspecified
Other Linux
: Normal normal
: ---
Assigned To: At-spi maintainer(s)
At-spi maintainer(s)
: 665599 (view as bug list)
Depends on: 665598
Blocks: 690008 690107
 
 
Reported: 2012-12-10 19:59 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2017-06-16 22:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add the new state (1.19 KB, patch)
2012-12-10 20:14 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2012-12-10 19:59:16 UTC
+++ This bug was initially created as a clone of Bug #665598 +++

The definition of ATSPI_STATE_EDITABLE is: "Indicates the user can change the contents of this object." This clearly addresses text widgets. However, whether or not it also applies to widgets which:

* are enabled
* are sensitive
* are focusable
* have a usually-changeable state
* presently lack the ability to change the state

Consider, for example, cb4 on this page [1]. The checkbox is for all intents
and purposes a normal checkbox, however the user is prevented from changing its
state.

If we agree that 'state' != 'contents', then ATSPI_STATE_EDITABLE (or the lack
thereof) would not be a means to communicate the present read-only condition of
this checkbox (and other widgets like it). In this case, a new ATK state would
be needed.

If, on the other hand, the conclusion is that 'state' == 'contents', then every
widget whose contents (including state) could be altered by the user would need
to have ATSPI_STATE_EDITABLE added to it. This would involve quite a few widgets
and multiple toolkits: quite a lot of work given how rare things like the
read-only dojo checkbox happen to be. Therefore, a new ATSPI state would be
preferable.

For this reason, I am proposing the creation of ATSPI_STATE_READ_ONLY.

The addition of this state would raise the question of what to do about
read-only text widgets. Should STATE_EDITABLE be removed and
ATSPI_STATE_READ_ONLY be added? Or something else? I personally think this is
less of a concern. But FWIW, we could deprecate STATE_EDITABLE and make the new
read-only state apply to all widgets.

[1] http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/test_CheckBox.html
Comment 1 Joanmarie Diggs (IRC: joanie) 2012-12-10 20:14:18 UTC
Created attachment 231210 [details] [review]
Patch to add the new state

Adds the new state; doesn't do anything w.r.t. STATE_EDITABLE as its fate seems somewhat uncertain at the moment. (This patch depends on the patch for bug 665598 being committed. And a minimum version for the ATK dependency should be added -- once we know what that version is.)
Comment 2 André Klapper 2013-08-14 10:06:21 UTC
[Mass-resetting default assignee, see bug 705890. Please reclaim this bug report by setting the assignee to yourself if you still plan to work on this. Thanks!]
Comment 3 Joanmarie Diggs (IRC: joanie) 2015-01-15 15:06:52 UTC
Mike, have you seen this patch?
Comment 4 Mike Gorse 2015-01-15 16:03:11 UTC
Comment on attachment 231210 [details] [review]
Patch to add the new state

Looks fine. Thanks.
Comment 5 Joanmarie Diggs (IRC: joanie) 2015-01-15 22:05:57 UTC
Comment on attachment 231210 [details] [review]
Patch to add the new state

https://git.gnome.org/browse/at-spi2-core/commit/?id=f3223e3
Comment 6 Joanmarie Diggs (IRC: joanie) 2017-06-16 22:25:12 UTC
*** Bug 665599 has been marked as a duplicate of this bug. ***