<table><tr><td style="">mglb 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><p>Default text color and this new background blended with white still has good contrast, but it doesn't look so good when colored text is used (not that it looks perfect now).<br />
Also, please consider how it looks with transparency but without blur (as it does not work everywhere). It can be distracting.<br />
Konsole is sometimes used for hours for looking at text, so it have to be readable by default.</p>

<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#563442" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25337#563442</a>, <a href="https://phabricator.kde.org/p/niccolove/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@niccolove</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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.</p></div>
</blockquote>

<p>View backgrounds are not transparent, so this breaks consistency anyway (see Konsole panel in e.g. Kate, Dolphin, KDevelop).</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>On the other hand, this patch does not change the background color by much, especially on dark-y wallpapers.</p></blockquote>

<p>In real use cases people use more than one window and don't see wallpaper that much (even behind the window). Sometimes the other window is e.g. white web page or some pdf. Anyway, default wallpaper is far from dark.<br />
Maybe try a color which gives Charocal Grey (breeze dark view background) when blended with white (i.e. #0b0e12)?</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>mglb, filipf, ngraham, konsole-devel, fbampaloukas, thsurrel, maximilianocuria, hindenburg<br /></div>