Re: [PATCH] frame plasmoid: Switch photo when the user clicks on it

["Éric ALBER" <eric.alber@xxxxxxxxx>]

Yes, probably, I'll just have to update the Slideshow class to enable navigation to the previous picture.
I'll do it this WE :)

2008/7/4 John Tapsell <johnflux@xxxxxxxxx>:

If clicking advances it forward to the next image, then shift+click or
similiar should advance it back again.  Would it be easy to add this?
Thanks!

John Tapsell

2008/7/4 Éric ALBER <eric.alber@xxxxxxxxx>:

> Hi,
>
> I wrote a litte patch for the frame plasmoid.
> The idea is that if you click on the photo in the frame, it switches
> immediately to the next photo in slideshow mode
>
> I have also fixed two compiler warnings (misuse of uint and a missing
> Q_UNUSED).
>
> I attached the patch to this mail. It applies to
> trunk/KDE/kdeplasmoids/applets
> /frame/frame.cpp
> trunk/KDE/kdeplasmoids/applets/frame/frame.h
> trunk/KDE/kdeplasmoids/applets/frame/slideshow.cpp
> I created the patch in the kdeplasmoids directory
>
> I also noticed that void mousePressEvent (QGraphicsSceneMouseEvent *event)
> and other methods are not declared virtual in Plasma/Applet but the same
> methods are virtual in QGraphicsItem. I also noticed that the destructor of
> the Applet class is not virtual either. Is this a design choice or has it
> just been forgotten ?
>
> Regards,
>
> Eric
>

>>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe
>>> <<
>
>

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<