Merge remote-tracking branch 'origin/develop' into claudio-dev

resources/library/search/Audio Planete.wgs/css/basic.css

@@ -73,7 +73,7 @@ a {
     overflow: hidden;
     text-align: center;
     margin-top: 50px;
-    margin-bottom: 50px;
+    margin-bottom: 85px;
 }
 
 .search{

resources/library/search/Audio Planete.wgs/index.html

@@ -8,7 +8,7 @@
         <script type="text/javascript">
             var category = "audio";
             var thumbnails = false;
-            var minHeight = 180;
+            var minHeight = 177;
             var minWidth = 150;
             
             var currentIndex = 0;

resources/library/search/Flash Planete.wgs/css/basic.css

@@ -73,7 +73,7 @@ a {
     overflow: hidden;
     text-align: center;
     margin-top: 50px;
-    margin-bottom: 50px;
+    margin-bottom: 85px;
 }
 
 .search{

resources/library/search/Flash Planete.wgs/index.html

@@ -8,7 +8,7 @@
         <script type="text/javascript">
             var category = "interactive";
             var thumbnails = false;
-            var minHeight = 180;
+            var minHeight = 177;
             var minWidth = 150;
             
             var currentIndex = 0;

resources/library/search/Google images.wgs/css/basic.css

@@ -73,7 +73,7 @@ a {
     overflow: hidden;
     text-align: center;
     margin-top: 50px;
-    margin-bottom: 50px;
+    margin-bottom: 85px;
 }
 
 .search{   

resources/library/search/Image Planete.wgs/css/basic.css

@@ -73,7 +73,7 @@ a {
     overflow: hidden;
     text-align: center;
     margin-top: 50px;
-    margin-bottom: 50px;
+    margin-bottom: 85px;
 }
 
 .search{

resources/library/search/Video Planete.wgs/css/basic.css

@@ -73,7 +73,7 @@ a {
     overflow: hidden;
     text-align: center;
     margin-top: 50px;
-    margin-bottom: 50px;
+    margin-bottom: 85px;
 }
 
 .search{

resources/library/search/Video Planete.wgs/index.html

@@ -8,7 +8,7 @@
         <script type="text/javascript">
             var category = "video";
             var thumbnails = false;
-            var minHeight = 180;
+            var minHeight = 177;
             var minWidth = 150;
             
             var currentIndex = 0;

src/board/UBBoardController.cpp

@@ -534,10 +534,12 @@ void UBBoardController::duplicateScene()
     duplicateScene(mActiveSceneIndex);
 }
 
-void UBBoardController::duplicateItem(UBItem *item)
+UBGraphicsItem *UBBoardController::duplicateItem(UBItem *item)
 {    
     if (!item)
-        return;
+        return NULL;
+
+    UBGraphicsItem *retItem = NULL;
 
     mLastCreatedItem = NULL;
 
@@ -611,6 +613,7 @@ void UBBoardController::duplicateItem(UBItem *item)
     case UBMimeType::Group:
     {
     	UBGraphicsGroupContainerItem* groupItem = dynamic_cast<UBGraphicsGroupContainerItem*>(item);
+        UBGraphicsGroupContainerItem* duplicatedGroup = NULL;
     	if(groupItem){
     		QTransform groupTransform = groupItem->transform();
     		groupItem->resetTransform();
@@ -629,13 +632,13 @@ void UBBoardController::duplicateItem(UBItem *item)
     		if(!selItems.empty()){
     			// I don't like this solution but for now this is the only way I found.
     			// Normally, at this state, only the duplicated group should be selected
-    			UBGraphicsGroupContainerItem* duplicatedGroup = dynamic_cast<UBGraphicsGroupContainerItem*>(selItems.at(0));
+    			duplicatedGroup = dynamic_cast<UBGraphicsGroupContainerItem*>(selItems.at(0));
     			if(NULL != duplicatedGroup){
     				duplicatedGroup->setTransform(groupTransform);
     			}
     		}
     	}
-    	return;
+    	retItem = dynamic_cast<UBGraphicsItem *>(duplicatedGroup);
     	break;
     }
 
@@ -649,10 +652,13 @@ void UBBoardController::duplicateItem(UBItem *item)
                 mLastCreatedItem = gitem;
                 gitem->setSelected(true);
             }
-            return;
+            retItem = dynamic_cast<UBGraphicsItem *>(gitem);
         }break;
     }    
     
+    if (retItem)
+        return retItem;
+
     UBItem *createdItem = downloadFinished(true, sourceUrl, contentTypeHeader, pData, itemPos, QSize(itemSize.width(), itemSize.height()), false);
     if (createdItem)
     {
@@ -664,7 +670,10 @@ void UBBoardController::duplicateItem(UBItem *item)
             createdGitem->setPos(itemPos);
         mLastCreatedItem = dynamic_cast<QGraphicsItem*>(createdItem);
         mLastCreatedItem->setSelected(true);
+
+        retItem = dynamic_cast<UBGraphicsItem *>(createdItem);
     }
+    return retItem;
 }
 
 void UBBoardController::deleteScene(int nIndex)
@@ -2082,7 +2091,7 @@ UBGraphicsWidgetItem *UBBoardController::addW3cWidget(const QUrl &pUrl, const QP
         QString snapshotPath = selectedDocument()->persistencePath() +  "/" + UBPersistenceManager::widgetDirectory + "/" + struuid + ".png";
         w3cWidgetItem->setSnapshotPath(QUrl::fromLocalFile(snapshotPath));
         UBGraphicsWidgetItem *tmpItem = dynamic_cast<UBGraphicsWidgetItem*>(w3cWidgetItem);
-        if (tmpItem)
+        if (tmpItem && tmpItem->scene())
            tmpItem->takeSnapshot().save(snapshotPath, "PNG");
 
     }
@@ -2295,17 +2304,24 @@ void UBBoardController::togglePodcast(bool checked)
         UBPodcastController::instance()->toggleRecordingPalette(checked);
 }
 
-
 void UBBoardController::moveGraphicsWidgetToControlView(UBGraphicsWidgetItem* graphicsWidget)
 {
     graphicsWidget->remove();
 
-    UBToolWidget *toolWidget = new UBToolWidget(graphicsWidget);
+    mActiveScene->setURStackEnable(false);
+    UBGraphicsItem *toolW3C = duplicateItem(dynamic_cast<UBItem *>(graphicsWidget));
+    UBGraphicsWidgetItem *copyedGraphicsWidget = NULL;
+
+    if (UBGraphicsWidgetItem::Type == toolW3C->type())
+        copyedGraphicsWidget = static_cast<UBGraphicsWidgetItem *>(toolW3C);
+
+    UBToolWidget *toolWidget = new UBToolWidget(copyedGraphicsWidget);
     mActiveScene->addItem(toolWidget);
     qreal ssf = 1 / UBApplication::boardController->systemScaleFactor();
 
     toolWidget->setScale(ssf);
     toolWidget->setPos(graphicsWidget->scenePos());
+    mActiveScene->setURStackEnable(true);
 }
 
 

src/board/UBBoardController.h

@@ -40,6 +40,7 @@ class UBGraphicsAudioItem;
 class UBGraphicsWidgetItem;
 class UBBoardPaletteManager;
 class UBItem;
+class UBGraphicsItem;
 
 
 class UBBoardController : public UBDocumentContainer
@@ -158,7 +159,7 @@ class UBBoardController : public UBDocumentContainer
 
         void moveSceneToIndex(int source, int target);
         void duplicateScene(int index);
-        void duplicateItem(UBItem *item);
+        UBGraphicsItem *duplicateItem(UBItem *item);
         void deleteScene(int index);
 
         bool cacheIsVisible() {return mCacheWidgetIsEnabled;}

src/desktop/UBDesktopPalette.cpp

@@ -23,8 +23,8 @@
 
 UBDesktopPalette::UBDesktopPalette(QWidget *parent, UBRightPalette* _rightPalette)
     : UBActionPalette(Qt::TopLeftCorner, parent)
-    , mShowHideAction(0)
-    , mDisplaySelectAction(0)
+    , mShowHideAction(NULL)
+    , mDisplaySelectAction(NULL)
     , rightPalette(_rightPalette)
 {
     QList<QAction*> actions;

src/domain/UBGraphicsScene.h

@@ -109,8 +109,9 @@ class UBGraphicsScene: public UBCoreGraphicsScene, public UBItem
     };
 
     //        tmp stub for divide addings scene objects from undo mechanism implementation
-    bool isURStackIsEnabled(){ return mUndoRedoStackEnabled;}
         void enableUndoRedoStack(){mUndoRedoStackEnabled = true;}
+        void setURStackEnable(bool enable){mUndoRedoStackEnabled = enable;}
+        bool isURStackIsEnabled(){return mUndoRedoStackEnabled;}
 
         UBGraphicsScene(UBDocumentProxy *parent, bool enableUndoRedoStack = true);
         virtual ~UBGraphicsScene();

src/domain/UBItem.h

@@ -99,6 +99,8 @@ protected:
 
 public:
 
+    virtual int type() const = 0;
+
     inline UBGraphicsItemDelegate *Delegate() const { return mDelegate; }
 
     static void assignZValue(QGraphicsItem*, qreal value);

src/gui/UBToolWidget.cpp

@@ -76,6 +76,15 @@ void UBToolWidget::initialize()
     if(!sUnpinPixmap)
         sUnpinPixmap = new QPixmap(":/images/unpin.svg");
 
+    UBGraphicsScene *wscene = dynamic_cast<UBGraphicsScene *>(mGraphicsWidgetItem->scene());
+    if (wscene)
+    {
+        wscene->removeItemFromDeletion(mGraphicsWidgetItem);
+        wscene->removeItem(mGraphicsWidgetItem);
+    }
+
+    mGraphicsWidgetItem->setParent(this);
+    
     QGraphicsLinearLayout *graphicsLayout = new QGraphicsLinearLayout(Qt::Vertical, this);
 
     mFrameWidth = UBSettings::settings()->objectFrameWidth;
@@ -83,7 +92,7 @@ void UBToolWidget::initialize()
     graphicsLayout->setContentsMargins(mContentMargin, mContentMargin, mContentMargin, mContentMargin);
     setPreferredSize(mGraphicsWidgetItem->preferredWidth() + mContentMargin * 2, mGraphicsWidgetItem->preferredHeight() + mContentMargin * 2);
 
-    mGraphicsWebView = new QGraphicsWebView();
+    mGraphicsWebView = new QGraphicsWebView(this);
     connect(mGraphicsWebView->page()->mainFrame(), SIGNAL(javaScriptWindowObjectCleared()), this, SLOT(javaScriptWindowObjectCleared()));
     mGraphicsWebView->load(mGraphicsWidgetItem->mainHtml());
     graphicsLayout->addItem(mGraphicsWebView);

src/tools/UBGraphicsTriangle.cpp

@@ -596,7 +596,7 @@ QCursor    UBGraphicsTriangle::flipCursor() const
 void UBGraphicsTriangle::mousePressEvent(QGraphicsSceneMouseEvent *event)
 {
     lastRect = rect().toRect();
-    lastPos = transform().inverted().map(event->screenPos());
+    lastPos = sceneTransform().inverted().map(event->screenPos());
 
     if (resize1Polygon().containsPoint(event->pos().toPoint(), Qt::OddEvenFill))
     {
@@ -636,7 +636,7 @@ void UBGraphicsTriangle::mouseMoveEvent(QGraphicsSceneMouseEvent *event)
     }
     else
     {
-        QPoint currPos = transform().inverted().map(event->screenPos());
+        QPoint currPos = sceneTransform().inverted().map(event->screenPos());
 
         if (mResizing1)
         {
@@ -804,6 +804,7 @@ void UBGraphicsTriangle::hoverEnterEvent(QGraphicsSceneHoverEvent *event)
 
 void UBGraphicsTriangle::hoverLeaveEvent(QGraphicsSceneHoverEvent *event)
 {
+    mResizing1 = mResizing2 = mRotating = false;
     mShowButtons = false;
     setCursor(Qt::ArrowCursor);
     mCloseSvgItem->setVisible(false);