<table><tr><td style="">sharvey marked 13 inline comments as done.<br />sharvey added inline comments.
</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/D12855">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12855#inline-66294">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ngraham</span> wrote in <span style="color: #4b4d51; font-weight: bold;">WidgetExplorer.qml:235</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I mean, could we do something like <tt style="background: #ebebeb; font-size: 13px;">newSearchRow.height = searchInput.height + (units.smallSpacing * 2)</tt> or something like that? Would that work?</p>

<p style="padding: 0; margin: 8px;">Now that I think about it, instead of conditionally adjusting the height in this if/else block, could we just have this toggle the <tt style="background: #ebebeb; font-size: 13px;">showingSearch</tt> property? Does that not work? In fact, I wonder why we need the <tt style="background: #ebebeb; font-size: 13px;">onClicked</tt> as well as <tt style="background: #ebebeb; font-size: 13px;">onCheckedChanged</tt> lines at all. Perhaps those could be simplified into just one.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">A method to my madness! The size is there so the search bar announces its arrival by expanding the row, pushing the widget grid downward, and appearing. And then again in reverse.</p>

<p style="padding: 0; margin: 8px;">Otherwise, there's just an empty space there, waiting for the search box to appear.</p>

<p style="padding: 0; margin: 8px;">It's not much eye candy, but just a little bit.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12855">https://phabricator.kde.org/D12855</a></div></div><br /><div><strong>To: </strong>sharvey, ngraham, davidedmundson<br /><strong>Cc: </strong>abetts, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>