<table><tr><td style="">ngraham accepted this revision as: VDG, ngraham.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D23049">View Revision</a></tr></table><br /><div><div><p>Thanks! We definitely need <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a>'s input on the right implementation of these details. But visually it looks great to me in System Settings (working on patches for the current users of these section headers), and but +1 on the concept of course.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>add-listsection-component (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23049">https://phabricator.kde.org/D23049</a></div></div><br /><div><strong>To: </strong>GB_2, Kirigami, VDG, mart, ngraham<br /><strong>Cc: </strong>ognarb, davidre, ngraham, VDG, plasma-devel, Kirigami, fbampaloukas, domson, dkardarakos, apol, davidedmundson, mart, hein<br /></div>