<table><tr><td style="">nicolasfella created this revision.<br />nicolasfella added reviewers: Kirigami, mart.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />nicolasfella requested review of this revision.
</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/D29308">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In this example</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">import QtQuick 2.5
import QtQuick.Layouts 1.1
import org.kde.kirigami 2.4 as Kirigami

Kirigami.ScrollablePage {

    Rectangle {
        width: parent.width
        height: 1000
        color: "red"
    }
}</pre></div>

<p>the gap between the rectangle and the scrollbar was larger than the distance between the left edge of the rectangle and the window. This was caused by the extra rightPadding of the RefreshableScrollView. It also seems unnecessary to me since the ScrollView itself takes care of dodging the scroll bar (ScrollView.qml:21).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>margin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29308">https://phabricator.kde.org/D29308</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/controls/ScrollablePage.qml</div></div></div><br /><div><strong>To: </strong>nicolasfella, Kirigami, mart<br /><strong>Cc: </strong>plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart<br /></div>