<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/D19575">View Revision</a></tr></table><br /><div><div><p>I just noticed, there is already a <tt style="background: #ebebeb; font-size: 13px;">go-top</tt> icon that looks just like <tt style="background: #ebebeb; font-size: 13px;">collapse-all</tt>. That's a problem. "Collapse all" is different enough from "Go to top" that they shouldn't be the same icon.</p>

<p>I do think that the arrow with the bar on top is more appropriate for "Collapse all" though, so I think <tt style="background: #ebebeb; font-size: 13px;">go-top</tt> should be changed, along with the other <tt style="background: #ebebeb; font-size: 13px;">go-*</tt> icons. Maybe double arrows like KWin's "Keep above" button? Then we would need to decide if "Go to top" is similar enough to "Keep above" that they should be allowed to look the same.</p>

<p>From the VDG chat:</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>Any thoughts on <a href="https://phabricator.kde.org/D19575" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D19575</a> ? Should I make them match them to the titlebar ones which I think have a shorter top line, or do some symlinking action with arrow-up-double which is similar to these as well?</p></blockquote>

<p>They should match the current version of <tt style="background: #ebebeb; font-size: 13px;">go-top</tt>. The problem with matching the titlebar button is that it's not perfectly aligned with the grid. The titlebar button should probably be changed.</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/D19575">https://phabricator.kde.org/D19575</a></div></div><br /><div><strong>To: </strong>trickyricky26, VDG, ngraham, ndavis<br /><strong>Cc: </strong>ndavis, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>