Review Request 125134: Add a role for screenshots in the widgetexplorer model

[Marco Martin]

> On Sept. 10, 2015, 10:39 a.m., Bhushan Shah wrote:
> > +1 to idea in general
> > 
> > However I am not so sure about providing screenshot in package, given same plasmoid might look different in different form factor. What about having seperate data dir to provide the screenshots for plasmoid which can be extended later to provide say screenshots in different theme and/or different form factor?

yes, would be something very approximate, they think it's better than nothing tough
and yes, the screenshot may be made formfactor specific, as plasma package already supports platform specific content


- Marco


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125134/#review85095
-----------------------------------------------------------


On Sept. 10, 2015, 10:24 a.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125134/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2015, 10:24 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> As suggested by the VDG, add the possibility to have screenshots for plasmoids.
> the widget explorer will be modified as well to show the screenshots in the applet list when available
> 
> 
> Diffs
> -----
> 
>   components/shellprivate/widgetexplorer/plasmaappletitemmodel.cpp 99e9add 
>   components/shellprivate/widgetexplorer/plasmaappletitemmodel_p.h f9cc89f 
> 
> Diff: https://git.reviewboard.kde.org/r/125134/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150910/382cdc6e/attachment.html>