[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Review Request: [Picture Frame] Update function for url's and
From:       etienne.rebetez () oberwallis ! ch
Date:       2010-03-21 19:46:39
Message-ID: 20100321194639.19073.36233 () localhost
[Download RAW message or body]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3334/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

I was quite happy that the picture frame could handle url's in 4.4. 
So i added the ability to update the picture with an url periodicaly (i.e. live cams, \
weather data or <picture that changes over time>).   This would be a rather small \
fix, but i also came accross Bug 222759 and with the updatefunction the download path \
got quite spamed. Now it's possible to set the download path and the ability to \
ovewrite the existing picture when downloaded. 

By the way, i also removed the picture class from the slideshow class. The slideshow \
now manages only the path's and the frame class acceses the picture class directly.

Let me know wat you think;)


Diffs
-----

  trunk/KDE/kdeplasma-addons/applets/frame/CMakeLists.txt 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/configdialog.h 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/configdialog.cpp 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/frame.h 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/frame.cpp 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/imageSettings.ui 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/picture.h 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/picture.cpp 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/slideshow.h 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/slideshow.cpp 1105894 
  trunk/KDE/kdeplasma-addons/applets/frame/urlSettings.ui PRE-CREATION 

Diff: http://reviewboard.kde.org/r/3334/diff


Testing
-------


Screenshots
-----------

Picture Frame settings
  http://reviewboard.kde.org/r/3334/s/335/


Thanks,

eti

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]