<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>My observations:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">There is no hint when you activate auto spell check (yes, it's out of this scope)</li>
<li class="remarkup-list-item">Before showing the combobox by "Change Dictionary" the popup appears</li>
<li class="remarkup-list-item">The combobox shows then some other setting, here it was the first entry</li>
<li class="remarkup-list-item">When open the config dialog the new popup block the processing. So the expected dialog is only shown after user action</li>
</ul>

<p><a href="https://phabricator.kde.org/F6576785" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6576785: 1548859202.png</a></p>

<p>My suggestion:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">For me are extra popups annoying. I would suggest to avoid them</li>
<li class="remarkup-list-item">Add some hint embedded to the Sonnet config dialog. IIRC I have seen in some config dialog a similar hint like these KTextEditor::Message, but I may wrong</li>
<li class="remarkup-list-item">Perhaps should the combobox shown an empty entry or an error hint instead of some (not expected) value</li>
<li class="remarkup-list-item">How about to emit the full message, not only the missed language? This way is no need to formulate everywhere an own error text</li>
</ul>

<p>Todo:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">A patch to KTextEditor to show the hint as KTextEditor::Message when some spell check is requested</li>
</ul></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>