ColumnView and partial C++ refactor of PageRow
ClosedPublic

Authored by mart on Feb 21 2019, 3:12 PM.

Details

Summary

ColumnView is a container component which encapsulates most of the logic from PageRow
(minus the instantiation of components from paths which stays in PageRow)
with that, new features are possible, such as having expanded pages which aren't the last one
or inserting/removing arbitrary pages, to make it behave not like a stack.
This solves also some glitches as ListView is quite buggy.

Test Plan

tested with Kirigmai Gallery and some other apps such as the new PureMaps which needs some of
the new features.
By normal usage, no visual change should occur.

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.Feb 21 2019, 3:12 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptFeb 21 2019, 3:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Feb 21 2019, 3:12 PM
mart updated this revision to Diff 52799.Feb 28 2019, 11:09 AM
  • a bit more drag distance to be sure not to conflict with horizontal
  • preventStealing attached property
  • hide items when removed
  • fix item position mapping
  • signal the visible items when forgetting one
  • update currentItem too when removing items
  • Merge branch 'master' into mart/columnsView
nicolasfella accepted this revision.Feb 28 2019, 12:02 PM
nicolasfella added a subscriber: nicolasfella.

Works fine with KDE Connect

This revision is now accepted and ready to land.Feb 28 2019, 12:02 PM
This revision was automatically updated to reflect the committed changes.