modify color change to pannel

modify menu docke
2025-11-09 22:08:13 +08:00 · 2025-11-09 20:58:24 +08:00
7 changed files with 197 additions and 85 deletions
--- a/src/ui/DockWidget.h
+++ b/src/ui/DockWidget.h
@ -4,6 +4,8 @@
 #include <QPushButton>
 #include <QLabel>
 #include <QTimer>
+#include <QPointer>
+#include <QMetaType>

 class DockWidgetTitleBar : public QWidget {
    Q_OBJECT
@ -56,3 +58,7 @@ private:
    DockWidgetTitleBar* titleBar_{ nullptr };
 	bool m_bMax;
 };
+
+// Register metatypes for safe QVariant storage/retrieval
+Q_DECLARE_METATYPE(DockWidget*)
+Q_DECLARE_METATYPE(QPointer<DockWidget>)
--- a/src/ui/Menu/SystemManagerMenu.cpp
+++ b/src/ui/Menu/SystemManagerMenu.cpp
@ -5,6 +5,8 @@
 #include "common/SpdLogger.h"
 #include "ui/MainFrame.h"
 #include "ui/DockWidget.h"
+#include "workspace/WorkSpaceManager.h"
+#include "workspace/WorkSpace.h"

 #include "ui_SystemManagerMenu.h"

@ -18,6 +20,10 @@ SystemManagerMenu::SystemManagerMenu(QWidget* parent)
    windowManagerMenu_ = new QMenu(this);
    windowManagerMenu_->setWindowFlags(windowManagerMenu_->windowFlags() | Qt::FramelessWindowHint);
    ui->menu_uisetting->setVisible(false);
+
+    // When workspace changes, clear and rebuild window manager menu
+    connect(&WorkSpaceManager::Get(), &WorkSpaceManager::WorkSpaceChanged,
+            this, &SystemManagerMenu::OnWorkspaceChanged);
 }

 SystemManagerMenu::~SystemManagerMenu() {
@ -25,6 +31,16 @@ SystemManagerMenu::~SystemManagerMenu() {
 }

 void SystemManagerMenu::AddDockWidget(class DockWidget* dockWidget) {
+    if (!dockWidget) return;
+    // Track dock widgets to support reload on workspace change
+    bool exists = false;
+    for (const auto& ptr : dockWidgets_) {
+        if (ptr == dockWidget) { exists = true; break; }
+    }
+    if (!exists) {
+        dockWidgets_.push_back(QPointer<DockWidget>(dockWidget));
+    }
+
    QAction* action = new QAction(dockWidget->windowTitle(), this);
    connect(dockWidget, &DockWidget::windowTitleChanged, [action](const QString& title) {
        action->setText(title);
@ -33,8 +49,8 @@ void SystemManagerMenu::AddDockWidget(class DockWidget* dockWidget) {
    action->setCheckable(true);
    // Keep action checked state consistent with dock visibility
    action->setChecked(dockWidget->isVisible());
-    // Bind dock widget to action via data for state sync
-    action->setData(QVariant::fromValue(dockWidget));
+    // Bind dock widget to action via data for state sync (use QPointer for safety)
+    action->setData(QVariant::fromValue(QPointer<DockWidget>(dockWidget)));
    connect(action, &QAction::triggered, [dockWidget, action]() {
        dockWidget->setVisible(action->isChecked());
        }
@ -57,11 +73,19 @@ void SystemManagerMenu::RemoveDockWidget(class DockWidget* dockWidget) {
    if (!windowManagerMenu_) {
        return;
    }
+    // Remove from tracked list
+    for (int i = 0; i < dockWidgets_.size(); ++i) {
+        if (dockWidgets_[i] == dockWidget) {
+            dockWidgets_.removeAt(i);
+            break;
+        }
+    }
    const auto actions = windowManagerMenu_->actions();
    for (QAction* act : actions) {
        QVariant v = act->data();
        if (v.isValid()) {
-            DockWidget* dock = v.value<DockWidget*>();
+            QPointer<DockWidget> ptr = v.value<QPointer<DockWidget>>();
+            DockWidget* dock = ptr.data();
            if (dock == dockWidget) {
                windowManagerMenu_->removeAction(act);
                act->deleteLater();
@ -95,11 +119,32 @@ void SystemManagerMenu::OnWindowManagerMenu() {
    for (QAction* act : windowManagerMenu_->actions()) {
        QVariant v = act->data();
        if (v.isValid()) {
-            DockWidget* dock = v.value<DockWidget*>();
+            QPointer<DockWidget> ptr = v.value<QPointer<DockWidget>>();
+            DockWidget* dock = ptr.data();
            if (dock) {
                act->setChecked(dock->isVisible());
+            } else {
+                // DockWidget disappeared; ensure action unchecked
+                act->setChecked(false);
            }
        }
    }
    windowManagerMenu_->exec(QCursor::pos());
 }
+
+void SystemManagerMenu::ReloadWindowManagerMenu() {
+    if (!windowManagerMenu_) return;
+    
+    windowManagerMenu_->clear();
+
+    for (const auto& ptr : dockWidgets_) {
+        DockWidget* dockWidget = ptr.data();
+        if (!dockWidget) continue;
+        AddDockWidget(dockWidget);
+    }
+}
+
+void SystemManagerMenu::OnWorkspaceChanged(WorkSpace* /*ws*/) {
+    // Clear and rebuild menu when workspace changes
+    ReloadWindowManagerMenu();
+}
--- a/src/ui/Menu/SystemManagerMenu.h
+++ b/src/ui/Menu/SystemManagerMenu.h
@ -2,6 +2,8 @@

 #include <QWidget>
 #include <QMenu>
+#include <QList>
+#include <QPointer>

 namespace Ui {
    class SystemManagerMenu;
@ -26,8 +28,13 @@ signals:
 private:
    void OnExit();
    void OnWindowManagerMenu();
+    void ReloadWindowManagerMenu();
+
+private slots:
+    void OnWorkspaceChanged(class WorkSpace* ws);

 private:
    Ui::SystemManagerMenu* ui;
    QMenu* windowManagerMenu_{ nullptr };
+    QList<QPointer<class DockWidget>> dockWidgets_;
 };
--- a/src/ui/Menu/WindowManagerMenu.cpp
+++ b/src/ui/Menu/WindowManagerMenu.cpp
@ -34,7 +34,8 @@ void WindowManagerMenu::AddDockWidget(DockWidget* dockWidget) {
    );
    // Initialize check state to match dock visibility
    item->setCheckState(dockWidget->isVisible() ? Qt::Checked : Qt::Unchecked);
-    item->setData(DockWidgetRole, QVariant::fromValue(dockWidget));
+    // Store as QPointer to prevent dangling pointer on dock destruction
+    item->setData(DockWidgetRole, QVariant::fromValue(QPointer<DockWidget>(dockWidget)));
    // Sync dock visibility changes to check state
    connect(dockWidget, &DockWidget::visibilityChanged, [item](bool visible) {
        item->setCheckState(visible ? Qt::Checked : Qt::Unchecked);
@ -55,7 +56,8 @@ void WindowManagerMenu::RemoveDockWidget(class DockWidget* dockWidget) {
    }
    for (int i = 0; i < ui->listWidget->count(); ++i) {
        QListWidgetItem* item = ui->listWidget->item(i);
-        DockWidget* dock = item->data(DockWidgetRole).value<DockWidget*>();
+        QPointer<DockWidget> ptr = item->data(DockWidgetRole).value<QPointer<DockWidget>>();
+        DockWidget* dock = ptr.data();
        if (dock == dockWidget) {
            // Remove and delete the list item
            delete ui->listWidget->takeItem(i);
@ -67,7 +69,8 @@ void WindowManagerMenu::RemoveDockWidget(class DockWidget* dockWidget) {
 void WindowManagerMenu::OnItemClicked(QListWidgetItem* item) {
    bool checked = !(item->checkState() == Qt::Checked);
    item->setCheckState(checked ? Qt::Checked : Qt::Unchecked);
-    DockWidget* dock = item->data(DockWidgetRole).value<DockWidget*>();
+    QPointer<DockWidget> ptr = item->data(DockWidgetRole).value<QPointer<DockWidget>>();
+    DockWidget* dock = ptr.data();
    if (dock) {
        dock->setVisible(checked);
    }
--- a/src/ui/Panel/LightPanel.cpp
+++ b/src/ui/Panel/LightPanel.cpp
@ -2,6 +2,9 @@
 #include "ui/DockWidget.h"
 #include "ui/DockTitleBar.h"
 #include "common/SpdLogger.h"
+#include "workspace/WorkSpaceManager.h"
+#include "workspace/WorkSpace.h"
+#include "workspace/Timestep.h"
 #include <QHBoxLayout>
 #include <QFileInfo>
 #include <QMessageBox>
@ -33,14 +36,14 @@ LightPanel::~LightPanel()

 void LightPanel::RefreshPanel()
 {
-	// Implement curve-specific refresh logic here
-	DataPanel::RefreshPanel();
+    // Implement curve-specific refresh logic here
+    DataPanel::RefreshPanel();

-	if (auto fileEntry = fileEntry_->AsLight()) {
-		OnDataPanelUpdated(fileEntry);
-	}
+    if (auto fileEntry = fileEntry_->AsLight()) {
+        OnDataPanelUpdated(fileEntry);
+    }

-	LOG_INFO("Refreshed TablePanel {}", GetIndex());
+    LOG_INFO("Refreshed LightPanel {}", GetIndex());
 }

 void LightPanel::InitUI()
@ -57,8 +60,8 @@ QString LightPanel::GetTypeDisplayName() const

 void LightPanel::OnDataPanelUpdated(FileEntryLight* fileEntry)
 {
-	QString strName = fileEntry->GetName();
-	updateTitle(strName);
+    QString strName = fileEntry->GetName();
+    updateTitle(strName);

 	FileEntryLight::ColorProperties propChart = fileEntry->GetColorProperties();

@ -66,35 +69,23 @@ void LightPanel::OnDataPanelUpdated(FileEntryLight* fileEntry)
 	QString closeColor = QColorToString(propChart.closeColor);
 	updateLampColor(openColor, closeColor);

-	QString strFile = fileEntry->GetPath() + "/" + fileEntry->GetFileName();
-	FileEntryLight::LightRowProperties listCurve = fileEntry->GetLightProperties();
-	updateParseFile(strFile, propChart.timeParam, listCurve);
+    QString strFile = fileEntry->GetPath() + "/" + fileEntry->GetFileName();
+    FileEntryLight::LightRowProperties listCurve = fileEntry->GetLightProperties();
+    updateParseFile(strFile, propChart.timeParam, listCurve);
+
+    // 颜色或数据更新后，按当前时间重新应用灯样式，确保即时可见
+    WorkSpace* ws = WorkSpaceManager::Get().GetCurrent();
+    if (ws) {
+        Timestep* ts = ws->GetTimestep();
+        if (ts) {
+            applyLampStylesForTime(ts->GetCurrent());
+        }
+    }
 }

 void LightPanel::OnTimeChanged(double time)
 {
-	if (m_dataLamp.size() > 0)
-	{
-		QMap< double, QVariantMap >::const_iterator ite = m_dataLamp.lowerBound(time);
-		if (ite == m_dataLamp.end())
-		{
-			ite--;
-		}
-
-		QVariantMap mapData = ite.value();
-		for (QVariantMap::Iterator it = mapData.begin(); it != mapData.end(); it++)
-		{
-			QString strKey = it.key();
-			int nState = it.value().toFloat();
-
-			SignalLabel* pLampLab = m_mapLamp.value(strKey);
-			if (pLampLab)
-			{
-				QString strStyle = m_lampColor.value(nState);
-				pLampLab->setStyleSheet(strStyle);
-			}
-		}
-	}
+    applyLampStylesForTime(time);
 }

 void LightPanel::updateTitle(const QString & title)
@ -109,7 +100,7 @@ void LightPanel::updateParseFile(const QString & strFile, int nT, FileEntryLight
 {
 	if (strFile.isEmpty())
 	{
-		QMessageBox::information(nullptr, QString::fromLocal8Bit("提示"), QString::fromLocal8Bit("请检查数据文件路径！"));
+		QMessageBox::information(nullptr, QString::fromLocal8Bit("<EFBFBD><EFBFBD>ʾ"), QString::fromLocal8Bit("<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ļ<EFBFBD>·<EFBFBD><EFBFBD><EFBFBD><EFBFBD>"));
 		return;
 	}

@ -183,16 +174,42 @@ void LightPanel::updateParseFile(const QString & strFile, int nT, FileEntryLight

 void LightPanel::updateLampColor(const QString & strOpenColor, const QString & strCloseColor)
 {
-	m_lampColor.clear();
+    m_lampColor.clear();

-	{
-		QString strStyle = "QLabel{background-color: rgb(" + strCloseColor + ");border-radius: 10px;}; ";
-		m_lampColor.insert(0, strStyle);
-	}
-	{
-		QString strStyle = "QLabel{background-color: rgb(" + strOpenColor + ");border-radius: 10px;}; ";
-		m_lampColor.insert(1, strStyle);
-	}
+    {
+        QString strStyle = "QLabel{background-color: rgb(" + strCloseColor + ");border-radius: 10px;}; ";
+        m_lampColor.insert(0, strStyle);
+    }
+    {
+        QString strStyle = "QLabel{background-color: rgb(" + strOpenColor + ");border-radius: 10px;}; ";
+        m_lampColor.insert(1, strStyle);
+    }
+}
+
+void LightPanel::applyLampStylesForTime(double time)
+{
+    if (m_dataLamp.isEmpty()) {
+        return;
+    }
+
+    auto ite = m_dataLamp.lowerBound(time);
+    if (ite == m_dataLamp.end()) {
+        ite--;
+    }
+
+    const QVariantMap mapData = ite.value();
+    for (auto it = mapData.constBegin(); it != mapData.constEnd(); ++it)
+    {
+        const QString strKey = it.key();
+        const int nState = it.value().toFloat();
+
+        SignalLabel* pLampLab = m_mapLamp.value(strKey);
+        if (pLampLab)
+        {
+            const QString strStyle = m_lampColor.value(nState);
+            pLampLab->setStyleSheet(strStyle);
+        }
+    }
 }

 void LightPanel::clearLightPanel()
--- a/src/ui/Panel/LightPanel.h
+++ b/src/ui/Panel/LightPanel.h
@ -60,15 +60,16 @@ protected:
 	virtual void OnTimeChanged(double time);

 private:
-	void updateTitle(const QString& title);
-	void updateParseFile(const QString& strFile, int nT, FileEntryLight::LightRowProperties listCurve);
-	void updateLampColor(const QString& strOpenColor, const QString& strCloseColor);
+    void updateTitle(const QString& title);
+    void updateParseFile(const QString& strFile, int nT, FileEntryLight::LightRowProperties listCurve);
+    void updateLampColor(const QString& strOpenColor, const QString& strCloseColor);
+    void applyLampStylesForTime(double time);

 	void clearLightPanel();

 private:
-	QMap<int, QString> m_lampColor;
-	QMap<QString, SignalLabel*> m_mapLamp;
-	QMap< double, QVariantMap > m_dataLamp;
+    QMap<int, QString> m_lampColor;
+    QMap<QString, SignalLabel*> m_mapLamp;
+    QMap< double, QVariantMap > m_dataLamp;
 };

--- a/src/ui/Panel/TablePanel.cpp
+++ b/src/ui/Panel/TablePanel.cpp
@ -1,6 +1,9 @@
 #include "ui/Panel/TablePanel.h"
 #include "ui/DockWidget.h"
 #include "ui/DockTitleBar.h"
+#include "workspace/WorkSpaceManager.h"
+#include "workspace/WorkSpace.h"
+#include "workspace/Timestep.h"
 #include "common/SpdLogger.h"
 #include <QHBoxLayout>
 #include <QFileInfo>
@ -80,17 +83,34 @@ QString TablePanel::GetTypeDisplayName() const

 void TablePanel::OnDataPanelUpdated(FileEntryTable* fileEntry) 
 {
-	QString strName = fileEntry->GetName();
-	updateTitle(strName);
+    QString strName = fileEntry->GetName();
+    updateTitle(strName);

-	FileEntryTable::ChartProperties propChart = fileEntry->GetChartProperties();
+    FileEntryTable::ChartProperties propChart = fileEntry->GetChartProperties();

-	QStringList tableHeader = propChart.headerString.split(',', Qt::SkipEmptyParts);
-	SetHeader(tableHeader);
+    QStringList tableHeader = propChart.headerString.split(',', Qt::SkipEmptyParts);
+    SetHeader(tableHeader);

-	QString strFile = fileEntry->GetPath() + "/" + fileEntry->GetFileName();
-	FileEntryTable::TableProperties listCurve = fileEntry->GetTableProperties();
-	updateParseFile(strFile, propChart.timeParam, listCurve);
+    QString strFile = fileEntry->GetPath() + "/" + fileEntry->GetFileName();
+    FileEntryTable::TableProperties listCurve = fileEntry->GetTableProperties();
+    updateParseFile(strFile, propChart.timeParam, listCurve);
+
+    // After parsing new data and settings, immediately refresh the table to current time
+    // so color changes from PropertyBrowser reflect in the UI right away
+    WorkSpace* ws = WorkSpaceManager::Get().GetCurrent();
+    if (ws) {
+        Timestep* ts = ws->GetTimestep();
+        if (ts) {
+            updateTable(ts->GetCurrent());
+            return;
+        }
+    }
+    // Fallback: render the latest snapshot if no workspace/timestep
+    if (!m_dataTable.isEmpty()) {
+        auto it = m_dataTable.end();
+        --it;
+        updateTable(it.key());
+    }
 }

 void TablePanel::OnTimeChanged(double time)
@ -118,7 +138,7 @@ void TablePanel::updateParseFile(const QString & strFile, int nT, FileEntryTable
 {
 	if (strFile.isEmpty())
 	{
-		QMessageBox::information(nullptr, QString::fromLocal8Bit("提示"), QString::fromLocal8Bit("请检查数据文件路径！"));
+		QMessageBox::information(nullptr, QString::fromLocal8Bit("<EFBFBD><EFBFBD>ʾ"), QString::fromLocal8Bit("<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ļ<EFBFBD>·<EFBFBD><EFBFBD><EFBFBD><EFBFBD>"));
 		return;
 	}

@ -187,9 +207,9 @@ void TablePanel::updateParseFile(const QString & strFile, int nT, FileEntryTable

 void TablePanel::updateTable(double t)
 {
-	if (m_dataTable.size() > 0)
-	{
-		clearTable();
+    if (m_dataTable.size() > 0)
+    {
+        clearTable();

 		QMap< double, QMap<int, QVariantList> >::const_iterator ite = m_dataTable.lowerBound(t);
 		if (ite == m_dataTable.end())
@ -197,24 +217,37 @@ void TablePanel::updateTable(double t)
 			ite--;
 		}

-		QMap<int, QVariantList> mapData = ite.value();
-		for (QMap<int, QVariantList>::Iterator it = mapData.begin(); it != mapData.end(); it++)
-		{
-			int nRow = it.key();
-			QVariantList dataList = it.value();
+        QMap<int, QVariantList> mapData = ite.value();
+        for (QMap<int, QVariantList>::Iterator it = mapData.begin(); it != mapData.end(); it++)
+        {
+            int nRow = it.key();
+            QVariantList dataList = it.value();

-			m_pTableWidget->insertRow(nRow);
+            m_pTableWidget->insertRow(nRow);

-			for (int nI = 0; nI < dataList.size(); nI++)
-			{
-				QString strVal;
-				strVal.sprintf("%.6f", dataList.at(nI).toFloat());
-				QTableWidgetItem *item = new QTableWidgetItem(strVal);
-				item->setTextAlignment(Qt::AlignCenter);
-				m_pTableWidget->setItem(nRow, nI, item);
-			}
-		}
-	}
+            for (int nI = 0; nI < dataList.size(); nI++)
+            {
+                QString strVal;
+                strVal.sprintf("%.6f", dataList.at(nI).toFloat());
+                QTableWidgetItem *item = new QTableWidgetItem(strVal);
+                item->setTextAlignment(Qt::AlignCenter);
+                m_pTableWidget->setItem(nRow, nI, item);
+
+                // Apply color linkage: use FileEntryTable::TableProperty color for this row
+                if (m_tableSetting.contains(nRow)) {
+                    const QColor rowColor = m_tableSetting.value(nRow).second;
+                    if (rowColor.isValid()) {
+                        // Prefer foreground text color to mirror curve color, keep readability
+                        item->setForeground(QBrush(rowColor));
+                        // Optionally add a subtle background tint to improve visibility
+                        QColor bg = rowColor;
+                        bg.setAlpha(40);
+                        item->setBackground(QBrush(bg));
+                    }
+                }
+            }
+        }
+    }
 }

 void TablePanel::clearTable()
Author	SHA1	Message	Date
brige	c8b53ec674	modify color change to pannel	2025-11-09 22:08:13 +08:00
brige	6389e530b4	modify menu docke	2025-11-09 20:58:24 +08:00