Summary: | update page gets freaky when scrolling past top | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-kirigami | Reporter: | Harald Sitter <sitter> |
Component: | general | Assignee: | Marco Martin <notmart> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nate, notuxius |
Priority: | NOR | ||
Version: | Master | ||
Target Milestone: | Not decided | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: |
video
flickering-scroll-past-top-discover flickering-scroll-past-top-discover-2 flickering-scroll-past-top-discover-3 |
Description
Harald Sitter
2017-07-10 10:17:30 UTC
Created attachment 106531 [details]
video
I cannot reproduce. Which version of Kirigami are you using? 2.2.0 This bug no longer appears for me in: Distribution: KDE neon Developer Edition Unstable Branches live iso 20170813 KDE Plasma Version: 5.10.90 KDE Frameworks Version: 5.37.0 Qt Version: 5.9.1 Kernel Version: 4.10.0-32-generic Discover: 5.10.90 Created attachment 107372 [details]
flickering-scroll-past-top-discover
Created attachment 107373 [details]
flickering-scroll-past-top-discover-2
Created attachment 107374 [details]
flickering-scroll-past-top-discover-3
I still managed to reproduce flickering - but not upon updates. It's reproducing sometimes upon regular navigating for me. Three recorder videos attached. It happens more often for me if scrolling with touchpad. Git commit 3c2d1e23e45c03321e110dabfe77c89d3c3e0703 by Marco Martin. Committed on 28/08/2017 at 13:28. Pushed by mart into branch 'master'. don't take into account the header 2 times when scrolling flickableItem.topMargin already comprises the header item, so the calculation where to stop scrolling was wrong, giving the erratic behavior Related: bug 383725 M +0 -4 src/controls/templates/private/ScrollView.qml https://commits.kde.org/kirigami/3c2d1e23e45c03321e110dabfe77c89d3c3e0703 can this be tried again in kirigami master after the above commit? Can't reproduce anymore in KDE neon kirigami-gallery 2.2.0+p16.04+git20170830 plasma-discover 5.10.5+p16.04+git20170825.1638-0 |