<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/D10325" 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/D10325#201884" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D10325#201884</a>, <a href="https://phabricator.kde.org/p/broulik/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@broulik</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Would it be possible to show it as if it were locked? That would solve all the issues with it, right?</p></blockquote>

<p>I don't get it. That "lock" feature is entirely a Dolphin invention. It does exactly what I do here:</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);">void DolphinDockWidget::setLocked(bool lock)
{
    ...
        if (lock) {
            ...
            setTitleBarWidget(m_dockTitleBar);
            setFeatures(QDockWidget::NoDockWidgetFeatures);</pre></div>

<p>with <tt style="background: #ebebeb; font-size: 13px;">m_dockTitleBar</tt> being a custom widget for some added padding</p></div>
</blockquote>

<p>Looks like i was looking at the wrong picture. I was looking as your <strong>after</strong> image and comparing that to the "locked" state in dolphin.<br />
The image i was expecting is the one you call "crammed at the top" :)</p>

<p>Imho, the "crammed at the top" version looks best as the "after" one just has some weird empty room above the panel now. But feel free to use the one you think fits best.</p>

<p>A suggestion though if you do choose for the "after" version. Would it be possible to rearrange the layout then?<br />
So:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">move the actions to the top, right above the panel.</li>
<li class="remarkup-list-item">move the location bar next to the actions</li>
</ul>

<p>I think that would look nice :)</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>To: </strong>broulik, Plasma, VDG, Frameworks, ngraham, mart<br /><strong>Cc: </strong>markg, ngraham, plasma-devel, michaelh, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>