Plugin: fix AdaptWindows in COperateDesktop bug

Author: KangLin

Src/CMakeLists.txt

@@ -107,6 +107,9 @@ set(PLUGIN_UI_FILES
 if(WIN32)
     list(APPEND PLUGIN_HEADER_FILES Windows/HookWindows.h)
     list(APPEND PLUGIN_SOURCE_FILES Windows/HookWindows.cpp)
+    if(MSVC)
+        list(APPEND PLUGIN_PRIVATE_DEFINITIONS NOMINMAX)
+    endif()
     list(APPEND PLUGIN_LIBS Ws2_32)
 endif()
 

Src/OperateDesktop.cpp

@@ -47,7 +47,7 @@ const QString COperateDesktop::Id()
         if(!GetParameter()->m_Net.GetHost().isEmpty())
             szId += "_" + GetParameter()->m_Net.GetHost()
                     + "_" + QString::number(GetParameter()->m_Net.GetPort());
-
+        
         CParameterNet* net = nullptr;
         QString szType;
         switch(GetParameter()->m_Proxy.GetUsedType())
@@ -72,7 +72,7 @@ const QString COperateDesktop::Id()
         default:
             break;
         }
-
+        
         if(!szType.isEmpty() && !net->GetHost().isEmpty()) {
             szId += "_" + szType + "_";
             szId += net->GetHost() + "_" + QString::number(net->GetPort());
@@ -89,7 +89,7 @@ const QString COperateDesktop::Name()
     if(GetParameter() && GetParameter()->GetGlobalParameters()
         && GetParameter()->GetGlobalParameters()->GetShowProtocolPrefix())
         szName = Protocol() + ":";
-
+    
     if(GetParameter() && !(GetParameter()->GetName().isEmpty()))
         szName += GetParameter()->GetName();
     else
@@ -102,10 +102,10 @@ const QString COperateDesktop::Description()
     QString szDescription;
     if(!Name().isEmpty())
         szDescription = tr("Name: ") + Name() + "\n";
-
+    
     if(!GetTypeName().isEmpty())
         szDescription += tr("Type:") + GetTypeName() + "\n";
-
+    
     if(!Protocol().isEmpty()) {
         szDescription += tr("Protocol: ") + Protocol();
 #ifdef DEBUG
@@ -165,21 +165,21 @@ int COperateDesktop::Initial()
     qDebug(log) << Q_FUNC_INFO;
     int nRet = 0;
     bool check = false;
-
+    
     nRet = COperate::Initial();
     if(nRet)
         return nRet;
-
+    
     Q_ASSERT(!(m_pFrmViewer && m_pScroll));
     m_pFrmViewer = new CFrmViewer(); // The owner is m_pScroll
     m_pScroll = new CFrmScroll(m_pFrmViewer);
-
+    
     check = connect(m_pFrmViewer, SIGNAL(sigViewerFocusIn(QWidget*)),
                     this, SIGNAL(sigViewerFocusIn(QWidget*)));
     Q_ASSERT(check);
-
+    
     nRet = InitialMenu();
-
+    
     return nRet;
 }
 
@@ -237,20 +237,28 @@ int COperateDesktop::InitialMenu()
     m_pZoomIn = pMenuZoom->addAction(QIcon::fromTheme("zoom-in"), tr("Zoom in"));
     m_pZoomIn->setCheckable(true);
     check = connect(
-        m_pZoomIn, &QAction::toggled, this,
+        m_pZoomIn, &QAction::triggered, this,
         [&](){
-            if(m_psbZoomFactor)
-                m_psbZoomFactor->setValue((m_pFrmViewer->GetZoomFactor() + 0.1) * 100);
+            double factor = 0;
+            if(m_psbZoomFactor) {
+                factor = (m_pFrmViewer->GetZoomFactor() + 0.1) * 100;
+                qDebug(log) << "Zoom in:" << factor;
+                m_psbZoomFactor->setValue(factor);
+            }
         });
     Q_ASSERT(check);
     m_pZoomOut = pMenuZoom->addAction(
         QIcon::fromTheme("zoom-out"), tr("Zoom out"));
     m_pZoomOut->setCheckable(true);
     check = connect(
-        m_pZoomOut, &QAction::toggled, this,
+        m_pZoomOut, &QAction::triggered, this,
         [&](){
-            if(m_psbZoomFactor)
-                m_psbZoomFactor->setValue((m_pFrmViewer->GetZoomFactor() - 0.1) * 100);
+            double factor = 100;
+            if(m_psbZoomFactor) {
+                factor = (m_pFrmViewer->GetZoomFactor() - 0.1) * 100;
+                qDebug(log) << "Zoom out:" << factor;
+                m_psbZoomFactor->setValue(factor);
+            }
         });
     Q_ASSERT(check);
     QActionGroup* pGBViewZoom = new QActionGroup(this);
@@ -260,6 +268,14 @@ int COperateDesktop::InitialMenu()
         pGBViewZoom->addAction(m_pZoomOriginal);
         pGBViewZoom->addAction(m_pZoomIn);
         pGBViewZoom->addAction(m_pZoomOut);
+        check = connect(pGBViewZoom, &QActionGroup::triggered,
+                        this, [&](QAction* a){
+                            if(a == m_pZoomIn || a == m_pZoomOut)
+                                m_psbZoomFactor->setEnabled(true);
+                            else {
+                                m_psbZoomFactor->setEnabled(false);
+                            }
+                        });
     }
     m_psbZoomFactor = new QSpinBox(pMenuZoom);
     m_psbZoomFactor->setRange(0, 9999999);
@@ -363,6 +379,7 @@ int COperateDesktop::SetGlobalParameters(CParameterPlugin *pPara)
                             this, SLOT(slotUpdateName()));
             Q_ASSERT(check);
         }
+        LoadAdaptWindows();
         return 0;
     } else {
         QString szMsg = "There is not parameters! "
@@ -424,25 +441,8 @@ int COperateDesktop::SetParameter(CParameterBase *p)
     return 0;
 }
 
-int COperateDesktop::Load(QSettings &set)
+int COperateDesktop::LoadAdaptWindows()
 {
-    int nRet = 0;
-    Q_ASSERT(m_pFrmViewer);
-    nRet = COperate::Load(set);
-    if(m_pPara)
-        nRet = m_pPara->Load(set);
-    else {
-        QString szMsg = "There is not parameters! "
-                        "please first create parameters, "
-                        "then call SetParameter() in the ";
-        szMsg += metaObject()->className() + QString("::")
-                 + metaObject()->className();
-        szMsg += QString("() or ") + metaObject()->className()
-                 + QString("::") + "Initial()";
-        szMsg += " to set the parameters pointer. ";
-        qWarning(log) << szMsg.toStdString().c_str();
-        Q_ASSERT(false);//TODO: delete it
-    }
     if(m_pFrmViewer && GetParameter())
     {
         m_pFrmViewer->slotSetZoomFactor(GetParameter()->GetZoomFactor());
@@ -463,6 +463,30 @@ int COperateDesktop::Load(QSettings &set)
                 m_pZoomIn->setChecked(true);
         }
     }
+    return 0;
+}
+
+int COperateDesktop::Load(QSettings &set)
+{
+    int nRet = 0;
+    Q_ASSERT(m_pFrmViewer);
+    nRet = COperate::Load(set);
+    if(m_pPara)
+        nRet = m_pPara->Load(set);
+    else {
+        QString szMsg = "There is not parameters! "
+                        "please first create parameters, "
+                        "then call SetParameter() in the ";
+        szMsg += metaObject()->className() + QString("::")
+                 + metaObject()->className();
+        szMsg += QString("() or ") + metaObject()->className()
+                 + QString("::") + "Initial()";
+        szMsg += " to set the parameters pointer. ";
+        qWarning(log) << szMsg.toStdString().c_str();
+        Q_ASSERT(false);//TODO: delete it
+    }
+    
+    LoadAdaptWindows();
     return nRet;
 }
 
@@ -514,6 +538,7 @@ void COperateDesktop::slotRecorderStateChanged(
 
 void COperateDesktop::slotValueChanged(int v)
 {
+    qDebug(log) << "zoom:" << v;
     if(!m_pScroll || !m_pFrmViewer) return;
     m_pFrmViewer->slotSetZoomFactor(((double)v) / 100);
     m_pScroll->slotSetAdaptWindows(CFrmViewer::ADAPT_WINDOWS::Zoom);

Src/OperateDesktop.h

@@ -169,6 +169,9 @@ private Q_SLOTS:
     //! \~chinese \note 仅由 CBackendDesktop::SetConnect() 使用
     //! \~english \note The slot only is used by CBackendDesktop::SetConnect()
     virtual void slotSetServerName(const QString &szName);
+    
+private:
+    int LoadAdaptWindows();
 
 protected:
     QAction* m_pMenuZoom;

snap/snapcraft.yaml

@@ -32,9 +32,9 @@ description: |
 # See: https://snapcraft.io/docs/reference-architectures
 # See: https://snapcraft.io/docs/architectures
 base: core24 # the base snap is the execution environment for this snap
-#platforms:
-#  amd64:
-#  arm64:
+platforms:
+  amd64:
+  arm64:
 #base: core22
 #architectures:
 #  - build-on: amd64
@@ -322,18 +322,18 @@ parts:
       snapcraftctl stage
     prime:
       - -usr/bin/cmark
-      - -opt/RabbitRemoteControl/bin/cmark
-      - -opt/RabbitRemoteControl/include
-      - -opt/RabbitRemoteControl/lib/cmake
-      - -opt/RabbitRemoteControl/lib/pkgconfig
-      - -opt/RabbitRemoteControl/lib/*.a
+      - -usr/local/bin/cmark
+      - -usr/local/include
+      - -usr/local/lib/cmake
+      - -usr/local/lib/pkgconfig
+      - -usr/local/lib/*.a
 
 apps:
  rabbitremotecontrol:
-   #command: opt/RabbitRemoteControl/bin/start.sh
-   command: opt/RabbitRemoteControl/bin/RabbitRemoteControlApp
+   #command: usr/local/bin/start.sh
+   command: usr/local/bin/RabbitRemoteControlApp
    # See: https://snapcraft.io/docs/desktop-menu-support#heading--desktop-key
-   desktop: opt/RabbitRemoteControl/share/applications/io.github.KangLin.RabbitRemoteControl.desktop
+   desktop: usr/local/share/applications/io.github.KangLin.RabbitRemoteControl.desktop
    environment:
       LD_LIBRARY_PATH: ${SNAP}/usr/local/lib::${LD_LIBRARY_PATH}
       PATH: $SNAP/usr/local/bin:$PATH