[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [PATCH] Picture navigation for the frame plasmoid
From: "=?UTF-8?Q?=C3=89ric_ALBER?=" <eric.alber () gmail ! com>
Date: 2008-07-16 12:26:47
Message-ID: 676e80e0807160526k75cb64b6k5ef92861498e5eb9 () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
[Attachment #4 (text/plain)]
Aaron>
Just a point on Plasm::PaintUtils::shadowText, I tested the method but it
doesn't center the text horizontally and vertically in the generated pixmap,
so if I use it, the text is not centered in the labels.
Anne-Marie>
The behavior should be exactly the same than for lightbox. If it is not the
case for you, there is a bug in my patch :).
The only difference is that the labels appear only after a timeout so they
don't popup when you just move the mouse over the frame.
Does it never work properly or do you have a special workflow ?
I find the behavior quite simple, but do you think a sort of toolbar would
be better ? I find it a bit too heavy and intrusive. Or maybe, instead of
hiding the non-active label, display both when the mouse is over the frame
and highlight "Next" or "Previous" depending on the mouse position.
2008/7/16 Anne-Marie Mahfouf <annemarie.mahfouf@free.fr>:
> On Wednesday 16 July 2008 00:25:36 Éric ALBER wrote:
> > Hello,
> >
> > I worked a bit on the frame plasmoid to add picture navigation in
> slideshow
> > mode.
> > I implemented a Lightbox
> > <http://www.lokeshdhakar.com/projects/lightbox2/>style image
> > navigation
> >
> > I attach the patch to this mail. I created it in the kdeplasma-addons
> > folder, so apply it in the same folder.
> >
> > Tell me what you think about that :)
> >
> > Regards,
> >
> > Eric
>
> It does not work very well: the Next label stays even if there is no mouse
> over the picture.
> Clicking on Next or Previous triggers a sort of zoom and does not switch on
> next picture or previous picture.
>
> Also I don't like very much the way the handle also appears. Visually you
> have
> 2 things happening at different places with the same mouse over.
> Not sure how to solve this so the eyes are not assaulted!
>
> I'll be away all next week from this Friday afternoon to Saturday 26th on
> holiday without internet so you'll have to rely on other people advices to
> implement this.
> I'll be on IRC this evening Paris time to discuss it if you want!
>
> Anne-Marie
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>
[Attachment #5 (text/html)]
<div dir="ltr">Aaron><br>Just a point on Plasm::PaintUtils::shadowText, I tested \
the method but it doesn't center the text horizontally and vertically in the \
generated pixmap, so if I use it, the text is not centered in the labels.<br> \
<br><br>Anne-Marie><br>The behavior should be exactly the same than for lightbox. \
If it is not the case for you, there is a bug in my patch :).<br>The only difference \
is that the labels appear only after a timeout so they don't popup when you just \
move the mouse over the frame.<br> Does it never work properly or do you have a \
special workflow ?<br><br>I find the behavior quite simple, but do you think a sort \
of toolbar would be better ? I find it a bit too heavy and intrusive. Or maybe, \
instead of hiding the non-active label, display both when the mouse is over the frame \
and highlight "Next" or "Previous" depending on the mouse \
position.<br> <br><br><br><div class="gmail_quote">2008/7/16 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;"> <div class="Ih2E3d">On Wednesday 16 July 2008 \
00:25:36 Éric ALBER wrote:<br> > Hello,<br>
><br>
> I worked a bit on the frame plasmoid to add picture navigation in slideshow<br>
> mode.<br>
> I implemented a Lightbox<br>
</div>> <<a href="http://www.lokeshdhakar.com/projects/lightbox2/" \
target="_blank">http://www.lokeshdhakar.com/projects/lightbox2/</a>>style \
image<br> <div class="Ih2E3d">> navigation<br>
><br>
> I attach the patch to this mail. I created it in the kdeplasma-addons<br>
> folder, so apply it in the same folder.<br>
><br>
> Tell me what you think about that :)<br>
><br>
> Regards,<br>
><br>
> Eric<br>
<br>
</div>It does not work very well: the Next label stays even if there is no mouse<br>
over the picture.<br>
Clicking on Next or Previous triggers a sort of zoom and does not switch on<br>
next picture or previous picture.<br>
<br>
Also I don't like very much the way the handle also appears. Visually you \
have<br> 2 things happening at different places with the same mouse over.<br>
Not sure how to solve this so the eyes are not assaulted!<br>
<br>
I'll be away all next week from this Friday afternoon to Saturday 26th on<br>
holiday without internet so you'll have to rely on other people advices to<br>
implement this.<br>
I'll be on IRC this evening Paris time to discuss it if you want!<br>
<br>
Anne-Marie<br>
<br>
_______________________________________________<br>
Panel-devel mailing list<br>
<a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/panel-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br> \
</blockquote></div><br></div>
_______________________________________________
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