[Digikam-devel] extragear/graphics/digikam

[Gilles Caulier]

SVN commit 536492 by cgilles:

digikam from trunk : Image Editor and Showfoto bugfix :

- Do not try to plug toolbar into Thumbbar (showfoto only)
- Like we force to hide all window dock (left/right/top/bottom) to have a complete black background in fullscreen mode, the toolbar is always hiden.
This commit force to show toolbar in this mode if the right option is on from setup.

Nota : a new bug have been discovered about Exif auto-rotation and SlideShow mode. Take a look in TODO file.

CCMAIL: digikam-devel at kde.org 

 M  +2 -1      TODO  
 M  +0 -2      showfoto/showfoto.cpp  
 M  +5 -3      utilities/imageeditor/editor/editorwindow.cpp  


--- trunk/extragear/graphics/digikam/TODO #536491:536492
@@ -48,8 +48,9 @@
 
 * Showfoto: 
 
-   - hide toolbar in fullscreen option don't work.
    - if no image is loaded, selection in canvas is possible.
+   - In SlideShow with fullscreen mode enable, the vertically oriented image automaticily rotated using 
+     Exif tags are toggle on to save when SlideShow is complete.
 
 B.K.O files to fix :
 
--- trunk/extragear/graphics/digikam/showfoto/showfoto.cpp #536491:536492
@@ -460,8 +460,6 @@
         {
             if (m_fullScreenHideThumbBar)
                 m_bar->hide();
-            else
-                m_fullScreenAction->plug(m_bar);
         }
     }
 }
--- trunk/extragear/graphics/digikam/utilities/imageeditor/editor/editorwindow.cpp #536491:536492
@@ -830,7 +830,7 @@
 
 void EditorWindow::slotToggleFullScreen()
 {
-    if (m_fullScreen)
+    if (m_fullScreen) // out of fullscreen
     {
         m_canvas->setBackgroundColor(d->bgColor);
 
@@ -877,7 +877,7 @@
         toggleGUI2FullScreen();
         m_fullScreen = false;
     }
-    else
+    else  // go to fullscreen
     {
         m_canvas->setBackgroundColor(QColor(Qt::black));
         
@@ -900,7 +900,9 @@
                 hideToolBars();
             }
             else
-            {    
+            {   
+                showToolBars();
+
                 if ( !m_fullScreenAction->isPlugged(toolBar) )
                 {
                     m_fullScreenAction->plug(toolBar);