From caab5fba78a71afa918046ff276690f936cb1a37 Mon Sep 17 00:00:00 2001
From: Kacper Donat <kadet1090@gmail.com>
Date: Sat, 29 Mar 2025 17:44:03 +0100
Subject: [PATCH] Gui: Fix saving light sources

---
 .../PreferencePages/DlgSettingsLightSources.ui   | 16 ++++++++--------
 src/Gui/View3DSettings.cpp                       |  5 +++++
 src/Gui/View3DSettings.h                         |  2 ++
 3 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/Gui/PreferencePages/DlgSettingsLightSources.ui b/src/Gui/PreferencePages/DlgSettingsLightSources.ui
index 6c09e0ec81..59f516abd5 100644
--- a/src/Gui/PreferencePages/DlgSettingsLightSources.ui
+++ b/src/Gui/PreferencePages/DlgSettingsLightSources.ui
@@ -164,7 +164,7 @@
          <cstring>AmbientLightIntensity</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -181,7 +181,7 @@
          <cstring>BacklightColor</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -267,7 +267,7 @@
          <cstring>AmbientLightColor</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -293,7 +293,7 @@
          <cstring>HeadlightIntensity</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -332,7 +332,7 @@
          <cstring>BacklightIntensity</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -367,7 +367,7 @@
          <cstring>FillLightIntensity</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -404,7 +404,7 @@
          <cstring>FillLightColor</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
@@ -431,7 +431,7 @@
          <cstring>HeadlightColor</cstring>
         </property>
         <property name="prefPath" stdset="0">
-         <cstring>View</cstring>
+         <cstring>View/LightSources</cstring>
         </property>
        </widget>
       </item>
diff --git a/src/Gui/View3DSettings.cpp b/src/Gui/View3DSettings.cpp
index b625158f0d..9bd0669627 100644
--- a/src/Gui/View3DSettings.cpp
+++ b/src/Gui/View3DSettings.cpp
@@ -45,22 +45,27 @@ using namespace Gui;
 View3DSettings::View3DSettings(ParameterGrp::handle hGrp,
                                View3DInventorViewer* view)
     : hGrp(hGrp)
+    , hLightSourcesGrp(hGrp->GetGroup("LightSources"))
     , _viewers{view}
 {
     hGrp->Attach(this);
+    hLightSourcesGrp->Attach(this);
 }
 
 View3DSettings::View3DSettings(ParameterGrp::handle hGrp,
                                const std::vector<View3DInventorViewer *>& view)
     : hGrp(hGrp)
+    , hLightSourcesGrp(hGrp->GetGroup("LightSources"))
     , _viewers(view)
 {
     hGrp->Attach(this);
+    hLightSourcesGrp->Attach(this);
 }
 
 View3DSettings::~View3DSettings()
 {
     hGrp->Detach(this);
+    hLightSourcesGrp->Detach(this);
 }
 
 int View3DSettings::stopAnimatingIfDeactivated() const
diff --git a/src/Gui/View3DSettings.h b/src/Gui/View3DSettings.h
index 0bc7f3af6d..ccef176218 100644
--- a/src/Gui/View3DSettings.h
+++ b/src/Gui/View3DSettings.h
@@ -54,6 +54,8 @@ public:
 
 private:
     ParameterGrp::handle hGrp;
+    ParameterGrp::handle hLightSourcesGrp;
+
     std::vector<View3DInventorViewer*> _viewers;
 };