<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/D20708">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/D20708#453690" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20708#453690</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>Overall very nice.</p>

<p>Instead of symlinking <tt style="background: #ebebeb; font-size: 13px;">input-mouse</tt> to <tt style="background: #ebebeb; font-size: 13px;">dialog-input-devices</tt>, I think it might make more sense to rename <tt style="background: #ebebeb; font-size: 13px;">dialog-input-devices</tt> to be <tt style="background: #ebebeb; font-size: 13px;">input-mouse</tt> and then change the <tt style="background: #ebebeb; font-size: 13px;">dialog-input-devices</tt> so that it depicts more than one input device, to reinforce its name. Maybe a mouse + keyboard?</p></div>
</blockquote>

<p>I don't think I can fit a whole keyboard, but I can probably fit a numpad.</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>While you're thinking about input device icons, we also have a few outstanding bugs:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The colorful version of the mouse icon is hard to see on a dark background, and also it maybe should have a more generic appearance rather than looking like a Razer mouse: <a href="https://bugs.kde.org/show_bug.cgi?id=406453" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=406453</a></li>
</ul></blockquote>

<p>That will be a lot of work and I don't want to do that in this diff since it'll slow the rest of the changes down considerably. The most important change in this diff is that this is fixed (22px icon where 16px should be): <a href="https://phabricator.kde.org/F6786267" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6786267: Screenshot_20190421_215747.png</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">We need a joystick icon that actually looks like a joystick: <a href="https://bugs.kde.org/show_bug.cgi?id=406679" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=406679</a></li>
</ul></blockquote>

<p>I'll do that with the 64px mouse icon fix when I get around to it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20708">https://phabricator.kde.org/D20708</a></div></div><br /><div><strong>To: </strong>ndavis, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>