Document mode: title page selection

src/document/UBDocumentContainer.cpp

@@ -52,12 +52,18 @@ void UBDocumentContainer::duplicatePages(QList<int>& pageIndexes)
     emit documentThumbnailsUpdated(this);
 }
 
-void UBDocumentContainer::movePageToIndex(int source, int target)
+bool UBDocumentContainer::movePageToIndex(int source, int target)
 {
+    if (source==0)
+    {
+        // Title page - cant be moved
+        return false;
+    }
     UBPersistenceManager::persistenceManager()->moveSceneToIndex(mCurrentDocument, source, target);
     deleteThumbPage(source);
     insertThumbPage(target);
     emit documentThumbnailsUpdated(this);
+    return true;
 }
 
 void UBDocumentContainer::deletePages(QList<int>& pageIndexes)

src/document/UBDocumentContainer.h

@@ -37,7 +37,7 @@ class UBDocumentContainer : public QObject
         static int sceneIndexFromPage(int sceneIndex); 
 
         void duplicatePages(QList<int>& pageIndexes);
-        void movePageToIndex(int source, int target);
+        bool movePageToIndex(int source, int target);
         void deletePages(QList<int>& pageIndexes);
         void addPage(int index);
         void updatePage(int index);

src/document/UBDocumentController.cpp

@@ -1032,12 +1032,13 @@ bool UBDocumentController::addFileToDocument(UBDocumentProxy* document)
 
 void UBDocumentController::moveSceneToIndex(UBDocumentProxy* proxy, int source, int target)
 {
-    UBDocumentContainer::movePageToIndex(source, target);
-
-    proxy->setMetaData(UBSettings::documentUpdatedAt, UBStringUtils::toUtcIsoDateTime(QDateTime::currentDateTime()));
-    UBMetadataDcSubsetAdaptor::persist(proxy);
+    if (UBDocumentContainer::movePageToIndex(source, target))
+    {
+        proxy->setMetaData(UBSettings::documentUpdatedAt, UBStringUtils::toUtcIsoDateTime(QDateTime::currentDateTime()));
+        UBMetadataDcSubsetAdaptor::persist(proxy);
     
-    mDocumentUI->thumbnailWidget->hightlightItem(target);
+        mDocumentUI->thumbnailWidget->hightlightItem(target);
+    }
 }
 
 

src/gui/UBThumbnailWidget.cpp

@@ -210,13 +210,18 @@ void UBThumbnailWidget::mousePressEvent(QMouseEvent *event)
     mMousePressPos = event->pos();
 
     UBSceneThumbnailPixmap* sceneItem = dynamic_cast<UBSceneThumbnailPixmap*>(itemAt(mMousePressPos));
-    if(sceneItem){
-    	int pageIndex = UBDocumentContainer::pageFromSceneIndex(sceneItem->sceneIndex());
-    	if(pageIndex == 0){
-            event->ignore();
-            return;
-    	}
+    if(sceneItem==NULL)
+    {
+        event->ignore();
+        return;
     }
+    //if(sceneItem){
+    //	int pageIndex = UBDocumentContainer::pageFromSceneIndex(sceneItem->sceneIndex());
+    //	if(pageIndex == 0){
+    //        event->ignore();
+    //        return;
+    //	}
+    //}
 
     mMousePressScenePos = mapToScene(mMousePressPos);
     QGraphicsItem* underlyingItem = itemAt(mMousePressPos);