Review Request 125135: show screenshots in the widget explorer

[kainz.a]

and when the Screenshot was only shown when the Screenshot is e.g. 6 month
old.
Am 10.09.2015 15:36 schrieb "Martin Klapetek" <martin.klapetek at gmail.com>:

> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125135/
>
> On September 10th, 2015, 1:43 p.m. CEST, *Kai Uwe Broulik* wrote:
>
> First of all, I like the idea.
>
> However, I think we should go with PNG with alpha channel rather than having the Breeze default wallpaper there which changes every release. Showing a fully working Plasmoid is probably too much in this list in terms of performance but might be worth thinking allowing a plasmoid to provide a demo.qml (eg.  analog clock's face is pretty isolated) which is then shown there and in sync with the actual Plasmoid by using parts of it.
> On the other hand, Android also just shows screenshots :)
>
> On September 10th, 2015, 1:51 p.m. CEST, *Marco Martin* wrote:
>
> i think as well having an alpha channel would be much prettier, however would require a specialized tool and would be slightly harder to do, needs to be as easy and fast for the VDG people to do them quickly. (also applets that make sense only as an expanded popup like kickoff would work better with the wallpaper i think?)
>
> On September 10th, 2015, 2:06 p.m. CEST, *Martin Gräßlin* wrote:
>
> maybe we could get an easy "screenshot mode" into plasmoidviewer? So that it
> - hides the background
> - uses correct size
> - saves it with a standardized name
>
> Kickoff exposes another problem: some applets will need KWin help. Taking the screenshot without a background won't work for them, though the background hardly matters if we get the area around as translucent.
>
> I too am worried about the screenshots getting out of date. Just for this release the digital-clock got some serious overhaul in its looks. Now I'm not opposed to the idea itself, but I don't think this scales too well. This would need a dedicated person to review _all_ applets before release and compare the screenshots...and from experience I don't believe we can keep this up.
>
>
> - Martin
>
> On September 10th, 2015, 12:53 p.m. CEST, Marco Martin wrote:
> Review request for Plasma and andreas kainz.
> By Marco Martin.
>
> *Updated Sept. 10, 2015, 12:53 p.m.*
> *Repository: * plasma-desktop
> Description
>
> As requested by the VDG, show screenshots in the widget explorer when available, to have a look similar to the activity manager (they'll do screenshots for all our plasmoids).
> when a screenshot is not available, it falls back to the old design.
> this is an in-between design, the latest design would be with two columns instead of one, but that would need to be tested as well beforehand, both in the widget explorer and the activity switcher
>
> Diffs
>
>    - desktoppackage/contents/explorer/AppletDelegate.qml (7ded06e)
>
> View Diff <https://git.reviewboard.kde.org/r/125135/diff/>
> File Attachments
> - snapshot6.png
> <https://git.reviewboard.kde.org/media/uploaded/files/2015/09/10/73db8a25-adaa-4d18-9aa5-35f2c5156ad4__snapshot6.png>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150910/e4d03bd2/attachment-0001.html>