Refactor of OverlaySheet
ClosedPublic

Authored by mart on Mar 25 2020, 4:43 PM.

Details

Summary

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

Test Plan

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.
mart created this revision.Mar 25 2020, 4:43 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptMar 25 2020, 4:43 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Mar 25 2020, 4:43 PM
mart edited the test plan for this revision. (Show Details)Mar 25 2020, 4:47 PM

FWIW this needs a rebase as there were apparently some changes to OverlaySheet on master since this was branched.

mart updated this revision to Diff 78532.Mar 26 2020, 10:34 AM
  • Merge branch 'master' into mart/overlaysheetrefactor2
anthonyfieroni added inline comments.
src/controls/templates/OverlaySheet.qml
5–7

Keep SPDX.

mart updated this revision to Diff 78542.Mar 26 2020, 11:49 AM
  • better response to window resizes

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?

mart updated this revision to Diff 78616.Mar 27 2020, 9:27 AM
mart marked 4 inline comments as done.
  • adress comments
mart added a comment.Mar 27 2020, 9:30 AM

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.

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

In D28289#635601, @mart wrote:

I'm thinking to maybe *always* show the header with separator line and all even if empty when there is a close button...

Makes sense to me, yeah.

ngraham accepted this revision.Mar 27 2020, 1:44 PM
This revision is now accepted and ready to land.Mar 27 2020, 1:44 PM
mart updated this revision to Diff 79015.Apr 1 2020, 7:39 AM
  • always headere when there is close button
This revision was automatically updated to reflect the committed changes.