D8982: Qml controls for kcm creation

[David Edmundson]

davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  I like the GridView
  
  Note that we'll want to use that from the WallpaperDialog in Plasma, which isn't a KCM. I don't think it will be a problem though ?
  
  We are missing docs everywhere. Every public type and property should have some.
  Also please check the years in your copyright.

INLINE COMMENTS

> GridView.qml:24
> +
> +QtControls.ScrollView {
> +    id: scroll

How do you use this and set the model & delegate?

> SimpleKCM.qml:23
> +
> +Kirigami.ScrollablePage {
> +    id: root

System settings adds it too for the KCM
KCMMultiPageDialog (kcmshell5) adds a scroll area for the KCM

So isn't this adding a QtQuick scrollarea within a parent widget scrollarea?

REPOSITORY
  R296 KDeclarative

REVISION DETAIL
  https://phabricator.kde.org/D8982

To: mart, #plasma, hein, davidedmundson
Cc: davidedmundson, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171124/69a5c807/attachment.html>