<table><tr><td style="">mludwig 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#619785" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26342#619785</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);"><p>without having used Sonnet much, this seems the wrong API to me.</p>

<p>Are you saying that it can happen that you tell Sonnet "use this language" and it goes and say "nah i'll ignore you and do my thing".</p></div>
</blockquote>

<p>Yep, that's right, and it won't even tell you which language it has detected. When the autodetection feature was introduced, it fundamentally changed the behaviour of Sonnet, breaking KatePart's spell checking in the process.</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>It seems to me that what would make sense is that the "use this language" is what sets autoDetectLanguageDisabled to false, or at least the function that should get a new overload saying setLanguage(¿qstring? language, bool disableAutoDetection) if we want to be sure not to change existing behaviour</p></blockquote>

<p>Ideally, the autodetection feature should be redesigned as it is only usable for simple text fields in its current form. KatePart requires a more sophisticated way of detecting languages(s).</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<br /><strong>Cc: </strong>aacid, mludwig, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>