You should not be able to drag the contents outside the view, it looks weird. This fixes it.
Details
- Reviewers
davidedmundson - Group Reviewers
Plasma Frameworks VDG
Open the Fonts KCM and try dragging the contents outside the view. It should not be possible anymore.
Diff Detail
- Repository
- R296 KDeclarative
- Branch
- prevent-dragging-simplekcm-contents-outside-view (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 15333 Build 15351: arc lint + arc unit
+1 visually
If the behavior is important on mobile let's keep it there of course, but on desktop it feel out of place.
+1, but maybe this doesn't go far enough. How about doing it for ScrollablePage instead? That would fix https://bugs.kde.org/show_bug.cgi?id=403837 as well as this.
I will fix it there, but it won't disable click and drag to scroll the view if it's scrollable. It just disables click and drag scroll when there is no scrollbar and disables the scroll overshoot on desktop.
Right, that's fine.
and disables the scroll overshoot on desktop.
This would disable it when using a touch swipe or a touchpad on the desktop too though, right? That doesn;t seem desirable. I think we on;y want scroll overshoot disabled when using a scroll wheel; for all other cases, IMO it should be on.
This would disable it when using a touch swipe or a touchpad on the desktop too though, right? That doesn;t seem desirable. I think we on;y want scroll overshoot disabled when using a scroll wheel; for all other cases, IMO it should be on.
That's how it is now, but click and drag lets you overshoot infinitely.