Bug 434173

I confirm the described behaviour. Would like to use spell checking in the webmail client but "Settings" --> "Configure Spell Checking" does not do anthing.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20210508
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.81.0
Qt Version: 5.15.2
Konqueror Version: 21.04.0
webenginepart version: 21.04.0
Kernel Version: 5.12.0-2-default
OS Type: 64-bit
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-7700T CPU @ 2.90GHz
Memory: 7.6 GiB of RAM
Graphics Processor: Mesa DRI Intel® HD Graphics 630

Created attachment 143161 [details]
Configure Spell Checking in Konqueror's Settings menu

The spell check still does not work in Konqueror (though it works well in Kate).  
Past updates have brought some improvements, though. Now, the Konqueror settings menu lets you choose "Configure Spell Checking". However, when working in any text input window, right click opens a multiple choice, where you can enable spell checking, but when right clicking again, "Spell Checking Enabled" is ticked but the menu point "Languages" below does not offer any choice.

Operating System: openSUSE Tumbleweed 20211031  
KDE Plasma Version: 5.23.2  
KDE Frameworks Version: 5.87.0  
Qt Version: 5.15.2  
Kernel Version: 5.14.14-1-default (64-bit)  
Graphics Platform: X11  

Spell checker: Hunspell  
Dictionaries: myspell de, de_DE, en, en_GB, fr_FR, pt_BR  
(No ispell nor aspell installed.)

Created attachment 143162 [details]
Related system settings for the spell checker configuration

Created attachment 143163 [details]
Related system settings for the spell checker configuration

Git commit 7cc349a1d6a1ff7ed54fc887f1abf911b20b1398 by David Faure, on behalf of Stefano Crocco.
Committed on 01/01/2022 at 19:53.
Pushed by dfaure into branch 'master'.

Perform spellchecker setup on creation

M  +3    -11   webenginepart/src/spellcheckermanager.cpp
M  +0    -3    webenginepart/src/spellcheckermanager.h

https://invent.kde.org/network/konqueror/commit/7cc349a1d6a1ff7ed54fc887f1abf911b20b1398

This bug is still present in Debian 12 Konqueor (v. 4:22.12.3-1). The spell checking is not working.

I checked that the patch (which was mentioned in comment #6) is present in the Debian 12 Konqueror's source code, but the Spelling -> Languages context menu is still empty and it does nothing, even I can see my installed spell checking languages in "Spell Checking Configuration" window, see the screenshots: https://imgur.com/a/IDFHA1p

In Kate the spell checking is working just fine.

(In reply to Miikka from comment #7)
> This bug is still present in Debian 12 Konqueor (v. 4:22.12.3-1). The spell
> checking is not working.
> 
> I checked that the patch (which was mentioned in comment #6) is present in
> the Debian 12 Konqueror's source code, but the Spelling -> Languages context
> menu is still empty and it does nothing, even I can see my installed spell
> checking languages in "Spell Checking Configuration" window, see the
> screenshots: https://imgur.com/a/IDFHA1p
> 
> In Kate the spell checking is working just fine.

Unfortunately, even if it uses the same configuration dialog, spellchecking in Konqueror uses a different engine than other KDE applications, one which is included in the web engine itself. That engine needs dictionary files in a specific format and this has caused several bugs in the past (even if they haven't been posted here). These bugs have been fixed in a more recent version of Konqueror than you are using.

Depending on how Debian builds Konqueror, it's likely that there's no way to fix your issue except by building Konqueror from source yourself. The first thing you need to check is whether you have dictionary files with a .bdic extension. You can look for them using the command
find /usr -name '*.bdic'

(In reply to Stefano Crocco from comment #8)
> (In reply to Miikka from comment #7)
> > This bug is still present in Debian 12 Konqueor (v. 4:22.12.3-1). The spell
> > checking is not working.
> > 
> > I checked that the patch (which was mentioned in comment #6) is present in
> > the Debian 12 Konqueror's source code, but the Spelling -> Languages context
> > menu is still empty and it does nothing, even I can see my installed spell
> > checking languages in "Spell Checking Configuration" window, see the
> > screenshots: https://imgur.com/a/IDFHA1p
> > 
> > In Kate the spell checking is working just fine.
> 
> Unfortunately, even if it uses the same configuration dialog, spellchecking
> in Konqueror uses a different engine than other KDE applications, one which
> is included in the web engine itself. That engine needs dictionary files in
> a specific format and this has caused several bugs in the past (even if they
> haven't been posted here). These bugs have been fixed in a more recent
> version of Konqueror than you are using.
> 
> Depending on how Debian builds Konqueror, it's likely that there's no way to
> fix your issue except by building Konqueror from source yourself. The first
> thing you need to check is whether you have dictionary files with a .bdic
> extension. You can look for them using the command
> find /usr -name '*.bdic'

Oh, Ok. So it's not using Sonnet as its spell checking library then, like almost all other KDE apps are using?

I'm willing to have Finnish spell checking (besides to the English), but Finnish language can't be spell checked by using static dictionaries, and that's why there's a separate library called libvoikko to spell check Finnish properly. But that's not an issue for Sonnet (or for libenchant), because they are able to use libvoikko and couple of other special spell checking libraries as their "backend" besides to those static dictionaries.

To sum up: Konqueror supports only those static dictionaries, and there's no plans to implement Sonnet in Konqueror / in its web engine? If so, then I guess I'm out of luck to achieve my goal...

(In reply to Miikka from comment #9)
> (In reply to Stefano Crocco from comment #8)
> 
> Oh, Ok. So it's not using Sonnet as its spell checking library then, like
> almost all other KDE apps are using?
> 
> I'm willing to have Finnish spell checking (besides to the English), but
> Finnish language can't be spell checked by using static dictionaries, and
> that's why there's a separate library called libvoikko to spell check
> Finnish properly. But that's not an issue for Sonnet (or for libenchant),
> because they are able to use libvoikko and couple of other special spell
> checking libraries as their "backend" besides to those static dictionaries.
> 
> To sum up: Konqueror supports only those static dictionaries, and there's no
> plans to implement Sonnet in Konqueror / in its web engine? If so, then I
> guess I'm out of luck to achieve my goal...

I think it would be impossible to use Sonnet in Konqueror, because QtWebEngine (the web engine Konqueror uses) only provides very basic functionality for it to interact with the page. This has caused several other issues (for example, Konqueror can't support all proxy options supported by other KDE applications). Regarding spell checking, the documentation (https://doc.qt.io/qt-5.15/qtwebengine-features.html#spellchecker) states:

"To be able to check the spelling, the spellchecker needs dictionaries. It supports dictionaries from the Hunspell project, but they have to be compiled into a special binary format. A Hunspell dictionary consists of two files: a .dic file [...] an .aff file "

If Hunspell doesn't support Finnish, I think there's no way to make Konqueror have it as a spellchecking language. You could try creating a bug report for QtWebEngine at https://bugreports.qt.io/secure/Dashboard.jspa (you'll need to register before doing so).