<table><tr><td style="">filipf 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/D20086">View Revision</a></tr></table><br /><div><div><p>Good fix, the lack of implicitHeight was leading to confusion before if there are even any wallpapers present.</p>

<p>One question though: wouldn't it be better to define the height in <tt style="background: #ebebeb; font-size: 13px;">units.gridUnit</tt> instead of raw pixels, or even absolutely by doubling the height of one grid item? From my understanding, when scaling is changed the grid items are also resized so the window won't open showing 2 rows of wallpapers anymore, as was desired.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R133 KScreenLocker</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20086">https://phabricator.kde.org/D20086</a></div></div><br /><div><strong>To: </strong>tigrang, davidedmundson<br /><strong>Cc: </strong>filipf, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>