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

[kainz.a]

Generate the preview would be nice BUT really difficult cause you should
look that the plasmoid fits into an square.

about kwin screenshots I will have a look at if, if you like because also
the KCM HIG is defined like that:

Preview where you can add or select and an panel on buttom of the preview
with an advanced button. so yes the previews should be maintained as the
icons too. It cost a lot of time to make the app icons for each plasmoid.

we are going in the direction of using more previews according to the HIGs.
Also how the text above the preview is looking is according to our HIGs. So
....

I understand that the previews of an plasmoid could be outdated but than
the plasma theme was changed or the plasmoid get rewritten. We will add in
the HIG that each plasmoid have to have an 256x256px preview in there
package (GHNS) and that's it.

thanks
Andreas

2015-09-10 13:02 GMT+02:00 Martin Gräßlin <mgraesslin at kde.org>:

> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125134/
>
> On September 10th, 2015, 12:39 p.m. CEST, *Martin Gräßlin* wrote:
>
> I'm questioning the distribution of screenshots. Wouldn't it make more sense to generate it at runtime to show how it will look like (theme, proper dpi, etc.)?
>
> On September 10th, 2015, 12:48 p.m. CEST, *Marco Martin* wrote:
>
> i don't think it's feasible, it would have to load each single plasmoid in order to grab an image.
> that means either screenshots are missing until the first time it's loaded or a very slow and expensive process is done the first time plasma starts (or each time a plasmoid is added)
>
> in that case we should start now thinking about we prevent that the screenshots are outdated like the preview videos of kwin effects.
>
>
> - Martin
>
> On September 10th, 2015, 12:24 p.m. CEST, Marco Martin wrote:
> Review request for Plasma.
> By Marco Martin.
>
> *Updated Sept. 10, 2015, 12:24 p.m.*
> *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)
>
> View Diff <https://git.reviewboard.kde.org/r/125134/diff/>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150910/bbb3aed7/attachment.html>