treewide: transition div tables to actual table markup

Modern browsers allow decomposing table markup equally well as nested div
constructs, therefor migrate our <div> table markup to actual <table> tags
but keep the old table/tr/th/td CSS classes for now to allow for a smooth
theme transition.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>

2 files changed, 20 insertions, 20 deletions

diff --git a/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/20_lan.js b/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/20_lan.js
index 2d786c8283..9f4edce288 100644
--- a/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/20_lan.js
+++ b/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/20_lan.js
@@ -25,15 +25,15 @@ return baseclass.extend({
 
 		var container_wapper = E('div', { 'class': 'router-status-lan dashboard-bg box-s1' });
 		var container_box = E('div', { 'class': 'lan-info devices-list' });
-		var container_devices = E('div', { 'class': 'table assoclist devices-info' }, [
-			E('div', { 'class': 'tr table-titles  dashboard-bg' }, [
-				E('div', { 'class': 'th nowrap' }, _('Hostname')),
-				E('div', { 'class': 'th' }, _('IP Address')),
-				E('div', { 'class': 'th' }, _('MAC')),
+		var container_devices = E('table', { 'class': 'table assoclist devices-info' }, [
+			E('tr', { 'class': 'tr table-titles  dashboard-bg' }, [
+				E('th', { 'class': 'th nowrap' }, _('Hostname')),
+				E('th', { 'class': 'th' }, _('IP Address')),
+				E('th', { 'class': 'th' }, _('MAC')),
 			])
 		]);
 
-		var container_deviceslist = E('div', { 'class': 'table assoclist devices-info' });
+		var container_deviceslist = E('table', { 'class': 'table assoclist devices-info' });
 
 		container_box.appendChild(E('div', { 'class': 'title'}, [
 			E('img', {
@@ -48,21 +48,21 @@ return baseclass.extend({
 		for(var idx in this.params.lan.devices) {
 			var deivce = this.params.lan.devices[idx];
 
-			container_deviceslist.appendChild(E('div', { 'class': 'tr cbi-rowstyle-1'}, [
+			container_deviceslist.appendChild(E('tr', { 'class': 'tr cbi-rowstyle-1'}, [
 
-				E('div', { 'class': 'td device-info'}, [
+				E('td', { 'class': 'td device-info'}, [
 					E('p', {}, [
 						E('span', { 'class': 'd-inline-block'}, [ deivce.hostname ]),
 					]),
 				]),
 
-				E('div', { 'class': 'td device-info'}, [
+				E('td', { 'class': 'td device-info'}, [
 					E('p', {}, [
 						E('span', { 'class': 'd-inline-block'}, [ deivce.ipv4 ]),
 					]),
 				]),
 
-				E('div', { 'class': 'td device-info'}, [
+				E('td', { 'class': 'td device-info'}, [
 					E('p', {}, [
 						E('span', { 'class': 'd-inline-block'}, [ deivce.macaddr ]),
 					]),
diff --git a/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/30_wifi.js b/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/30_wifi.js
index 03c9ee6060..76dd0881c7 100644
--- a/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/30_wifi.js
+++ b/modules/luci-mod-dashboard/htdocs/luci-static/resources/view/dashboard/include/30_wifi.js
@@ -78,20 +78,20 @@ return baseclass.extend({
 
 		container_box.appendChild(container_radio);
 
-		var container_devices = E('div', { 'class': 'table assoclist devices-info' }, [
-			E('div', { 'class': 'tr table-titles  dashboard-bg' }, [
-				E('div', { 'class': 'th nowrap' }, _('Hostname')),
-				E('div', { 'class': 'th' }, _('Wireless')),
-				E('div', { 'class': 'th' }, _('Signal')),
-				E('div', { 'class': 'th' }, '%s / %s'.format( _('Up.'), _('Down.')))
+		var container_devices = E('table', { 'class': 'table assoclist devices-info' }, [
+			E('tr', { 'class': 'tr table-titles  dashboard-bg' }, [
+				E('th', { 'class': 'th nowrap' }, _('Hostname')),
+				E('th', { 'class': 'th' }, _('Wireless')),
+				E('th', { 'class': 'th' }, _('Signal')),
+				E('th', { 'class': 'th' }, '%s / %s'.format( _('Up.'), _('Down.')))
 			])
 		]);
 
 		var container_devices_item;
-		var container_devices_list = E('div', { 'class': 'table assoclist devices-info' });
+		var container_devices_list = E('table', { 'class': 'table assoclist devices-info' });
 
 		for (var i =0; i < this.params.wifi.devices.length; i++) {
-			container_devices_item = E('div', { 'class': 'tr cbi-rowstyle-1' });
+			container_devices_item = E('tr', { 'class': 'tr cbi-rowstyle-1' });
 
 			for(var idx in this.params.wifi.devices[i]) {
 				var device = this.params.wifi.devices[i];
@@ -109,7 +109,7 @@ return baseclass.extend({
 						])
 					]);
 				} else if ('rate' == idx) {
-					container_content = E('div', { 'class': 'td device-info'  }, [
+					container_content = E('td', { 'class': 'td device-info'  }, [
 						E('p', {}, [
 							E('span', { 'class': ''}, [ device[idx].value.rx ]),
 							E('br'),
@@ -117,7 +117,7 @@ return baseclass.extend({
 						])
 					]);
 				} else {
-					container_content = E('div', { 'class': 'td device-info'}, [
+					container_content = E('td', { 'class': 'td device-info'}, [
 						E('p', {}, [
 							E('span', { 'class': ''}, [ device[idx].value ]),
 						])