Review Request: Show wallpaper resolution if possible in selection dialog

Aaron J. Seigo aseigo at kde.org
Sat Jan 17 18:56:22 CET 2009


On Saturday 17 January 2009, Beat Wolf wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/332/

patch seems broken.

> Partially solves the wish: http://bugs.kde.org/show_bug.cgi?id=177087
> If possible, shows the resolution of a wallpaper. This seems not really
> possible for wallpaper packages (comming with different resolutions).

you should be able to do this by showing the resolution of the wallpaper that 
would actually get used. there is already a BackgroundPackage::findBackground 
that returns a path in wallpapers/image/backgroundpackage.cpp. a similar 
method could return a QSize of the resolution?

> In the screenshot, only the last wallpaper is a single image where the
> resolution can be shown.

i think this would be useful, yes.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20090117/8dba0ff7/attachment.sig 


More information about the Plasma-devel mailing list