<table><tr><td style="">dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26342">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26342#675180" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D26342#675180</a>, <a href="https://phabricator.kde.org/p/aacid/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@aacid</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26342#675164" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D26342#675164</a>, <a href="https://phabricator.kde.org/p/sdepiets/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@sdepiets</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>I don't think that's a regression, in the previous behavior you could try to set any language to proofread, it would always auto-detect "Bonjour" as French, thus the "Tools / Spelling / change language" had not effect if autodetect was enabled at system level (while autodetection should be an application or even case by case decision).</p></div>
</blockquote></div>
</blockquote>
<p>Autodetection *is* an application setting, it's in Settings / Spellchecker... / [x] Enable autodetection of language</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>It may not seem like an issue for simple cases, but actually for mixed contents (i.e. an email that is 50% French, 50% English) that would be detected as English, you would have no way at all to check the French text without disabling system-wide autodectection.</p></blockquote></blockquote>
<p>In my experience a mixed email was perfectly handled, with each paragraph having a different language.<br />
See <a href="http://www.davidfaure.fr/2020/Screenshot_20200613_194827.png" class="remarkup-link" target="_blank" rel="noreferrer">http://www.davidfaure.fr/2020/Screenshot_20200613_194827.png</a></p>
<p>But I think you're talking about the spellchecking dialog (which I never use, except this month because of the need to find a workaround).<br />
My concern is for autodetection and background spellchecking. If a fix for the spellcheck dialog introduces a change of behaviour for the background spellchecking, that's a regression.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Calling setAutoDetectLanguageDisabled(false) restores the previous behavior</p></blockquote></blockquote>
<p>Changes in KF5 should NOT require applications to adapt in order to restore previous *working* behaviour.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The problem here is that David probably never set any language on purpose.</p></blockquote>
<p>Right.<br />
In fact, if I open the settings dialog, it has "français" checked in the top listview, and only that one.<br />
Meanwhile the Tools / Spelling dialog shows "American English (UnitedStates)" in the combobox.<br />
This particular issue happens even with this patch reverted. Something's rather broken.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>you could argue that this is a deficiency of the UI, should have a checkbox for "enable spellchecking" and another for "enable spellchecking *exactly* in this language"</p></blockquote>
<p>Not sure I understand this.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>And this is were we failed, we changed the default behaviour and that's probably not the greatest of the ideas in retrospect. Even if we could not see why anyone would set a language and would still want the auto language detection to be enabled, well it seems that at least David wanted because it's been the behaviour for ages :D</p></blockquote>
<p>*I* don't really set a language. Maybe KMail does though. <a href="https://phabricator.kde.org/p/mlaurent/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@mlaurent</a> might know more?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R246 Sonnet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26342">https://phabricator.kde.org/D26342</a></div></div><br /><div><strong>To: </strong>sdepiets, Frameworks, cullmann, mlaurent, mludwig, aacid<br /><strong>Cc: </strong>dfaure, aacid, mludwig, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>