<table><tr><td style="">markg 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/D8243" rel="noreferrer">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/D8243#157346" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D8243#157346</a>, <a href="https://phabricator.kde.org/p/renatoo/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@renatoo</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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/D8243#157336" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D8243#157336</a>, <a href="https://phabricator.kde.org/p/markg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@markg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>We have such a nice modular design with panels yet we keep cramming everything in the places panel... Why?<br />
 It's not just here, I've seen patches like this for Dolphin as well.</p>

<p>Imho, the places panel should only contain the top places - favorites if you will - that you might want to access quickly. Nothing else and should stay small.<br />
 Baloo -> new panel<br />
 Devices -> new panel<br />
 Remote devices -> Either in Devices or a new panel as well<br />
 etc..</p>

<p>Having more panels will give a whole different issues. Scrolling. Since each panel would be relatively small if you add many you will get scrollbars within the panels. Well, lets just get rid of the per panel scrollbar and merely have one for the whole "panel container" (like in macOS finder).</p>

<p>Just my 5 cents...</p></div>
</blockquote>

<p><a href="https://phabricator.kde.org/p/markg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@markg</a>, I never used macOs finder, but looking at the internet images. This is exactly what I am proposing and what we have on dolphin today. A single list with only one scrollbar split in sections.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">I am just trying to keep the same visuals on both (dolphin and file dialog) to avoid confusion for users,  they can get lost if you show different information.</pre></div>

<p>Our final idea is that you will be able to hide sections that you do not want, For example the local devices and network.</p></div>
</blockquote>

<p>You "intend" to, yes :)<br />
But with your changes you get the situation where you have multiple scrollbars if you have multiple panels enabled, one scrollbar in each panel.<br />
In Finder, you have 1 scrollbar for the whole panel area.</p>

<p>I like the goal! I want to get there as well! But it would be my preference to get there with individual panels. That would also make the code much simpler.<br />
Anyhow, that's just a "i would like..." opinion. It's not a critic to slow you down or stop this patch.</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/D8243" rel="noreferrer">https://phabricator.kde.org/D8243</a></div></div><br /><div><strong>To: </strong>renatoo, Frameworks, dfaure, ervin, VDG<br /><strong>Cc: </strong>markg, ngraham, mlaurent, ervin, anthonyfieroni, cfeck, Frameworks<br /></div>