Re: Autocorrect functionality

[Volker Krause]

On Mittwoch, 19. Oktober 2022 01:05:19 CEST Albert Astals Cid wrote:
> Sorry for the cross-posting.
> Please CC me, i'm on none of the two lists
> 
> 
> Both pimcommon and calligra have Autocorrect functionality, but PIM one
> broke at some point and I guess no one realized?

[...]

> Solutions:
>  A) Make PIM look for files also in GenericDataDir/calligra and live with
> the dependency to calligra for the feature B) Accept that the feature in
> PIM was broken for a long time and just remove it C) Make a library that
> both PIM and Calligra use
> 
> Ideally we would go with C but we all know it's not like we have lots of
> people with time available to jump and work on this...

Right, that's ultimately what I'd like to see as well, all the auto-
correction, grammar checking and translation features moving into Frameworks, 
alongside spell-checking. Those are all features you would like to have 
everywhere you write more than a few words of text IMHO.

But lacking time for this, it's not more than a wishlist item unfortunately...

Regards,
Volker

signature.asc
Description: This is a digitally signed message part.

Re: Autocorrect functionality

[Albert Astals Cid]

El dimecres, 19 d’octubre de 2022, a les 7:31:26 (CEST), Laurent Montel va 
escriure:
> Le mercredi 19 octobre 2022, 01:05:19 CEST Albert Astals Cid a écrit :
> > Sorry for the cross-posting.
> > Please CC me, i'm on none of the two lists
> > 
> > 
> > Both pimcommon and calligra have Autocorrect functionality, but PIM one
> > broke at some point and I guess no one realized?
> 
> Indeed I saw that it didn’t load default file long time ago but didn’t have
> time to investigate it.
> 
> > PIM is looking for files in
> > 
> >   GenericDataDir/autocorrect
> > 
> > https://invent.kde.org/pim/pimcommon/-/blob/master/src/pimcommon/autocorre
> > c
> > tion/autocorrection.cpp#L1063
> > 
> > PIM does not install nor contain any autocorrect.xml file
> > 
> > 
> > 
> > Calligra is looking for files both in
> > 
> >   GenericDataDir/autocorrect and GenericDataDir/calligra/autocorrect
> > 
> > https://invent.kde.org/office/calligra/-/blob/master/plugins/textediting/a
> > u
> > tocorrection/Autocorrect.cpp#L727
> > 
> > Calligra does contains an autocorrect.xml file that gets installed into
> > GenericDataDir/calligra/autocorrect/autocorrect.xml
> > 
> > 
> > So unless there's something wrong in my assessment, PIM autocorrect is
> > broken, and if it was not broken it would depend on calligra for the files
> > to exist since it does not ship its own autocorrect files (that's good,
> > since duplication is bad)
> > 
> > Solutions:
> >  A) Make PIM look for files also in GenericDataDir/calligra and live with
> > 
> > the dependency to calligra for the feature
> 
> Not a good solution as even me I don’t install it.
> 
> > B) Accept that the feature in
> > PIM was broken for a long time and just remove it
> 
> Not acceptable for sure.
> 
> > C) Make a library that both PIM and Calligra use
> 
> If you have time ok :)
> 
> D) duplicate files, it’s the easy solution.

I don't think that's a good idea, counting language files it's 15 files, 
copying 
them means they will definitely diverge at some point

Cheers,
  Albert

> 
> > Ideally we would go with C but we all know it's not like we have lots of
> > people with time available to jump and work on this...
> > 
> > Comments? Opinions?
> > 
> > Sorry for the cross-posting.
> > Please CC me, i'm on none of the two lists
> > 
> > Cheers,
> > 
> >   Albert

Re: Autocorrect functionality

[Laurent Montel]

Le mercredi 19 octobre 2022, 18:02:58 CEST Albert Astals Cid a écrit :
> > 
> > D) duplicate files, it’s the easy solution.
> 
> I don't think that's a good idea, counting language files it's 15 files,
> copying them means they will definitely diverge at some point
> 

Or we need to move them in a repo which is a dependancy for pim* and calligra
=> in a framework module.
Otherwise kmail autocorrect will be always broken.

After that is calligra word is still in development ? Is still release ? 

Regards.

Re: Autocorrect functionality

[René J . V . Bertin]

On Wednesday October 19 2022 21:57:14 Laurent Montel wrote:
>
>Or we need to move them in a repo which is a dependancy for pim* and calligra
>=> in a framework module.
>Otherwise kmail autocorrect will be always broken.

judging from the results of autocorrect I see posted I'd remove the 1st two 
words from that last sentence...

and I'd be more in favour of a framework that teaches people to ego-correct :-/

R.