Bug 376565

Summary: Automatic spell checking is not working
Product: [Applications] kmail2 Reporter: avlas <jsardid>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: bogdan.mihaila, kde_bugzilla_2, montel
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Kate showing spellcheck wiggle
Kate NOT showing spellcheck wiggle
attachment-5166-0.html

Description avlas 2017-02-16 23:33:23 UTC
I am using Kmail 5.4.2 in KDE neon User Edition 5.9.2 Frameworks 5.31.0

Automatic spell checking is enabled but mispelled words in any language do not get underscored.

This function works elsewhere, e.g. Kate.
Comment 1 avlas 2017-02-24 14:53:50 UTC
I tried creating a new user from scratch for testing purposes and it works. This doesn't seem then a packaging issue, neither an application bug. 

Most likely this relates to a configuration issue that is making it not working for my (production) user, perhaps something has change between versions and the config files that used to work, stopped working.

Following this strategy, I tried to temporarily rename kmail2 files in my home (in .config/.local/.cache) to see if that would fix the issue and identify the problem. Unfortunately this has been unsuccesful so far.

I wonder if this may be related to kdepim...

Btw, another application that seems to suffer the same (or at least similar issue) is choqok.

I'm willing to test further but also a bit out of ideas, so any suggestion on how to test this further will be highly appreciated. 

Thanks in advance.
Comment 2 avlas 2017-02-25 20:02:28 UTC
I found the root issue. It is due to the font that is being used.

The same happens in other applications such as Kate. The reason why this didn't seem to happen is because I use a monospace font in Kate where the bug didn't happen (Hack 10), but it emerge as soon as I changed it to the font of my system (Cabin 11).


This bug has been reported against Kate/Kwrite/etc before and it seems to be tagged as fixed:

https://bugs.kde.org/show_bug.cgi?id=335988
https://bugs.kde.org/show_bug.cgi?id=343313
https://bugs.kde.org/show_bug.cgi?id=366291
https://bugs.kde.org/show_bug.cgi?id=335079

However this is not fixed at all in my system (up-to-date KDE neon User Edition 5.9). It might be a regression...

Reading more carefully to https://bugs.kde.org/show_bug.cgi?id=335079 it seems the fix only works in a subset of fonts, perhaps this is enough for Kate as people tend to use monospace fonts (my included) but it's far from ideal for the rest of KDE apps that rely on spellchecking (Kate, Choqok, Konqueror...)
Comment 3 avlas 2017-02-26 14:39:03 UTC
I workarounded the issue by decreasing the font size. Not ideal to have a so small font size, but recovered automatic spell checking :-/
Comment 4 Bogdan Mihaila 2018-02-26 07:22:32 UTC
Created attachment 111001 [details]
Kate showing spellcheck wiggle

Font size 8 shows the underline wiggle on typos
Comment 5 Bogdan Mihaila 2018-02-26 07:23:38 UTC
Created attachment 111002 [details]
Kate NOT showing spellcheck wiggle

Font size 9 and higher do not show the underline wiggle on typos. On high-dpi systems such font sizes are a minimum
Comment 6 Bogdan Mihaila 2018-02-26 07:29:21 UTC
Can confirm that the bug still exists on Ubuntu.
Kate: 17.04.3
KDE Frameworks: 5.43.0
Qt: 5.9.1

This affects all apps using Katepart, e.g. Kmail, Kile, Kate, ...
As can be in the attached screenshots at font size 8 the spellcheck markers are shown but as soon as changing the font size to 9 or higher they disappear. The automatic spellchecking is thus not usable on high-dpi systems where font sizes >10 are the norm for text.
Comment 7 Laurent Montel 2018-02-26 16:11:14 UTC
you report as it's not a specific kmail bug
=> please reassign to Qt or kf5
thanks
Comment 8 avlas 2018-02-26 16:20:16 UTC
I cannot reproduce this behavior anymore. Perhaps it got fixed in Qt 5.10 ?
Comment 9 Bogdan Mihaila 2018-02-28 07:17:28 UTC
(In reply to avlas from comment #8)
> I cannot reproduce this behavior anymore. Perhaps it got fixed in Qt 5.10 ?

Interesting. This would be very good news as the bug had spellchecking unusable for more than 2 years now. I'll try to get my hands on a Kate build with Qt 5.10

Or if someone else has then please confirm the fix.

Thanks
Bogdan
Comment 10 Bogdan Mihaila 2018-02-28 07:26:02 UTC
(In reply to Laurent Montel from comment #7)
> you report as it's not a specific kmail bug
> => please reassign to Qt or kf5

Hi Laurent

I'm having a hard time to find any of above in the product drop down list of Bugzilla. Guess those just do not exist. I could reassign to frameworks-ktextedit but not sure if that is any better? Would not want to cause noise elsewhere if this might actually be fixed in Qt upstream.
Suggestions?
Comment 11 avlas 2018-02-28 16:01:02 UTC
Created attachment 111089 [details]
attachment-5166-0.html

https://bugs.kde.org/show_bug.cgi?id=376565

--- Comment #10 from Bogdan Mihaila <bogdan.mihaila@gmx.de> ---
(In reply to Laurent Montel from comment #7)
> you report as it's not a specific kmail bug
> => please reassign to Qt or kf5

Hi Laurent

I'm having a hard time to find any of above in the product drop down list of
Bugzilla. Guess those just do not exist. I could reassign to
frameworks-ktextedit but not sure if that is any better? Would not want to
cause noise elsewhere if this might actually be fixed in Qt upstream.
Suggestions?


I think there is frameworks-kpart...
Comment 12 Stephan Sokolow 2020-05-26 09:44:34 UTC
For some additional information, here's what else I found.

First, https://tug.org/pipermail/texworks/2016q1/006411.html (a reply to a report of the same bug in Texworks 2015) claims that the mechanism by which the bug occurs is:

1. The underline's position is determined by the font
2. Depending on how the font positions the underline, it may get clipped away

Second, it's been reported as https://bugreports.qt.io/browse/QTBUG-50499 with the claim that the bug should be fixed in Qt 5.10 and a request for the fix to be tested that no one has responded to.

It's also observed that using `QTextCharFormat::WaveUnderline` instead of `QTextCharFormat::SpellCheckUnderline` functions as a workaround.

(Something I can confirm on Kubuntu 16.04 LTS's Qt 5.5.1 with the PyQt5 project that led me here, but it does so by being tall enough that only the bottom third of the wavy line is clipped away.)
Comment 13 avlas 2023-03-27 21:47:54 UTC
I cannot reproduce this anymore