On Wednesday 11 August 2010, Aaron J. Seigo wrote: > On August 11, 2010, Aaron J. Seigo wrote: > > which brings me to the next issue: sharing the previewer. this could be > > done by sharing the delegate that paints the items via Plasma::Wallpaper, > > requiring a view with a model that provides the BackgroundDelegate roles. > > actually, looking at the code, it seems that it would be better to just > provide the whole view and hide the delegate. this will allow standardizing > things such as icon vs list and what not. i'm a little concerned about > sharing this via libplasma though ... it would mean a couple more private > classes and one more method in Plasma::Wallpaper specific to configuration > UI which would return a QAbsractItemView or some-such. > > i thought about making it possible for a Wallpaper plugin to provide a > QAbastractModel, but the Wallpaper really needs to be able to decide where > to put the listing in its configuration.
yeah, a bit too much public api i think... also i'm not too happy about returning a qwidget based qabstractitemview from libplasma, what about providing models in plasmagenericshell, that use wallpaper to render thumbnails in pixmaps that would be provided in the model? -- Marco Martin _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
