Re: Full Screen Information

Hello,

On Jan/30/2008, Aurélien Gâteau wrote:
> Carles Pina i Estany a écrit :
> > Hello,
> > 
> > On Jan/26/2008, Carles Pina i Estany wrote:
> > 
> >> Patch is attached again. As always: any suggestion of change is
> > 
> > patch was not attached, now yes :-)
> 
> It's in, thanks!

There is (at least :-) ) one bug:

if you switch to to FullScreen clicking on the small icon on image
thumbail, this image is not getting the FullScreen Information. I think
that this SIGNAL(metaDataUpdated() is called before full screen Window
is created and in updateFullScreenInformation I do:
---------
        if (!d->mFullScreenAction->isChecked()) {
	                return;
        }
---------

I send a patch that is calling updateFullScreenInformation from
toggleFullScreen.

-- 
Carles Pina i Estany		GPG id: 0x8CBDAE64
	http://pinux.info	Manresa - Barcelona

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

_______________________________________________
Gwenview-general mailing list
Gwenview-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/gwenview-general

Index: app/mainwindow.cpp
===================================================================
--- app/mainwindow.cpp	(revision 769171)
+++ app/mainwindow.cpp	(working copy)
@@ -996,6 +996,7 @@
 			d->createFullScreenBar();
 		}
 		d->mFullScreenBar->setActivated(true);
+		updateFullScreenInformation();
 	} else {
 		d->mStateBeforeFullScreen.mActiveViewModeAction->trigger();
 		d->mSideBarContainer->setVisible(d->mStateBeforeFullScreen.mSideBarVisible);