[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [PATCH] frame plasmoid: Switch photo when the user clicks on it
From: "=?UTF-8?Q?=C3=89ric_ALBER?=" <eric.alber () gmail ! com>
Date: 2008-07-04 14:48:07
Message-ID: 676e80e0807040748u6834a22ekd77dcacccc6c1d82 () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
[Attachment #4 (text/plain)]
My use case:
I have a frame on the desktop configured in slideshow mode with 20 secs
between the photo switch.
I'm currently doing a task like coding and I have a compilation process
running which will take 1min so I just have a look to the photo in the frame
(I mean that my main occupation at the moment is not watching photos).
- Maybe the currently displayed photo is not nice and I want to quickly
switch to the next one without waiting for the timeout.
- Or on the other side if the current photo is really interesting and the
timout occurs, the photo is switched and I would like to come back to the
previous one.
- Maybe the double click could open the photo with the default image viewer
so I can see it in full-screen.
Tell me what you think of these ideas.
Regards,
Eric
2008/7/4 Anne-Marie Mahfouf <annemarie.mahfouf@free.fr>:
> Le Friday 04 July 2008 14:MM:49 Éric ALBER, vous avez écrit :
> > 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
> CCing panel-devel mailing list as well - please reply to both
>
> OK this needs a bit more thinking. There's a bug report (wish)
> http://bugs.kde.org/show_bug.cgi?id=165704
> asking for a preview, pause and next action.
>
> There are several ways to do it (in my opinion):
> - mouse clicks: like the patch suggests. I am not keen on actions that the
> user cannot be aware of like clicking. If SHIFT-click makes it move
> backward,
> how is the user aware of that?
> - right click menu as suggested in the wish report: we'll end up with a
> right
> click menu with too much options, people want a Open With... and whatsnot
> action action as well.
> - triangles like the Comic widget does
> - on hover, integrated arrows + a pause button in the handle (the handle
> being
> the thing where you get the 4 icons on the left)
> - other?
>
> For next/previous and pause I'd like to hear about your use case of those
> (why
> do you want them, why isn't the timer enough for your needs, why you don't
> use GWenView,...) then if those are needed I'd like to hear about the
> several
> different ways suggested above.
>
> Anne-Marie
>
> PS I'll apply the warning and virtual fixes!
>
>
>
>
>
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> unsubscribe <<
>
[Attachment #5 (text/html)]
My use case:<br><br>I have a frame on the desktop configured in slideshow mode with \
20 secs between the photo switch.<br>I'm currently doing a task like coding and I \
have a compilation process running which will take 1min so I just have a look to the \
photo in the frame (I mean that my main occupation at the moment is not watching \
photos).<br>
- Maybe the currently displayed photo is not nice and I want to quickly switch to the \
next one without waiting for the timeout.<br>- Or on the other side if the current \
photo is really interesting and the timout occurs, the photo is switched and I would \
like to come back to the previous one.<br>
- Maybe the double click could open the photo with the default image viewer so I can \
see it in full-screen.<br><br>Tell me what you think of these \
ideas.<br><br>Regards,<br><br>Eric<br><br><div class="gmail_quote">2008/7/4 \
Anne-Marie Mahfouf <<a \
href="mailto:annemarie.mahfouf@free.fr">annemarie.mahfouf@free.fr</a>>:<br> \
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); \
margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Le Friday 04 July 2008 14:MM:49 Éric \
ALBER, vous avez écrit :<br> <div><div></div><div class="Wj3C7c">> Hi,<br>
><br>
> I wrote a litte patch for the frame plasmoid.<br>
> The idea is that if you click on the photo in the frame, it switches<br>
> immediately to the next photo in slideshow mode<br>
><br>
> I have also fixed two compiler warnings (misuse of uint and a missing<br>
> Q_UNUSED).<br>
><br>
> I attached the patch to this mail. It applies to<br>
> trunk/KDE/kdeplasmoids/applets/frame/frame.cpp<br>
> trunk/KDE/kdeplasmoids/applets/frame/frame.h<br>
> trunk/KDE/kdeplasmoids/applets/frame/slideshow.cpp<br>
> I created the patch in the kdeplasmoids directory<br>
><br>
> I also noticed that void mousePressEvent (QGraphicsSceneMouseEvent *event)<br>
> and other methods are not declared virtual in Plasma/Applet but the same<br>
> methods are virtual in QGraphicsItem. I also noticed that the destructor of<br>
> the Applet class is not virtual either. Is this a design choice or has it<br>
> just been forgotten ?<br>
><br>
> Regards,<br>
><br>
> Eric<br>
</div></div>CCing panel-devel mailing list as well - please reply to both<br>
<br>
OK this needs a bit more thinking. There's a bug report (wish)<br>
<a href="http://bugs.kde.org/show_bug.cgi?id=165704" \
target="_blank">http://bugs.kde.org/show_bug.cgi?id=165704</a><br> asking for a \
preview, pause and next action.<br> <br>
There are several ways to do it (in my opinion):<br>
- mouse clicks: like the patch suggests. I am not keen on actions that the<br>
user cannot be aware of like clicking. If SHIFT-click makes it move backward,<br>
how is the user aware of that?<br>
- right click menu as suggested in the wish report: we'll end up with a right<br>
click menu with too much options, people want a Open With... and whatsnot<br>
action action as well.<br>
- triangles like the Comic widget does<br>
- on hover, integrated arrows + a pause button in the handle (the handle being<br>
the thing where you get the 4 icons on the left)<br>
- other?<br>
<br>
For next/previous and pause I'd like to hear about your use case of those \
(why<br> do you want them, why isn't the timer enough for your needs, why you \
don't<br> use GWenView,...) then if those are needed I'd like to hear about \
the several<br> different ways suggested above.<br>
<font color="#888888"><br>
Anne-Marie<br>
</font><br>
PS I'll apply the warning and virtual fixes!<br>
<div><div></div><div class="Wj3C7c"><br>
<br>
<br>
<br>
<br>
>> Visit <a href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" \
target="_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to \
unsubscribe <<<br> </div></div></blockquote></div><br>
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic