<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/D16905">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/D16905#360586" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16905#360586</a>, <a href="https://phabricator.kde.org/p/trickyricky26/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@trickyricky26</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><ul class="remarkup-list">
<li class="remarkup-list-item">Fix naming of ascending and descending versions; make ascending versions less confusing; fix symlink sort-name
<br /><br />
I have flipped the naming of the ascending and descending versions to be correct. The arrow in the ascending versions is now always at the top and does not cut off lines. The symbolic link <tt style="background: #ebebeb; font-size: 13px;">sort-name</tt> was changed from <tt style="background: #ebebeb; font-size: 13px;">view-sort-ascending</tt> to <tt style="background: #ebebeb; font-size: 13px;">view-sort-ascending-name</tt></li>
</ul></div>
</blockquote>

<p>I hate to have you going back and forth, but are you (and anyone who suggested you do this) sure that this is right?<br />
<a href="https://phabricator.kde.org/F6428508" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6428508: Screenshot_20181117_064300.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>BRANCH</strong><div><div>add-sort-options-icon (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16905">https://phabricator.kde.org/D16905</a></div></div><br /><div><strong>To: </strong>trickyricky26, VDG, Breeze, ndavis<br /><strong>Cc: </strong>cfeck, emateli, ndavis, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>