<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/D29137">View Revision</a></tr></table><br /><div><div><p>Thanks! Implementation-wise, I would have re-arranged things a bit to have only one header for the <tt style="background: #ebebeb; font-size: 13px;">PC3.Page</tt>, with the contents changing depending on the page in the stack, rather than your approach of giving each page in the stack its own header, but that works fine too and it's perfectly valid so I'll accept it.</p>

<p>Everything works great. Just one thing: I feel like we need a <tt style="background: #ebebeb; font-size: 13px;">units.smallSpacing</tt> bottom margin underneath the headers for both pages.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29137">https://phabricator.kde.org/D29137</a></div></div><br /><div><strong>To: </strong>niccolove, VDG, ngraham<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>