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


+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?

- Bhushan Shah


On Sept. 10, 2015, 3:54 p.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, 3:54 p.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
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to