> On Jan. 22, 2014, 2:32 p.m., Marco Martin wrote: > > well, actually ContainmentInterface of the qml scruiptengine does use it, > > to store wether loading and draw a wallpaper or not. > > > > The issue is to keep it in Containment, or having it only in > > ContaimentInterface. > > Either choice is fine with me... > > Personally i would limit the amount of api that is only in > > containmentinterface and not containment (basically the concept now is that > > Applet and Containment are models for appletintterface/containmentinterface)
I agree. appletinterface and containmentinterface should stay as small as possible and only have things in there that are specific for the QtQuick implementation. That reduces boilerplate and keeps Plasma::Applet and Plasma::Containment useful. - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115224/#review48028 ----------------------------------------------------------- On Jan. 22, 2014, 2:24 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115224/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 2:24 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > Remove unused property drawWallpaper > > As suggested here: > http://community.kde.org/Plasma/libplasma2/API_Review/Containment > kde-workspace doesn't use it. > > > Diffs > ----- > > src/plasmaquick/plasmaquickview.cpp 03fe00e > src/scriptengines/qml/plasmoid/containmentinterface.h 0ed5868 > src/scriptengines/qml/plasmoid/containmentinterface.cpp 23edb67 > src/plasma/containment.h 1d747c6 > src/plasma/containment.cpp 590402a > src/plasma/corona.cpp 9a937b0 > src/plasma/private/containment_p.h 597f26e > src/plasma/scripting/appletscript.h 65301d4 > src/plasma/scripting/appletscript.cpp cb9df7d > > Diff: https://git.reviewboard.kde.org/r/115224/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel