GNOME Bugzilla – Bug 527688
inability to set the spellchecker's language
Last modified: 2011-12-17 04:13:29 UTC
Some Pan users subscribe to multiple newsgroups that host discussions in various different languages. Under that scenario, those Pan users aren't able to benefit from any spell checking. It would be great if Pan supported multiple spell checking languages. It would be even greater if it was possible to set the default language according to the newsgroup being posted to. Other information:
Created attachment 117132 [details] [review] Patch implementing custom, per-newsgroup spellchecker language Patch made against rev. 360 of http://svn.gnome.org/svn/pan2/trunk
I believe that this patch does the trick although, due to the limitations of GtkSpell v2 and the desire not to add another dependency to Pan, manually specifying the language tag isn't exactly an ideal solution. Nonetheless, as soon as GtkSpell v3 is released then I belive it will be trivial to implement this feature The Right Way ®
The patch applies cleanly here (to rev 361), and pan's running fine. I see the new widget in group properties, but as I just do English, I haven't tested whether it actually /works/. But at least it doesn't seem to break anything! =:^) I'm submitting it to Gentoo for trials in at least their live-svn ebuild (which I submitted). Whether it's introduced beyond that we'll see. Duncan
Gentoo bug here: http://bugs.gentoo.org/show_bug.cgi?id=235446 BTW, that version field should be 0.132 now, not the years-old 0.13.2, but that's not a field I can change. Correcting it, if you can GB, will likely get way more attention from Charles. He's pretty much ignoring the 0.14.x and previous bugs as that code is now unsupported and won't have any more releases. Duncan
Could you please guys at least review the patch ?
CC'ing Heinrich on this bug as I'm unsure whether he's aware of this.
Review of attachment 117132 [details] [review]: seems ok. thanks a lot. but the "combo" box is still just a single entry. but it's a start. i'll implement the rest.
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.