This revision was automatically updated to reflect the committed changes.
Closed by commit R169:518444e4191a: make handle follow arbitrary height of the
drawer (authored by camiloh).
REPOSITORY
R169 Kirigami
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D22648?vs=62345=62535
camiloh updated this revision to Diff 62345.
camiloh added a comment.
now using root.parent instead of applicationWindow()
REPOSITORY
R169 Kirigami
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D22648?vs=62309=62345
BRANCH
handle-follow-drawer (branched from master)
REVISION
mart added inline comments.
INLINE COMMENTS
> OverlayDrawer.qml:210
> +if(root.height < applicationWindow().height) {
> +margin = applicationWindow().height - root.height -
> root.y + Units.smallSpacing;
> +}
applicationWindow() should be
camiloh updated this revision to Diff 62309.
camiloh added a comment.
take into consideration the y position
Summary: if the drawer height prop has been set to a custom value, might as
well the y position too, so take it into account to position the handle
REPOSITORY
R169 Kirigami
camiloh created this revision.
camiloh added a reviewer: mart.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
camiloh requested review of this revision.
REVISION SUMMARY
If the height of the drawer is set to a different value from the
applicationWindow height, then