Bug 174026

Summary:	too narrow area to show buttons in slideshow
Product:	[Applications] digikam	Reporter:	Eduard Sukharev <kraplax>
Component:	Plugin-Generic-SlideShow	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	marcel.wiesweg, valerio.fuoglio
Priority:	NOR
Version:	0.10.0
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:		Version Fixed In:	0.10.0
Sentry Crash Report:

Description Eduard Sukharev 2008-11-01 12:09:16 UTC

Version:           0.10.0-beta5 (using 4.1.2 (KDE 4.1.2), MS Visual Studio 2005 SP1)
Compiler:          cl.exe
OS:                Microsoft Windows (i686) release 5.1 (XP Professional)

the area on the top and on the bottom of the slideshow that is responsible for showing the control buttons is too narrow.
It's about twice thinner than buttons themself. This way we have buttons that hides when mouse cursor is not on their "poor" half. To reproduce do the following:
1) start a slideshow
2) move your mouse pointer to the top of the screen.
3) hover the mouse cursor over some button (the top half of it)
4) slowly move your cursor down to the other half of the button that doesn't lie inside that autoshow area.

the effect - buttons disappear, even though the cursor was still on them.
expected - not only they shouldn't have disappeared, i expected to have at least some 3-5 px area below them so that in case cursor was too "quick" and went too far down.

It's not a wish, it's a bug because it affects ability to control the slodeshow directly.

Comment 1 Marcel Wiesweg 2008-12-22 22:53:55 UTC

SVN commit 900453 by mwiesweg:

Slightly change the logic to move, show and hide the toolbar in Slideshow.
Use actual toolbar size to calculate the autohide/autoshow area instead of hardcoding 20px.
Add a margin (10px, 25px) around the toolbar size for this area.
Fix flicker-like behavior where the toolbar was shortly shown at the bottom right when showing at top right.

CCBUG: 174026

 M  +54 -27    slideshow.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=900453

Comment 2 Marcel Wiesweg 2008-12-22 22:55:14 UTC

Eduard, as you report from Windows, please test if this fixes your problem.
For me on Linux it is fixed.

Comment 3 caulier.gilles 2008-12-25 14:30:22 UTC

Eduard,

Can you test again using code from svn trunk. Thanks in advance

Gilles Caulier

Comment 4 caulier.gilles 2009-01-02 09:22:30 UTC

Marcel,

Tested under windows. Work fine for me.

Gilles Caulier

Comment 5 caulier.gilles 2009-01-02 09:30:14 UTC

Valerio,

I recommend to take a look in Marcel patch and backport code to Slideshow plugin if it's possible.

Happy new year

Gilles Caulier

Comment 6 Valerio Fuoglio 2009-01-02 11:42:36 UTC

(In reply to comment #5)
> Valerio,
> 
> I recommend to take a look in Marcel patch and backport code to Slideshow
> plugin if it's possible.

I've made a lot of changes to the toolbar (and many more I planned) since your fork.
Gilles, please, are you able to reproduce this bug under Windows using "Advanced Slideshow" ?

> Happy new year
> 
> Gilles Caulier

Happy new year ! :)

Valerio

Comment 7 caulier.gilles 2009-01-02 12:13:20 UTC

Valerio,

No i cannot reproduce it under Windows.

Gilles

Comment 8 Valerio Fuoglio 2009-01-02 17:10:40 UTC

(In reply to comment #7)
> Valerio,
> 
> No i cannot reproduce it under Windows.
> 
> Gilles
> 

That's a good news! :)

Thanks a lot
Valerio