From 4f566ce903b5558ce01d2b8e0a407887d2959b1d Mon Sep 17 00:00:00 2001
From: Self-Hosting-Group
 <155233284+Self-Hosting-Group@users.noreply.github.com>
Date: Mon, 30 Dec 2024 00:00:00 +0000
Subject: [PATCH] UPnP IGD: Improve wording and UI

Close #452
---
 src/router/kromo/dd-wrt/UPnP.asp             | 14 +++++------
 src/router/kromo/dd-wrt/lang_pack/english.js | 26 ++++++++++----------
 2 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/router/kromo/dd-wrt/UPnP.asp b/src/router/kromo/dd-wrt/UPnP.asp
index 5fe06b808b56..93f036141c3b 100644
--- a/src/router/kromo/dd-wrt/UPnP.asp
+++ b/src/router/kromo/dd-wrt/UPnP.asp
@@ -67,7 +67,7 @@ function setUPnPTable(forwards) {
 		//proto
 		var cell = row.insertCell(-1);
 		cell.innerHTML = data[i].proto;
-		cell.align = "center";
+		cell.align = "left";
 
 		row.style.height = "15px";
 		row.className = (data[i].enabled ? '' : 'disabled');
@@ -84,7 +84,7 @@ function setUPnPTable(forwards) {
 function deleteForward(x) {
 	if (x != 'all') {
 		var e = data[x];
-		if (!confirm(share.del + " " + e.desc + "? [" + e.wanPorts + "->" + e.lanPorts + " " + e.lanIP + " " + e.proto + "]")) return;
+		if (!confirm(share.del + " " + e.lanIP + ":" + e.lanPorts + "/" + e.proto + " (" + e.desc + ")?")) return;
 	}
 	else {
 		if (!confirm(upnp.msg2)) return;
@@ -152,11 +152,11 @@ addEvent(window, "unload", function() {
 								<table class="table" cellspacing="6" id="upnp_table" summary="UPnP table">
 									<tbody>
 									<tr>
-										<th width="10%" class="center"><% tran("share.ip"); %></th>
-										<th width="5%" class="center"><% tran("share.from2"); %>&nbsp;(WAN)</th>
-										<th width="5%" class="center"><% tran("share.to2"); %>&nbsp;(LAN)</th>
-										<th width="5%" class="center"><% tran("share.proto"); %></th>
-										<th width="30%" class="center"><% tran("share.descr"); %></th>
+										<th width="10%"><% tran("share.ip"); %></th>
+										<th width="7%"><% tran("pforward.from"); %></th>
+										<th width="7%"><% tran("pforward.to"); %></th>
+										<th width="5%"><% tran("share.proto"); %></th>
+										<th width="30%"><% tran("share.descr"); %></th>
 										<th width="10%" class="center"><% tran("share.del"); %></th>
 									</tr>
 									</tbody>
diff --git a/src/router/kromo/dd-wrt/lang_pack/english.js b/src/router/kromo/dd-wrt/lang_pack/english.js
index 819a4015290d..f68c4d93e368 100644
--- a/src/router/kromo/dd-wrt/lang_pack/english.js
+++ b/src/router/kromo/dd-wrt/lang_pack/english.js
@@ -523,7 +523,7 @@ bmenu.applicationsprforwarding="Port Range Forwarding";
 bmenu.applicationspforwarding="Port Forwarding";
 bmenu.applicationsipforwarding="IP Forwarding (1:1 NAT)";
 bmenu.applicationsptriggering="Port Triggering";
-bmenu.applicationsUpnp="UPnP";
+bmenu.applicationsUpnp="UPnP IGD";
 bmenu.applicationsDMZ="DMZ";
 bmenu.applicationsQoS="QoS";
 bmenu.applicationsP2P="P2P";
@@ -2574,24 +2574,24 @@ hupgrad.page1="<dd>New firmware versions are posted at <a href=\"https:\/\/dd-wr
 
 // ** UPnP.asp **//
 var upnp=new Object();
-upnp.titl="UPnP";
-upnp.h2="Universal Plug and Play (UPnP)";
-upnp.legend="Forwards";
-upnp.legend2="UPnP Configuration";
-upnp.serv="UPnP Service";
+upnp.titl="UPnP IGD";
+upnp.h2="UPnP IGD Service";
+upnp.legend="Active Port Forwards";
+upnp.legend2="Service Setup";
+upnp.serv="UPnP IGD Service";
 upnp.url="Send presentation URL";
-upnp.msg1="Click to delete entry";
-upnp.msg2="Delete all entries?";
+upnp.msg1="Click to delete port forward";
+upnp.msg2="Delete all port forwards?";
 
 //help container
 var hupnp=new Object();
-hupnp.right2="Click the remove icon to delete an individual entry.<br /> Click the <em>" + sbutton.delall + "</em> button to remove all entries.";
-hupnp.right4="Allows applications to automatically configure port forwarding.";
+hupnp.right2="List of ports opened by clients.";
+hupnp.right4='Allows clients to autonomously configure port forwards using the <abbr title="UPnP Internet Gateway Device (Control Protocol)">UPnP IGD</abbr> protocol.';
 
 //help page
-hupnp.page1="<dd>Also known as UPnP it is a set of network protocols used for the automatic configuration of devices. The goals of UPnP are to allow devices to connect seamlessly and to simplify the implementation of home or corporate environment networks. UPnP achieves this by defining and publishing device control protocols built upon open, Internet-based communication standards.</dd>";
-hupnp.page2="<dd>The UPnP forwards table shows all open ports forwarded automatically by the UPnP process. You can delete forwards by clicking <i>remove icon</i> or the <em>" + sbutton.delall + "</em> button to clear the undesired entries.</dd>";
-hupnp.page3="<dd>Allows applications to automatically setup port forwarding rules.</dd>";
+hupnp.page1='<dd>The <a href="https://en.wikipedia.org/wiki/Internet_Gateway_Device_Protocol" target="_blank" rel="noreferrer"><abbr title="UPnP Internet Gateway Device (Control Protocol)">UPnP IGD</abbr></a> protocol allow clients on the local network to configure port forwards on the router autonomously.</dd>';
+hupnp.page2="<dd>List of ports opened by clients.</dd>";
+hupnp.page3="<dd>Allows clients to autonomously configure port forwards.</dd>";
 hupnp.page4="<dd>If enabled, all UPnP port forwarding rules are deleted when the router starts up.</dd>";
 hupnp.page5="<dd>If enabled, a presentation URL tag is sent with the device description. This allows the router to show up in <em>Windows's My Network Places</em>.<br /><br />div class=\"note\"><h4>Note:</h4><div>When enabling this option you may need to reboot your computer.</div></div></dd><dd>Click the <em>" + sbutton.save + "</em> button to save your settings. Click the <em>" + sbutton.cancel + "</em> button to cancel your unsaved changes.</dd>";