From 951dcdcd7977e250058f0372dbb744203f6c96ea Mon Sep 17 00:00:00 2001
From: Monviech <gitacc@pischem.com>
Date: Fri, 22 May 2026 13:41:46 +0200
Subject: [PATCH] Services: Kea DHCPv4/6: Fix missing visual cues for manual
 mode in DDNS and DHCPv4/6

---
 .../OPNsense/Kea/forms/ddnsSettings.xml        |  8 ++++++++
 .../mvc/app/views/OPNsense/Kea/ddns.volt       | 18 ++++++++++++++++++
 .../mvc/app/views/OPNsense/Kea/dhcpv4.volt     |  2 +-
 .../mvc/app/views/OPNsense/Kea/dhcpv6.volt     |  2 +-
 4 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/src/opnsense/mvc/app/controllers/OPNsense/Kea/forms/ddnsSettings.xml b/src/opnsense/mvc/app/controllers/OPNsense/Kea/forms/ddnsSettings.xml
index b1a3f1772e..504e5a7388 100644
--- a/src/opnsense/mvc/app/controllers/OPNsense/Kea/forms/ddnsSettings.xml
+++ b/src/opnsense/mvc/app/controllers/OPNsense/Kea/forms/ddnsSettings.xml
@@ -1,4 +1,8 @@
 <form>
+    <field>
+        <type>header</type>
+        <label>Service</label>
+    </field>
     <field>
         <id>ddns.general.enabled</id>
         <label>Enabled</label>
@@ -12,6 +16,10 @@
         <advanced>true</advanced>
         <help>Disable configuration file generation and manage the file (/usr/local/etc/kea/kea-dhcp-ddns.conf) manually.</help>
     </field>
+    <field>
+        <type>header</type>
+        <label>General settings</label>
+    </field>
     <field>
         <id>ddns.general.server_ip</id>
         <label>Bind address</label>
diff --git a/src/opnsense/mvc/app/views/OPNsense/Kea/ddns.volt b/src/opnsense/mvc/app/views/OPNsense/Kea/ddns.volt
index 329b18867a..df2f5321cb 100644
--- a/src/opnsense/mvc/app/views/OPNsense/Kea/ddns.volt
+++ b/src/opnsense/mvc/app/views/OPNsense/Kea/ddns.volt
@@ -33,6 +33,24 @@
             updateServiceControlUI('kea');
         });
 
+        /* Manual configuration, hide all config elements except the service section*/
+        $("#ddns\\.general\\.manual_config").change(function(){
+            let manual_config = $(this).is(':checked');
+            if (manual_config) {
+                if (!$("#show_advanced_frm_generalsettings").hasClass('fa-toggle-on')) {
+                    /* enforce advanced mode so the user notices the checkbox */
+                    $("#show_advanced_frm_generalsettings").click();
+                }
+            }
+            $("#frm_generalsettings").find('table').each(function(){
+                if (manual_config && $(this).find('#ddns\\.general\\.manual_config').length == 0) {
+                    $(this).hide();
+                } else {
+                    $(this).show();
+                }
+            });
+        });
+
         $("#reconfigureAct").SimpleActionButton({
             onPreAction: function() {
                 const dfObj = new $.Deferred();
diff --git a/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv4.volt b/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv4.volt
index ede4fdaed3..460a6616f3 100644
--- a/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv4.volt
+++ b/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv4.volt
@@ -199,7 +199,7 @@
     <li><a data-toggle="tab" href="#settings" id="tab_settings">{{ lang._('Settings') }}</a></li>
     <li><a data-toggle="tab" href="#subnets" id="tab_pools" class="is_managed"> {{ lang._('Subnets') }} </a></li>
     <li><a data-toggle="tab" href="#reservations" id="tab_reservations" class="is_managed"> {{ lang._('Reservations') }} </a></li>
-    <li><a data-toggle="tab" href="#options" id="tab_options">{{ lang._('Options') }}</a></li>
+    <li><a data-toggle="tab" href="#options" id="tab_options" class="is_managed">{{ lang._('Options') }}</a></li>
     <li><a data-toggle="tab" href="#ha-peers" id="tab_ha-peers" class="is_managed"> {{ lang._('HA Peers') }} </a></li>
 </ul>
 <div class="tab-content content-box">
diff --git a/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv6.volt b/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv6.volt
index a8b8fcc66e..68976e86f3 100644
--- a/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv6.volt
+++ b/src/opnsense/mvc/app/views/OPNsense/Kea/dhcpv6.volt
@@ -237,7 +237,7 @@
     <li><a data-toggle="tab" href="#subnets" id="tab_pools" class="is_managed"> {{ lang._('Subnets') }} </a></li>
     <li><a data-toggle="tab" href="#pdpools" id="tab_reservations" class="is_managed"> {{ lang._('PD Pools') }} </a></li>
     <li><a data-toggle="tab" href="#reservations" id="tab_reservations" class="is_managed"> {{ lang._('Reservations') }} </a></li>
-    <li><a data-toggle="tab" href="#options" id="tab_options">{{ lang._('Options') }}</a></li>
+    <li><a data-toggle="tab" href="#options" id="tab_options" class="is_managed">{{ lang._('Options') }}</a></li>
     <li><a data-toggle="tab" href="#ha-peers" id="tab_ha-peers" class="is_managed"> {{ lang._('HA Peers') }} </a></li>
 </ul>
 <div class="tab-content content-box">