<table><tr><td style="">ndavis 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/D11880">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/D11880#330788" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11880#330788</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>I like <tt style="background: #ebebeb; font-size: 13px;">firewall-config</tt> and <tt style="background: #ebebeb; font-size: 13px;">firewall-applet-error</tt> as they are.</p>

<p>I think that <tt style="background: #ebebeb; font-size: 13px;">firewall-applet</tt> looks maybe a bit too plain at its 22px size. The wall seems to need something.</p></div>
</blockquote>

<p>Maybe the emblem-checked icon should go in the lower right corner to match the style of the other two firewall-applet icons? I choose to make it fairly simple so that it would normally fit in with other icons in the system tray except for when there is an error or Panic Mode is enabled.</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 like the lock on <tt style="background: #ebebeb; font-size: 13px;">firewall-applet-panic</tt>. I'm not sure about the orange color though. Orange means "warning". If I'm understanding you correctly, <tt style="background: #ebebeb; font-size: 13px;">firewall-applet-panic</tt> should evoke feelings of maximum safety.</p></blockquote>

<p>Yes, but I also agree with <a href="https://phabricator.kde.org/p/svenmauch/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@svenmauch</a> about the previous version being too positive. If a user enables panic mode by accident, how will they know something might be wrong with their settings? As it is, firewall-applet-panic is the firewall-applet icon with the emblem-locked icon in the corner and I've applied the color of emblem-locked to the wall. While orange may mean "warning", it could mean any kind of warning. In this case, it's a warning that an extreme setting is being used, but there is a lock in the corner to show that it is at least safe.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>BRANCH</strong><div><div>firewalld-icons (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11880">https://phabricator.kde.org/D11880</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, Breeze, ngraham<br /><strong>Cc: </strong>bruns, abetts, alex-l, svenmauch, kde-frameworks-devel, ngraham, michaelh<br /></div>