<table><tr><td style="">ngraham 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/D10475" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@rkflx</a> Not naive at all!</p>

<p>In this patch, we are trying to solve the general case of how a navigation header with context actions should look. Once we implement this control in Discover, we will <em>definitely</em> be removing the title from the app page's use of it, since as you point out, it's redundant as the page content below it already shows the app name. We may also even remove the context actions and put them on the page too, in a non-scrolling header as you suggest. So this patch is really trying to work out the "worst case scenario" for a header with two navigation buttons, a title, and 2 or more context actions.</p>

<p>On another note, what would be your preference for title alignment: left-aligned, or centered?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10475" rel="noreferrer">https://phabricator.kde.org/D10475</a></div></div><br /><div><strong>To: </strong>apol, Kirigami, mart, ngraham, abetts<br /><strong>Cc: </strong>rkflx, mart, ngraham, plasma-devel, apol, davidedmundson, hein<br /></div>