<table><tr><td style="">ahmadsamir 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/D18317">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/D18317#395928" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18317#395928</a>, <a href="https://phabricator.kde.org/p/loh.tar/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@loh.tar</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Just my thoughts:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I think there shouldn't be the (default) dictionary changed by some smart logic. Just hint the user that the setting is not applicable.</li>
</ul></div>
</blockquote>

<p>The user will open the sonnet config dialog and see that the dictionary he wants is already selected (e.g. the dictionary files have just been renamed, but it's the same dictionary); that is confusing, as can be gleaned from the linked bug report.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">To set the dict to the system locale seems to me the less smartest trick. If everybody want such "auto-fix" should then the bad setting investigated and tried to find some similar setting, e.g. Bad "de_AT_ost" -> "de_AT" -> "de_DE". Why? Someone may have a locale of "de_DE" but a dict setting "en_US" for whatever reason. Besides I guess Loader::createSpeller is not only called when the default dict will loaded. IIRC has Sonnet some functionallity to guess a language and choose a fitting dict (May that help?)</li>
</ul></blockquote>

<p>It's what sonnet settings default to initially:<br />
<a href="https://cgit.kde.org/sonnet.git/tree/src/core/settings.cpp#n291" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/sonnet.git/tree/src/core/settings.cpp#n291</a>:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">d->defaultLanguage = settings.value(QStringLiteral("defaultLanguage"),
                                        QLocale::system().name()).toString();</pre></div>

<p>Language auto detection seems to cause some havoc: <a href="https://bugs.kde.org/show_bug.cgi?id=386611" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=386611</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">The Config-GUI should show some hint in case of trouble, src/ui/configui.ui</li>
</ul></blockquote>

<p>[..]</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Don't overwrite permanently some bad setting with a new value. Perhaps has the user just set up a new system and only missed to install some package</li>
</ul></blockquote>

<p>Same problem as above, the user opens the config dialog and the dictionary he wants is already selected....</p>

<p>[...]</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/D18317">https://phabricator.kde.org/D18317</a></div></div><br /><div><strong>To: </strong>ahmadsamir, sandsmark, loh.tar<br /><strong>Cc: </strong>pino, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>