[Digikam-devel] [Bug 145236] small wishes for the light-table

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=145236         




------- Additional Comments From caulier.gilles gmail com  2007-05-13 09:54 -------
SVN commit 664131 by cgilles:

digikam from trunk : Light Table : do not lost zoom level synchronization between Left and Right panels using "NAvigate by Pair" function.
CCBUGS: 145236


 M  +4 -4      lighttableview.cpp  
 M  +14 -11    lighttablewindow.cpp  


--- trunk/extragear/graphics/digikam/utilities/lighttable/lighttableview.cpp #664130:664131
 @ -329,22 +329,22  @
 
 void LightTableView::slotLeftPreviewLoaded(bool success)
 {
-    emit signalLeftPreviewLoaded(success);
-
     checkForSyncPreview();
     d->leftLoading = false;
     slotRightContentsMoved(d->rightPreview->contentsX(), 
                            d->rightPreview->contentsY());
+
+    emit signalLeftPreviewLoaded(success);
 }
 
 void LightTableView::slotRightPreviewLoaded(bool success)
 {
-    emit signalRightPreviewLoaded(success);
-
     checkForSyncPreview();
     d->rightLoading = false;
     slotLeftContentsMoved(d->leftPreview->contentsX(), 
                           d->leftPreview->contentsY());
+
+    emit signalRightPreviewLoaded(success);
 }
 
 void LightTableView::checkForSyncPreview()
--- trunk/extragear/graphics/digikam/utilities/lighttable/lighttablewindow.cpp #664130:664131
 @ -666,6 +666,20  @
     LightTableBarItem *item = d->barView->findItemByInfo(d->previewView->leftImageInfo());
     if (item) item->setOnLeftPanel(true);
     d->barView->update();
+
+    if (d->navigateByPairAction->isChecked() && item)
+    {
+        LightTableBarItem* next = dynamic_cast<LightTableBarItem*>(item->next());
+        if (next)
+        {
+            d->barView->setOnRightPanel(next->info());
+            slotSetItemOnRightPanel(next->info());
+        }
+        else
+        {
+            slotSetItemOnRightPanel(0);
+        }
+    }
 }
 
 void LightTableWindow::slotRightPreviewLoaded(bool b)
 @ -718,17 +732,6  @
   
                 d->barView->setOnLeftPanel(info);
                 slotSetItemOnLeftPanel(info);
-
-                LightTableBarItem* next = dynamic_cast<LightTableBarItem*>(curr->next());
-                if (next)
-                {
-                    d->barView->setOnRightPanel(next->info());
-                    slotSetItemOnRightPanel(next->info());
-                }
-                else
-                {
-                    slotSetItemOnRightPanel(0);
-                }
             }
         }
     }