Rework OverlaySheet to remove some sizing hacks that made ListView and GridView
not work well, fix some appearance issues with the new scrollbar, make it easier
to dismiss sheets via swipe.
the internal flickabe is never interactive, flicking is only managed by an
external flickable that manages both normal flicking and dismissal swipe
Details
- Reviewers
ngraham - Group Reviewers
Kirigami - Commits
- R169:800fd5b04c0e: Refactor of OverlaySheet
Before:
After:
Diff Detail
- Repository
- R169 Kirigami
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
FWIW this needs a rebase as there were apparently some changes to OverlaySheet on master since this was branched.
src/controls/templates/OverlaySheet.qml | ||
---|---|---|
5–7 | Keep SPDX. |
Generally much nicer looking. I found one visual issue: when I switch from touchpad scrolling to touchscreen scrolling, the close button in the corner gets overlapped by the content area since the scroll track gets narrower.
src/controls/OverlaySheet.qml | ||
---|---|---|
22–23 | Remove commented code | |
src/controls/templates/OverlaySheet.qml | ||
4 | Is this intentional? | |
266 | I thought we weren't supposed to multiply duration values. :) is the new veryLongDuration not long enough? do we need superDuperLongDuration? | |
351 | what's this +1 for? |
in general the x button when there is not an header is giving some problems, uglyness and complications.
I'm thinking to maybe *always* show the header with separator line and all even if empty when there is a close button...
src/controls/templates/OverlaySheet.qml | ||
---|---|---|
4 | ah, sorry no derives from conflicts resolution | |
266 | was a debug thing forgotten in, sorry :) | |
351 | to make the view always scrolling, or the swipe to dismiss can get broken |