<table><tr><td style="">kfunk requested changes to this revision.<br />kfunk 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#100990" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D5139#100990</a>, <a href="https://phabricator.kde.org/p/rjvbb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@rjvbb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Is it to much to expect that people actually *test drive* a different implementation (for a not-insignificant duration) before emitting an opinion?</p></div>
</blockquote>

<p>What makes you think we don't do this?</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's well known that our perception is coloured by familiarity ... and can make us prefer things that are inferior or downright wrong.</p></blockquote>

<p>Please just underline the /problematic parts/ as it's being done *throughout* KDevelop in all text edits. You say "colored by familiarity", I say: please let's stay "consistent" with what we have. Using an italic font there definitely *is* confusing to existing users. Modifying the letter spacing is a no-go.</p>

<p>Note that Kate uses underlines for indicating spelling errors, kdev-python also uses underlines (yellow ones) to indicate style/formatting issues. So we're in line.</p>

<p>Please, René, it takes lot of energy to do these kind of reviews, it would be appreciated if you'd sometimes just accept the recommendations from core developers who've been working on KDevelop for so many years already. This patch could've been pushed (in a revised form) weeks ago, but now we just keep on discussing reinventing KDevelop's/KDE's human interface guidelines for a tiny text edit.</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, kfunk<br /><strong>Cc: </strong>kfunk, flherne, mwolff, kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>