From e321d1563e0d2971c7bfc420b722ebd67b8b077d Mon Sep 17 00:00:00 2001 From: Jan Mulder Date: Sun, 30 Sep 2018 20:42:28 +0200 Subject: QML UI: clip divedetails Listview A technically trival commit, but one with long story. This commit basically reverts dd1d90b5295b146e (1.5 year ago). While upgrading Kirigami after Kirigami commit 26b8bdea24c39, we suddenly have overlapping divelist and details pages in case they are both on the pageStack (this occurrs when navigating from divedetails to the divelist using breadcrumb navigation). At this point, its not clear (to me) if this the by design of Kirigami, or an unintended effect of the mentioned Kirigami commit. This all said. Simply clipping resolves our issue of overlapping pages, and it does not harm. Signed-off-by: Jan Mulder --- mobile-widgets/qml/DiveDetails.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'mobile-widgets/qml/DiveDetails.qml') diff --git a/mobile-widgets/qml/DiveDetails.qml b/mobile-widgets/qml/DiveDetails.qml index 2f8cf895a..3fab6d887 100644 --- a/mobile-widgets/qml/DiveDetails.qml +++ b/mobile-widgets/qml/DiveDetails.qml @@ -294,7 +294,7 @@ Kirigami.Page { orientation: ListView.Horizontal highlightFollowsCurrentItem: false focus: true - clip: false + clip: true snapMode: ListView.SnapOneItem highlightRangeMode: ListView.StrictlyEnforceRange onMovementEnded: { -- cgit v1.2.3-70-g09d2