differentiate dockNo perScreen and total

--improve the signaling between the configuration window
and the dockview concerning freeEdges for current screens
and docks count. Now the docks counting is different
between screens and in general

app/dockcorona.cpp

@@ -633,8 +633,21 @@ int DockCorona::docksCount() const
     int docks{0};
 
     for (const auto &view : m_dockViews) {
-        if (view && view->containment()
+        if (view && view->containment() && !view->containment()->destroyed()) {
-            && !view->containment()->destroyed()) {
+            ++docks;
+        }
+    }
+
+    // qDebug() << docks << "docks on screen:" << screen;
+    return docks;
+}
+
+int DockCorona::docksCount(QScreen *screen) const
+{
+    int docks{0};
+
+    for (const auto &view : m_dockViews) {
+        if (view && view->screen() == screen && !view->containment()->destroyed()) {
             ++docks;
         }
     }

app/dockcorona.h

@@ -69,8 +69,10 @@ public:
     QList<Plasma::Types::Location> freeEdges(int screen) const;
     QList<Plasma::Types::Location> freeEdges(QScreen *screen) const;
 
-    int docksCount(int screen) const;
     int docksCount() const;
+    int docksCount(int screen) const;
+    int docksCount(QScreen *screen) const;
+
     int noDocksWithTasks() const;
     int screenForContainment(const Plasma::Containment *containment) const override;
 

app/dockview.cpp

@@ -124,6 +124,7 @@ DockView::DockView(Plasma::Corona *corona, QScreen *targetScreen, bool dockWindo
 
     if (dockCorona) {
         connect(dockCorona, &DockCorona::docksCountChanged, this, &DockView::docksCountChanged);
+        connect(this, &DockView::docksCountChanged, this, &DockView::totalDocksCountChanged);
         connect(dockCorona, &DockCorona::dockLocationChanged, this, &DockView::dockLocationChanged);
         connect(dockCorona, &DockCorona::dockLocationChanged, this, [&]() {
             //! check if an edge has been freed for a primary dock
@@ -788,7 +789,17 @@ int DockView::docksCount() const
 {
     auto dockCorona = qobject_cast<DockCorona *>(corona());
 
-    if (!dockCorona || !this->containment())
+    if (!dockCorona)
+        return 0;
+
+    return dockCorona->docksCount(screen());
+}
+
+int DockView::totalDocksCount() const
+{
+    auto dockCorona = qobject_cast<DockCorona *>(corona());
+
+    if (!dockCorona)
         return 0;
 
     return dockCorona->docksCount();
@@ -1247,12 +1258,14 @@ bool DockView::event(QEvent *e)
 
 QList<int> DockView::freeEdges() const
 {
-    if (!this->corona() || !this->containment()) {
+    DockCorona *dockCorona = qobject_cast<DockCorona *>(this->corona());
+
+    if (!dockCorona) {
         const QList<int> emptyEdges;
         return emptyEdges;
     }
 
-    const auto edges = corona()->freeEdges(this->containment()->screen());
+    const auto edges = dockCorona->freeEdges(screen());
     QList<int> edgesInt;
 
     foreach (Plasma::Types::Location edge, edges) {

app/dockview.h

@@ -60,6 +60,7 @@ class DockView : public PlasmaQuick::ContainmentView {
 
     Q_PROPERTY(int alignment READ alignment WRITE setAlignment NOTIFY alignmentChanged)
     Q_PROPERTY(int docksCount READ docksCount NOTIFY docksCountChanged)
+    Q_PROPERTY(int totalDocksCount READ totalDocksCount NOTIFY totalDocksCountChanged)
     Q_PROPERTY(int x READ x NOTIFY xChanged)
     Q_PROPERTY(int y READ y NOTIFY yChanged)
     Q_PROPERTY(int width READ width NOTIFY widthChanged)
@@ -102,6 +103,7 @@ public:
     int currentThickness() const;
 
     int docksCount() const;
+    int totalDocksCount() const;
 
     bool behaveAsPlasmaPanel() const;
     void setBehaveAsPlasmaPanel(bool behavior);
@@ -218,6 +220,7 @@ signals:
     void screenGeometryChanged();
     void sessionChanged();
     void shadowChanged();
+    void totalDocksCountChanged();
     void xChanged();
     void yChanged();
 

shell/package/contents/configuration/LatteDockConfiguration.qml

@@ -277,11 +277,14 @@ PlasmaCore.FrameSvgItem {
 
             spacing: units.largeSpacing
 
-            property int docksCount: dock.docksCount
+            Connections{
+                target: dock
+                onDocksCountChanged: actionButtons.updateEnabled();
+            }
 
-            onDocksCountChanged: {
+            function updateEnabled() {
-                addDock.enabled = docksCount < 4 && dock.freeEdges().length > 0
+                addDock.enabled = dock.docksCount < 4 && dock.freeEdges().length > 0
-                removeDock.enabled = (docksCount>1) && !(dock.docksWithTasks()===1 && dock.tasksPresent())
+                removeDock.enabled = dock.docksCount>1 && !(dock.docksWithTasks()===1 && dock.tasksPresent())
             }
 
             PlasmaComponents.Button {
@@ -291,17 +294,27 @@ PlasmaCore.FrameSvgItem {
 
                 PlasmaComponents.ComboBox {
                     id: actionsCmb
-
                     anchors.fill: parent
+                    enabled: addDock.enabled
 
-                    Component.onCompleted:{
+                    function addModel() {
                         var actions = []
-
                         actions.push("    " + i18n("Copy Dock"));
                         actionsCmb.model = actions;
                         actionsCmb.currentIndex = -1;
                     }
 
+                    function emptyModel() {
+                        var actions = []
+                        actions.push("  ");
+                        actionsCmb.model = actions;
+                        actionsCmb.currentIndex = -1;
+                    }
+
+                    Component.onCompleted:{
+                        addModel();
+                    }
+
                     onActivated: {
                         if (index==0) {
                             dock.copyDock();
@@ -309,6 +322,13 @@ PlasmaCore.FrameSvgItem {
 
                         actionsCmb.currentIndex = -1;
                     }
+
+                    onEnabledChanged: {
+                        if (enabled)
+                            addModel();
+                        else
+                            emptyModel();
+                    }
                 }
 
 
@@ -338,8 +358,7 @@ PlasmaCore.FrameSvgItem {
 
                 text: i18n("Remove")
                 iconSource: "edit-delete"
-                opacity: enabled ? 1 : 0
+                opacity: dock.totalDocksCount > 1 ? 1 : 0
-                //enabled: dock.docksCount > 1
                 tooltip: i18n("Remove current dock")
 
                 onClicked: dock.removeDock()