Improved and fixed handling of CDockWidget::DockWidgetMovable - moving the tab in the tabbar is always allowed, only moving the complete dock widget can be blocked by clearing this flag

Author: Uwe Kindler

src/DockAreaTabBar.h

@@ -18,9 +18,9 @@ class CDockAreaWidget;
 struct DockAreaTabBarPrivate;
 
 /**
- * Custom scroll bar implementation for dock area tab bar
- * This scroll area enables floating of a whole dock area including all
- * dock widgets
+ * Custom tabbar implementation for tab area that is shown on top of a
+ * dock area widget.
+ * The tabbar displays the tab widgets of the contained dock widgets.
  */
 class CDockAreaTabBar : public QScrollArea
 {

src/DockAreaWidget.h

@@ -45,7 +45,7 @@ class CDockWidget;
 
 
 /**
- * DockAreaWidget manages multiple instances of DckWidgets.
+ * DockAreaWidget manages multiple instances of DockWidgets.
  * It displays a title tab, which is clickable and will switch to
  * the contents associated to the title when clicked.
  */

src/DockWidgetTab.cpp

@@ -240,19 +240,16 @@ void CDockWidgetTab::mouseReleaseEvent(QMouseEvent* ev)
 	// End of tab moving, change order now
 	if (d->isDraggingState(DraggingTab) && d->DockArea)
 	{
-        if (d->DockWidget->features() & CDockWidget::DockWidgetMovable) {
-            // Find tab under mouse
-            QPoint pos = d->DockArea->mapFromGlobal(ev->globalPos());
-            int fromIndex = d->DockArea->tabIndex(d->DockWidget);
-            int toIndex = d->DockArea->indexOfContentByTitlePos(pos, this);
-            if (-1 == toIndex)
-            {
-                toIndex = d->DockArea->count() - 1;
-            }
-            qDebug() << "Move tab from " << fromIndex << " to " << toIndex;
-            d->DockArea->reorderDockWidget(fromIndex, toIndex);
-        }
-
+		// Find tab under mouse
+		QPoint pos = d->DockArea->mapFromGlobal(ev->globalPos());
+		int fromIndex = d->DockArea->tabIndex(d->DockWidget);
+		int toIndex = d->DockArea->indexOfContentByTitlePos(pos, this);
+		if (-1 == toIndex)
+		{
+			toIndex = d->DockArea->count() - 1;
+		}
+		qDebug() << "Move tab from " << fromIndex << " to " << toIndex;
+		d->DockArea->reorderDockWidget(fromIndex, toIndex);
 	}
 
     if (!d->DragStartMousePosition.isNull())
@@ -276,37 +273,36 @@ void CDockWidgetTab::mouseMoveEvent(QMouseEvent* ev)
         return;
     }
 
+    // move floating winwdow
     if (d->isDraggingState(DraggingFloatingWidget))
     {
-        if (d->DockWidget->features() & CDockWidget::DockWidgetMovable) {
-            d->FloatingWidget->moveFloating();
-        }
+        d->FloatingWidget->moveFloating();
         QFrame::mouseMoveEvent(ev);
         return;
     }
 
     // move tab
     if (d->isDraggingState(DraggingTab))
     {
-        if (d->DockWidget->features() & CDockWidget::DockWidgetMovable) {
-            d->moveTab(ev);
-        }
+        // Moving the tab is always allowed because it does not mean moving the
+    	// dock widget around
+    	d->moveTab(ev);
     }
 
     bool MouseInsideTitleArea = d->titleAreaGeometryContains(ev->globalPos());
     if (!MouseInsideTitleArea)
 	{
-        if (d->DockWidget->features() & CDockWidget::DockWidgetMovable) {
+    	// Floating is only allowed for widgets that are movable
+        if (d->DockWidget->features().testFlag(CDockWidget::DockWidgetMovable))
+        {
             d->startFloating();
         }
     	return;
 	}
     else if (d->DockArea->count() > 1
      && (ev->pos() - d->DragStartMousePosition).manhattanLength() >= QApplication::startDragDistance()) // Wait a few pixels before start moving
 	{
-        if (d->DockWidget->features() & CDockWidget::DockWidgetMovable) {
-            d->DragState = DraggingTab;
-        }
+        d->DragState = DraggingTab;
 		return;
 	}