----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125135/#review85105 -----------------------------------------------------------
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 :) - Kai Uwe Broulik On Sept. 10, 2015, 10:53 vorm., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125135/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 10:53 vorm.) > > > 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
