[Psi-devel] Re: a patch for auto-replace
Michail Pishchagin
mblsha-psi at affinix.com
Sun Feb 6 21:48:52 PST 2005
Remko Troncon wrote:
> On 06 Feb 2005, at 15:29, Michail Pishchagin wrote:
> > And rename
> > workOnTextChange() to "public slot: correctTextEdit()", or something
> > like that.
>
> That function doesn't make much sense from a design point of view. I
> would expect an Auto Correction framework to get a string as input, and
> give another string as output. I don't understand why you would pass a
> widget to something that has nothing to do with a user interface.
Well... It has not only to replace one string with another, it also needs to
know cursor position, to know what last typed word was. Also it needs to
reposition the cursor after the successful replace.
-mblsha
More information about the Psi-devel-affinix.com
mailing list