Review Request 125135: show screenshots in the widget explorer

Fri Sep 11 15:24:05 UTC 2015

> On Sept. 10, 2015, 1:43 p.m., 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 :)
> 
> 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?)
> 
> 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.
> 
> Martin Klapetek wrote:
>     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.
> 
> andreas kainz wrote:
>     ok the proposal from the VDG (Uri and me)
>     
>     1. we will push 128x128px Previews in our kde:breeze icon set in breeze/icons/apps/128 in an first step we ONLY have screenshots
>     2. we will make 128x128px colorefull Icons and remove the screenshots with svg breeze and breeze dark icons where the user can recognize the difference behavior for the different widgets instead of something like this (https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/screenshot1.png)
>     
>     So in the first step you have to live with screenshots BUT in the end you get large widget icons where the user can recognize the widget.
>     
>     Also for GHNS most widgets will only have at lease screenshots so it will be always a mix between icons and screenshots, but as you know me will focus on feature complete.
>     
>     would this way work for you? For now the screenshots would work and before the screenshots are not up to date the screenshots will be removed with icons AND the widget icons are useing in the same behavior than all other icons so the screenshots are in our structure and we will care about it.
> 
> Marco Martin wrote:
>     that's ok for me, so it will have screenshots in the short term and ad hoc icons in the long term.
>     the icon names will be the same as the plugin name, so all desktop files will need to have an updated Icon=org.kde.plasma.analogclock entry.
>     screenshots inside the plasmoid package still supported, intended for kdelook plasmoids that won't have an icon in the breeze theme.
>     how does it sound?
> 
> Martin Klapetek wrote:
>     I don't want to discourage you from your effort, not at all, but isn't it kind of a wasted work? I mean, you spend all that time on making all those screenshots and then just remove then again anyway...? Wouldn't it be better to spend the time on the icons directly?
> 
> andreas kainz wrote:
>     No it would be much more usefull to use the screenshots instead of some app icons cause you add widgets cause you want to have the screenshot on your desktop. The Icons are the wast of time cause they doesn't make sense but make widget icons is better for the user than it is now.

I'm just worried that the screenshots will get outdated very soon. At which point the icon is indeed better for the user than shipping outdated screenshots.

- Martin

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

On Sept. 10, 2015, 12:53 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125135/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2015, 12:53 p.m.)
> 
> 
> Review request for Plasma and andreas kainz.
> 
> 
> 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 
> 
> Diff: https://git.reviewboard.kde.org/r/125135/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> snapshot6.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/10/73db8a25-adaa-4d18-9aa5-35f2c5156ad4__snapshot6.png
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150911/03b47e61/attachment.html>