Code review comment for lp:~nmarshall23/do-plugins/spellcheck-plugin

Revision history for this message
Nicholas Marshall (nmarshall23) wrote :

> it would be better to check the user's locale, and use that.
Chris this sound resonable, I will start working on that.
Every app I have looked at seems to do just that.

Now as an aditional action after CheckSpelling has been selected, if you
could change Dictionaries sure, that seems to fit in the use case.

I have mixed feeling on a config widget, sure it maybe useful.. but I can't
see it being used all that much. We will just have to think about it, and
get feedback from users.

Nicholas Marshall

On 8/13/09, Alex Launi <email address hidden> wrote:
>
> I just think it's silly to create a config widget for one option in a
> corner
> case which is probably never going to pop up. The BEST solution would be to
> parse the word, create a list of possible languages based on some heuristic
> (there is research in this area), and check each dictionary to see if it's
> spelled correctly!
>
> But that might be more work than our friendly new contributer wants to do..
> :P
>
> --
> -- Alex Launi
>
>
> https://code.launchpad.net/~nmarshall23/do-plugins/spellcheck-plugin/+merge/10048
> You are the owner of lp:~nmarshall23/do-plugins/spellcheck-plugin.
>

--
И

« Back to merge proposal