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 684630 - Documentation of the signal "visible-data-changed" is not enough
Documentation of the signal "visible-data-changed" is not enough
Status: RESOLVED OBSOLETE
Product: atk
Classification: Platform
Component: docs
unspecified
Other Linux
: Normal normal
: ---
Assigned To: ATK maintainer(s)
ATK maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-09-22 17:05 UTC by Alejandro Piñeiro Iglesias (IRC: infapi00)
Modified: 2021-06-10 11:25 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-09-22 17:05:17 UTC
Looking at the current ATK implementations:
  * gtk emit it sometimes
  * clutter/st not emit it
  * WebKitGTK doesn't emit it either.

This signal is also available on atspi and KDE developers are wondering when they need to emit it.

Lookin at the current documentation:

http://developer.gnome.org/atk/stable/AtkObject.html#AtkObject-visible-data-changed
The "visible-data-changed" signal is emitted when the visual appearance of the object changed. 

This is not enough, IMHO, to know when and how to emit it. Gtk is the example that the signal is emitted, so I assume that useful for AT, but obviously is not enough to know when to emit it, as it is not emitted on any "visible-data" change.

This is also related with all the work about common, shareable and cross-toolkit accessibility implementation guidelines.
Comment 1 José Millán Soto 2012-09-22 19:48:00 UTC
IAccessible2 has a similar event.
Acording to [1]: "IA2_EVENT_VISIBLE_DATA_CHANGED  A visible data event indicates the change of the visual appearance of an accessible object. This includes for example most of the attributes available via the IAccessibleComponent interface." (IAccessibleComponent has 3 properties: location, background color and foreground color)

[1] http://accessibility.linuxfoundation.org/a11yspecs/ia2/docs/html/_accessible_event_i_d_8idl.html
Comment 2 Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-09-22 21:33:14 UTC
(In reply to comment #1)
> IAccessible2 has a similar event.
> Acording to [1]: "IA2_EVENT_VISIBLE_DATA_CHANGED  A visible data event
> indicates the change of the visual appearance of an accessible object. This
> includes for example most of the attributes available via the
> IAccessibleComponent interface." (IAccessibleComponent has 3 properties:
> location, background color and foreground color)
> 
> [1]
> http://accessibility.linuxfoundation.org/a11yspecs/ia2/docs/html/_accessible_event_i_d_8idl.html

That seems to suggest that each time something like that changed, the signal need to be emitted. Something that seems an overkill.

Looking at Orca, I only found a reference to that signal, onVisibleDataChanged at thunderbird/script.py, and it seems that it is never called/used (so probably it was something used in the past, but not anymore).

Anyway, this let me to take a look to Gecko code, and I only found a reference to this signal:


    case nsIAccessibleEvent::EVENT_SECTION_CHANGED:
        MAI_LOG_DEBUG(("\n\nReceived: EVENT_SECTION_CHANGED\n"));
        g_signal_emit_by_name(atkObj, "visible_data_changed");
        break;

So it really seems that it is not only badly defined, but not used at all. So I see two options right now:

 * It is not used because it was badly defined, so badly implemented => improve documentation/guidelines, ask implementors to implement it properly, notify AT that they can use it again
 * It is not used because it is not required => deprecate it documenting why it is not required, add that documention to the guidelines, ask implementors to stop to emit it
Comment 3 André Klapper 2021-06-10 11:25:15 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version of atk, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a ticket at
  https://gitlab.gnome.org/GNOME/atk/-/issues/

Thank you for your understanding and your help.