<table><tr><td style="">ngraham 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/D13899">View Revision</a></tr></table><br /><div><div><p>This patch works as advertised, and does not imposes any visual regressions when using the Breeze color scheme.</p>

<p>And yet, when trying it out with non-default color schemes, I can't help but wonder what we're actually accomplishing here. While the Positive color message looks sane enough with all of the standard non-default color schemes that we ship with, the warning, negative, and informational message colors are all over the map, and a lot of them totally lose the intended meaning of the default colors. It becomes impossible to tell at-a-glance whether the message is good or bad or neutral by looking at its color, which is the whole point of using colors here. Here are some combinations that I think don't work at all:</p>

<p>Informational message, Oxygen:<br />
<a href="https://phabricator.kde.org/F6109824" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109824: Info - Oxygen.png</a></p>

<p>Informational Message, Steel:<br />
<a href="https://phabricator.kde.org/F6109826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109826: Info - Steel.png</a></p>

<p>Informational message, Wonton Soup:<br />
<a href="https://phabricator.kde.org/F6109829" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109829: Info - Wonton Soup.png</a></p>

<p>Informational message, Honeycomb:<br />
<a href="https://phabricator.kde.org/F6109831" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109831: Info - Honeycomb.png</a></p>

<p>Informational message, Norway:<br />
<a href="https://phabricator.kde.org/F6109833" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109833: Info - Norway.png</a></p>

<p>Negative Message, Steel:<br />
<a href="https://phabricator.kde.org/F6109838" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109838: Negative - Steel.png</a></p>

<p>Negative Message, Wonton Soup:<br />
<a href="https://phabricator.kde.org/F6109841" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109841: Negative - Wonton Soup.png</a></p>

<p>Warning message: Honeycomb:<br />
<a href="https://phabricator.kde.org/F6109845" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6109845: Warning - Honeycomb.png</a></p>

<p>It's very easy for me to imagine 3rd-party color schemes and user-created color schemes producing equally nonsensical and impossible-to-parse color combinations.</p>

<p>On one hand, that's their choice, right? But on the other hand, what do we gain from all this? What do the users gain?</p>

<p>Since Kirigami already uses colors from the theme, I guess we need to follow suit here to maintain consistency. But especially for informational messages, the message's background color seems just totally wrong for quite a lot of color schemes.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13899">https://phabricator.kde.org/D13899</a></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks, VDG<br /><strong>Cc: </strong>ngraham, aacid, broulik, kde-frameworks-devel, michaelh, crozbo, firef, bruns, skadinna, aaronhoneycutt, mbohlender<br /></div>