GNOME Bugzilla – Bug 684705
AtkText insert-text, remove-text and update-text lacks documentation
Last modified: 2013-08-13 09:49:16 UTC
This signals were added by this bug: https://bugzilla.gnome.org/show_bug.cgi?id=638377 They don't have any documentation at all, and don't mark text-changed as deprecated.
Created attachment 225054 [details] [review] Adds the documentation This patch update the documentation for AtkText:: signals: * text-changed: marked as deprecated * text-insert: explain usage and parameters * text-remove: explain usage and parameters The pending thing is text-update. Anyway, I don't know which documentation to add. I made the same question when the signals were added: https://mail.gnome.org/archives/gnome-accessibility-devel/2010-December/msg00008.html Almost two years ago, wihout answer. I was able to infer text-insert and text-remove documentation because Mozilla and WebkitGTK are already implementing those signals (GTK and Clutter are still using text-changed::<detail>). Bug 638377 also updated at-spi2-atk, but not used -update at all. As far as I see, nobody is implementing or using text-update. I'm tempted to just remove that signal (it was added without any ABI change, so removing it could be also safe).
'text-update' is still incomplete, but the other 2 were fixed with this commit: commit 10782c952a715562a611b7b69d5b20cf3b389519 Author: Alejandro Piñeiro <apinheiro@igalia.com> Date: Tue Aug 13 11:46:50 2013 +0200 doc: little update on 'text-insert' and 'text-remove' documentation https://bugzilla.gnome.org/show_bug.cgi?id=684705 I will manage 'text-update' outside this bug, as the discussion is still ongoing: https://mail.gnome.org/archives/gnome-accessibility-devel/2013-August/msg00013.html