Re: [g-a-devel] About the signal "text-update"



Hi. The idea of text_update event is a substitution of
text_removed/text_inserted events pair. It's about less events and
describes better what the user does. So if the user selected the text
and typed new text then it's a case for text_update event. We never
implemented it in Firefox because AT didn't implemented it, we can't
proceed it without AT since it's not backward compatible approach.
Thank you.
Alex.

On Mon, Aug 12, 2013 at 1:58 PM, Piñeiro <apinheiro igalia com> wrote:
Hi everybody,

I'm right now doing some cleaning at the documentation of atk (as a
result of doing some long-waited deprecations). As you can recall from
the atk-hackfest of 2012 [1], one of the conclusions was adding some new
signals [2] in order to deprecate AtkText::text-changed [3].  And the
signals text-insert, text-remove and text-update was added with that bug
[2].

But, the patch didn't add any documentation at all for those methods.
And although I was able to infer the meaning (and more important, the
parameters) of 'text-insert' and 'text-remove" from Fernando
announcement [4] and the code itself (at-spi2-atk+firefox), I was not
able to do the same for 'text-update'. Additionaly, and as I said some
years ago [5], I'm not sure about needing that method.

And, in fact, right now, that signal is not used at all on at-spi2-atk,
and is not emitted by firefox (that was the first application
implementing the other two methods).

So, I'm tempted to just mark that signal as deprecated (without
replacement), unless someone remember the meaning of that signal, or why
it was created.

BR

[1] https://wiki.gnome.org/Accessibility/Hackfests/ATK2012
[2] https://bugzilla.gnome.org/show_bug.cgi?id=638377
[3] https://bugzilla.gnome.org/show_bug.cgi?id=653291
[4]
https://mail.gnome.org/archives/gnome-accessibility-devel/2010-December/msg00007.html
[5]
https://mail.gnome.org/archives/gnome-accessibility-devel/2010-December/msg00008.html

--
Alejandro Piņeiro Iglesias



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]