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 779009 - Missing property-change::accessible-description events when the tooltip is used as description
Missing property-change::accessible-description events when the tooltip is us...
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Accessibility
3.22.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2017-02-21 10:59 UTC by Colomban Wendling
Modified: 2017-12-10 02:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix updating the widget accessible description when using its tooltip (1.21 KB, patch)
2017-02-21 10:59 UTC, Colomban Wendling
reviewed Details | Review
Fix updating the widget accessible description when using its tooltip (GTK+2 version) (1.09 KB, patch)
2017-02-21 11:01 UTC, Colomban Wendling
none Details | Review
proposed fix (1.24 KB, patch)
2017-10-23 09:47 UTC, Samuel Thibault
none Details | Review
proposed fix (gtk2) (1.09 KB, patch)
2017-10-23 09:47 UTC, Samuel Thibault
none Details | Review
proposed fix (gtk2) (1.12 KB, patch)
2017-10-23 09:59 UTC, Samuel Thibault
none Details | Review
proposed fix (1.08 KB, patch)
2017-10-29 15:13 UTC, Samuel Thibault
committed Details | Review
proposed fix (gtk2) (984 bytes, patch)
2017-10-29 15:13 UTC, Samuel Thibault
accepted-commit_now Details | Review

Description Colomban Wendling 2017-02-21 10:59:18 UTC
Created attachment 346310 [details] [review]
Fix updating the widget accessible description when using its tooltip

The the widget tooltip is used as the accessible description, the property-change::accessible-description event should be fired when the tooltip changes as `get_description()` will give a different result.

Attached patch fixes it.
Comment 1 Colomban Wendling 2017-02-21 11:01:28 UTC
Created attachment 346311 [details] [review]
Fix updating the widget accessible description when using its tooltip (GTK+2 version)

GTK 2.24 version of the patch
Comment 2 Daniel Boles 2017-09-13 22:12:12 UTC
Review of attachment 346310 [details] [review]:

fwiw, it makes perfect sense to me, though I'd quibble about the comment:

::: gtk/a11y/gtkwidgetaccessible.c
@@ +485,3 @@
                                             widget);
+
+      /* As we return custom description in get_description(), notify

I think something like this would be clearer: "If the accessible has no specific description, :accessible-description gets the tooltip text, so notify that the observed value has changed."
Comment 3 Samuel Thibault 2017-10-23 09:47:18 UTC
Created attachment 362086 [details] [review]
proposed fix
Comment 4 Samuel Thibault 2017-10-23 09:47:40 UTC
Created attachment 362087 [details] [review]
proposed fix (gtk2)
Comment 5 Samuel Thibault 2017-10-23 09:48:29 UTC
Hello,

Any news on this?

FWIW, I have attached versions of the gtk3 and gtk2 patches with the proposed comment improvement.

Samuel
Comment 6 Samuel Thibault 2017-10-23 09:59:45 UTC
Created attachment 362088 [details] [review]
proposed fix (gtk2)
Comment 7 Matthias Clasen 2017-10-28 13:14:09 UTC
Review of attachment 362086 [details] [review]:

I think the comment in the code is unnecessary. Ok to push without it
Comment 8 Matthias Clasen 2017-10-28 13:15:26 UTC
Review of attachment 362088 [details] [review]:

same here
Comment 9 Samuel Thibault 2017-10-29 15:13:24 UTC
Created attachment 362490 [details] [review]
proposed fix
Comment 10 Samuel Thibault 2017-10-29 15:13:50 UTC
Created attachment 362491 [details] [review]
proposed fix (gtk2)

Here are the patches without the comment.
Comment 11 Matthias Clasen 2017-11-01 14:06:11 UTC
Review of attachment 362490 [details] [review]:

.
Comment 12 Matthias Clasen 2017-11-01 14:06:34 UTC
Review of attachment 362491 [details] [review]:

.
Comment 13 Matthias Clasen 2017-12-10 02:25:37 UTC
Attachment 346310 [details] pushed as 6934432 - Fix updating the widget accessible description when using its tooltip