<table><tr><td style="">loh.tar 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><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>
<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>
<li class="remarkup-list-item">The Config-GUI should show some hint in case of trouble, src/ui/configui.ui</li>
<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>
<li class="remarkup-list-item">Perhaps should the error message saved in a QString(List), retrievable later so it can be shown e.g by KTextEditor::Message</li>
<li class="remarkup-list-item">Perhaps should Loader::createSpeller return some "Error-Helper-Dictionary" instead of a nullptr, so that Sonnet::defaultLanguage may give something like "ERROR-de_AT_ost" instead of e.g "de_AT_ost".</li>
<li class="remarkup-list-item">Accordingly should Sonnet::preferredDictionaries filled with data like "ERROR-de_AT_ost"/"ERROR-Deutsch (Östereich something)"</li>
<li class="remarkup-list-item">See also Sonnet::DictionaryComboBox. There should then the error hints are be visible</li>
<li class="remarkup-list-item">Take a look at <a href="https://phabricator.kde.org/D18125" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18125</a>, why I think that may helpful. In the new button should then the hint possible to be shown "ERROR-foo".</li>
</ul>

<p>Not investigated if all of these is needed or possible.</p>

<p>PS: Typo in SUMMARY "..of the the dictionary.." 2x the</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>