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 738982 - Add support for ATK_STATE_HAS_TOOLTIP
Add support for ATK_STATE_HAS_TOOLTIP
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Accessibility
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on: 407539
Blocks:
 
 
Reported: 2014-10-22 03:35 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2014-10-23 16:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (note: still needs ATK version check or minimum-version bump) (5.19 KB, patch)
2014-10-22 03:35 UTC, Joanmarie Diggs (IRC: joanie)
accepted-commit_now Details | Review

Description Joanmarie Diggs (IRC: joanie) 2014-10-22 03:35:24 UTC
Created attachment 289100 [details] [review]
proposed patch (note: still needs ATK version check or minimum-version bump)

We should have ATK_STATE_HAS_TOOLTIP soon (see bug 407539). Once we do, Gtk+ should support it.

The attached patch here works with the attached patch on bug 407539 and can be tested via assistive technologies if you also have the patch attached to bug 738953. It includes a new test. It does not include an ATK version check or minimum-version bump. I'll fix that, along with any other needed changes, once the new state is in a released version of ATK.
Comment 1 Matthias Clasen 2014-10-22 20:39:58 UTC
Review of attachment 289100 [details] [review]:

looks good to me; if we have that state in 3.14 already, we should cherry-pick this on the gtk-3-14 branch as well
Comment 2 Joanmarie Diggs (IRC: joanie) 2014-10-23 10:49:42 UTC
As I see you discovered yesterday, this state is not in ATK 2.14. It's new API. Related to that, it's only just now added to ATK master. As I mentioned in the opening report:

> The attached patch [...] does not include an ATK version check or
> minimum-version bump. I'll fix that, along with any other needed changes, once
> the new state is in a released version of ATK.

My plan was to wait for Piñeiro to roll a release of ATK that included it, and then provide you with a new patch that had that check/bump. Since you committed the patch as-is, are you planning on adding that change to Gtk+ yourself after there is a new ATK release?
Comment 3 Matthias Clasen 2014-10-23 16:33:24 UTC
yeah, we'll add the version check when the version exists, and when somebody complains