<table><tr><td style="">niccolove 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/D25337">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/D25337#563329" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25337#563329</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>If we do this, I'd like to see the background color changed to pure black so we don't regress contrast too much (or at all). This could go along with the proposal to do the same thing for text too; if so, we should land them at the same time rather than doing this in an un-coordinated manner.</p></div>
</blockquote>

<p>I'm not too sure about this, mainly because the background currently uses the breeze dark colors, and would probably look not as good and consistent with black. The text could be made totally white instead of #fcfcfc, but that would not be that much of a difference. On the other hand, this patch does not change the background color by much, especially on dark-y wallpapers.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25337">https://phabricator.kde.org/D25337</a></div></div><br /><div><strong>To: </strong>niccolove, Konsole, VDG<br /><strong>Cc: </strong>filipf, ngraham, konsole-devel, fbampaloukas, thsurrel, maximilianocuria, hindenburg<br /></div>