<table><tr><td style="">rjvbb 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/D5139" rel="noreferrer">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/D5139#100983" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D5139#100983</a>, <a href="https://phabricator.kde.org/p/flherne/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@flherne</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Note that red underlines are used for many non-spelling errors in KDevelop, such as syntax and semantic errors. There's no reason for the user to be surprised by that.</p></div>
</blockquote>

<p>Not for syntax and semantic errors, as they are comparable to what spelling errors are to natural language. Everyone who codes will understand that an underlined function probable indicates some other issue than a deviation from the usual spelling in language used to communicate with humans.</p>

<p>Commit messages *are* natural language, and spell (and grammar) checking would be very beneficial. Once you accept that you cannot reasonably continue to use underlining to indicate other issues.</p>

<p>I agree that my implementation isn't idea either. The simplest and least ambiguous indicator would be a pair of fixed vertical lines, a subdued one at the soft (title) length limit, a more salient one at the suggested hard limit. I haven't yet been able to figure out how draw such lines on the background.</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>I agree with mwolff; there's nothing wrong with underlining and your proposed changes seem far more confusing.</p></blockquote>

<p>The keyword being *seem*.</p>

<p>Is it to much to expect that people actually *test drive* a different implementation (for a not-insignificant duration) before emitting an opinion? It's well known that our perception is coloured by familiarity and can make us prefer things that are inferior or downright wrong.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5139" rel="noreferrer">https://phabricator.kde.org/D5139</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, mwolff<br /><strong>Cc: </strong>flherne, mwolff, kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>