[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 18:20 -------
SVN commit 664297 by cgilles:

digikam from trunk : Light Table : 
- when thumbbar is empty, enable properlly all menu options when new item are inserted.
- when "Navigate by Pair" option is enable and when an item is removed from thumbbar, reorganize properlly the pair.
CCBUGS: 145236



 M  +16 -6     lighttablewindow.cpp  


--- trunk/extragear/graphics/digikam/utilities/lighttable/lighttablewindow.cpp #664296:664297
 @ -596,14 +596,23  @
     {
         if (!d->barView->findItemByInfo(*it))
         {
-            LightTableBarItem *item = new LightTableBarItem(d->barView, *it);
-            if (*it == imageInfoCurrent)
-            {
-                d->barView->setSelectedItem(item);
-            }
+            new LightTableBarItem(d->barView, *it);
         }
     }   
 
+    if (imageInfoCurrent)
+    {
+        LightTableBarItem *ltItem = dynamic_cast<LightTableBarItem*>(d->barView->findItemByInfo(imageInfoCurrent));
+        if (ltItem) 
+            d->barView->setSelectedItem(ltItem);
+    }
+    else
+    {
+        LightTableBarItem *ltItem = dynamic_cast<LightTableBarItem*>(d->barView->firstItem());
+        if (ltItem) 
+            d->barView->setSelectedItem(ltItem);
+    }
+
     // if window is iconified, show it
     if (isMinimized())
     {
 @ -921,7 +930,8  @
     }
 
     d->barView->removeItem(info);
-    d->previewView->checkForSelection(d->barView->currentItemImageInfo());
+    d->barView->setSelected(d->barView->currentItem());
+    //d->previewView->checkForSelection(d->barView->currentItemImageInfo());
 }
 
 void LightTableWindow::slotEditItem()