<table><tr><td style="">ngraham created this revision.<br />ngraham added a reviewer: VDG.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />ngraham requested review of this revision.
</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/D15931">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The current Breeze icon used for the default <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Network</span></span></span>(<tt style="background: #ebebeb; font-size: 13px;">network-workgroup</tt>) is not ideal for several reasons:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">It it not clear what it depicts (I was told it's a server rack, but there's not enough detail to make this apparent)</li>
<li class="remarkup-list-item">Even if the visuals were clear, its iconography (a server rack) is not familiar to people who are not sysadmins</li>
</ul>

<p>This patch replaces it with <tt style="background: #ebebeb; font-size: 13px;">folder-network</tt>, which yields the following improvements:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">This icon includes the "sideways dotted V" imagery that is used in other contexts (shared folder emblem, <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Share</span></span></span> menu, which teaches the user its meaning</li>
<li class="remarkup-list-item">Many other platforms use circular globe or world iconography for their network icons. Even though this new icon is not a globe or world, it is circular, which helps to re-use some of a user's prior experience on other platforms</li>
<li class="remarkup-list-item">Semantically, <tt style="background: #ebebeb; font-size: 13px;">folder-network</tt> is just a more appropriate icon to use for this than <tt style="background: #ebebeb; font-size: 13px;">network-workgroup</tt>. The item leads to the whole network, not a specific workgroup.</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>New icon shows up in all Places panels for new user accounts, or if you delete <tt style="background: #ebebeb; font-size: 13px;">~/.local/share/user-places.xbel</tt> in an existing user account</p>

<p>Dolphin Places panel:<br />
[image goes here]</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>better-network-icon (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15931">https://phabricator.kde.org/D15931</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/kfileplacesmodeltest.cpp<br />
src/filewidgets/kfileplacesmodel.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>