<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG, Frameworks.<br />Restricted Application added projects: Plasma, Frameworks.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik 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/D10325" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This avoids a double "Places" header and also gets rid of the superfluous frame around the widget, similar to Dolphin. It has never been possible to detach the dock from the window anyway.<br />
The close button is also removed as a side-effect of this but should a user really want to hide it, this can still be done by pressing F9 or in the menu. Resizing the places list is still possible.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Before<br />
<a href="https://phabricator.kde.org/F5693291" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5693291: Screenshot_20180205_161053.png</a><br />
After<br />
<a href="https://phabricator.kde.org/F5693292" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5693292: Screenshot_20180205_161002.png</a></p>

<p>I just noticed in Oxygen it always paints a frame around it and you end up with a lot of padding at the top.<br />
When I completely remove the header item, it looks strange with the places crammed at the top:<br />
<a href="https://phabricator.kde.org/F5693295" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5693295: Screenshot_20180205_161901.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10325" rel="noreferrer">https://phabricator.kde.org/D10325</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kfilewidget.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, Frameworks<br /><strong>Cc: </strong>plasma-devel, michaelh, ZrenBot, ngraham, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>