From d0b5f0c1e04c2e8c95b4f69193485ab82b5efd3c Mon Sep 17 00:00:00 2001
From: Nicola Lambathakis <modxit@modx.it>
Date: Tue, 1 Nov 2016 16:20:14 +0100
Subject: [PATCH 1/3] restyled modules config table and setdefault buttons

---
 manager/actions/mutate_module.dynamic.php | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/manager/actions/mutate_module.dynamic.php b/manager/actions/mutate_module.dynamic.php
index 91e9cc6ba7..d065102f1e 100644
--- a/manager/actions/mutate_module.dynamic.php
+++ b/manager/actions/mutate_module.dynamic.php
@@ -142,7 +142,7 @@ function showParameters(ctrl) {
         currentParams = JSON.parse(props);
     }
 
-    t = '<table width="100%" class="displayparams"><thead><tr><td width="1%"><?php echo $_lang['parameter']; ?></td><td width="99%"><?php echo $_lang['value']; ?></td></tr></thead>';
+    t = '<table width="100%" class="displayparams"><thead><tr><td><?php echo $_lang['parameter']; ?></td><td><?php echo $_lang['value']; ?></td><td style="text-align:right;"><?php echo $_lang["set_default"]; ?> </td></tr></thead>';
 
     try {
         var type, options, found, info, sd;
@@ -180,7 +180,7 @@ function showParameters(ctrl) {
                         c = '<input type="text" name="prop_' + key + '" value="' + value + '" size="30" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
                         break;
                     case 'menu':
-                        c = '<select name="prop_' + key + '" style="width:auto" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
+                        c = '<select name="prop_' + key + '" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
                         if (currentParams[key] == options) currentParams[key] = ls[0]; // use first list item as default
                         for (i = 0; i < ls.length; i++) {
                             c += '<option value="' + ls[i] + '"' + ((ls[i] == value) ? ' selected="selected"' : '') + '>' + ll[i] + '</option>';
@@ -189,7 +189,7 @@ function showParameters(ctrl) {
                         break;
                     case 'list':
                         if (currentParams[key] == options) currentParams[key] = ls[0]; // use first list item as default
-                        c = '<select name="prop_' + key + '" size="' + ls.length + '" style="width:auto" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
+                        c = '<select name="prop_' + key + '" size="' + ls.length + '" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
                         for (i = 0; i < ls.length; i++) {
                             c += '<option value="' + ls[i] + '"' + ((ls[i] == value) ? ' selected="selected"' : '') + '>' + ll[i] + '</option>';
                         }
@@ -199,7 +199,7 @@ function showParameters(ctrl) {
                         // value = typeof ar[3] !== 'undefined' ? (ar[3] + '').replace(/^\s|\s$/, "") : '';
                         arrValue = value.split(",");
                         if (currentParams[key] == options) currentParams[key] = ls[0]; // use first list item as default
-                        c = '<select name="prop_' + key + '" size="' + ls.length + '" multiple="multiple" style="width:auto" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
+                        c = '<select name="prop_' + key + '" size="' + ls.length + '" multiple="multiple" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
                         for (i = 0; i < ls.length; i++) {
                             if (arrValue.length) {
                                 found = false;
@@ -233,18 +233,18 @@ function showParameters(ctrl) {
                         }
                         break;
                     case 'textarea':
-                        c = '<textarea name="prop_' + key + '" style="width:80%" rows="4" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">' + value + '</textarea>';
+                        c = '<textarea name="prop_' + key + '" style="width:100%" rows="4" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">' + value + '</textarea>';
                         break;
                     default:  // string
-                        c = '<input type="text" name="prop_' + key + '" value="' + value + '" style="width:80%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
+                        c = '<input type="text" name="prop_' + key + '" value="' + value + '" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
                         break;
                 }
 
                 info = '';
                 info += desc ? '<br/><small>' + desc + '</small>' : '';
-	            sd = defaultVal != undefined ? ' <ul class="actionButtons" style="position:absolute;right:0px;bottom:6px;min-height:0;"><li><a href="#" class="primary btn-small btnSetDefault" onclick="setDefaultParam(\'' + key + '\',1);return false;"><?php echo $_lang["set_default"]; ?></a></li></ul>' : '';
+                sd = defaultVal != undefined ? '<ul class="actionButtons" style="float:right;margin-top:12px;"><li><a title="<?php echo $_lang["set_default"]; ?>" href="#" class="btnSetDefault" onclick="setDefaultParam(\'' + key + '\',1);return false;"><i class="fa fa-refresh"></i></a></li></ul>' : '';
 
-                t += '<tr><td class="labelCell" bgcolor="#FFFFFF" width="20%"><span class="paramLabel">' + label + '</span><span class="paramDesc">'+ info + '</span></td><td class="inputCell relative" bgcolor="#FFFFFF" width="80%">' + c + sd + '</td></tr>';
+                t += '<tr><td class="labelCell" bgcolor="#FFFFFF" width="20%"><span class="paramLabel">' + label + '</span><span class="paramDesc">'+ info + '</span></td><td class="inputCell relative" bgcolor="#FFFFFF" width="74%">' + c + '</td><td style="align:center" bgcolor="#FFFFFF" >' + sd + '</td></tr>';
             });
 
         t += '</table>';

From 43965f91f939a1160b95e3908bf7e22666b3a6b0 Mon Sep 17 00:00:00 2001
From: Nicola Lambathakis <modxit@modx.it>
Date: Tue, 1 Nov 2016 16:20:31 +0100
Subject: [PATCH 2/3] restyled plugins config table and setdefault buttons

---
 manager/actions/mutate_plugin.dynamic.php | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/manager/actions/mutate_plugin.dynamic.php b/manager/actions/mutate_plugin.dynamic.php
index d28821e696..c50fe874f0 100644
--- a/manager/actions/mutate_plugin.dynamic.php
+++ b/manager/actions/mutate_plugin.dynamic.php
@@ -128,7 +128,7 @@ function showParameters(ctrl) {
         currentParams = JSON.parse(props);
     }
 
-    t = '<table width="100%" class="displayparams"><thead><tr><td width="1%"><?php echo $_lang['parameter']; ?></td><td width="99%"><?php echo $_lang['value']; ?></td></tr></thead>';
+    t = '<table width="100%" class="displayparams"><thead><tr><td><?php echo $_lang['parameter']; ?></td><td><?php echo $_lang['value']; ?></td><td style="text-align:right;"><?php echo $_lang["set_default"]; ?> </td></tr></thead>';
 
     try {
         var type, options, found, info, sd;
@@ -166,7 +166,7 @@ function showParameters(ctrl) {
                         c = '<input type="text" name="prop_' + key + '" value="' + value + '" size="30" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
                         break;
                     case 'menu':
-                        c = '<select name="prop_' + key + '" style="width:auto" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
+                        c = '<select name="prop_' + key + '" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
                         if (currentParams[key] == options) currentParams[key] = ls[0]; // use first list item as default
                         for (i = 0; i < ls.length; i++) {
                             c += '<option value="' + ls[i] + '"' + ((ls[i] == value) ? ' selected="selected"' : '') + '>' + ll[i] + '</option>';
@@ -175,7 +175,7 @@ function showParameters(ctrl) {
                         break;
                     case 'list':
                         if (currentParams[key] == options) currentParams[key] = ls[0]; // use first list item as default
-                        c = '<select name="prop_' + key + '" size="' + ls.length + '" style="width:auto" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
+                        c = '<select name="prop_' + key + '" size="' + ls.length + '" style="width:100%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">';
                         for (i = 0; i < ls.length; i++) {
                             c += '<option value="' + ls[i] + '"' + ((ls[i] == value) ? ' selected="selected"' : '') + '>' + ll[i] + '</option>';
                         }
@@ -209,7 +209,7 @@ function showParameters(ctrl) {
                         lv = (value + '').split(",");
                         c = '';
                         for (i = 0; i < ls.length; i++) {
-                            c += '<label><input type="checkbox" name="prop_' + key + '[]" value="' +  ls[i] + '"' + ((contains(lv, ls[i]) == true) ? ' checked="checked"' : '') + ' onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />'+ll[i]+'</label>&nbsp;';
+                            c += '<label><input type="checkbox" name="prop_' + key + '[]" value="' +  ls[i] + '"' + ((contains(lv, ls[i]) == true) ? ' checked="checked"' : '') + ' onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />&nbsp;';
                         }
                         break;
                     case 'radio':
@@ -219,18 +219,18 @@ function showParameters(ctrl) {
                         }
                         break;
                     case 'textarea':
-                        c = '<textarea name="prop_' + key + '" style="width:80%" rows="4" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">' + value + '</textarea>';
+                        c = '<textarea name="prop_' + key + '" style="width:100%" rows="4" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)">' + value + '</textarea>';
                         break;
                     default:  // string
-                        c = '<input type="text" name="prop_' + key + '" value="' + value + '" style="width:80%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
+                        c = '<input type="text" style="width:100%" name="prop_' + key + '" value="' + value + '" style="width:80%" onchange="setParameter(\'' + key + '\',\'' + type + '\',this)" />';
                         break;
                 }
 
                 info = '';
                 info += desc ? '<br/><small>' + desc + '</small>' : '';
-                sd = defaultVal != undefined ? ' <ul class="actionButtons" style="position:absolute;right:0px;bottom:6px;min-height:0;"><li><a href="#" class="primary btn-small btnSetDefault" onclick="setDefaultParam(\'' + key + '\',1);return false;"><?php echo $_lang["set_default"]; ?></a></li></ul>' : '';
+                sd = defaultVal != undefined ? '<ul class="actionButtons" style="float:right;margin-top:12px;"><li><a title="<?php echo $_lang["set_default"]; ?>" href="#" class="btnSetDefault" onclick="setDefaultParam(\'' + key + '\',1);return false;"><i class="fa fa-refresh"></i></a></li></ul>' : '';
 
-                t += '<tr><td class="labelCell" bgcolor="#FFFFFF" width="20%"><span class="paramLabel">' + label + '</span><span class="paramDesc">'+ info + '</span></td><td class="inputCell relative" bgcolor="#FFFFFF" width="80%">' + c + sd + '</td></tr>';
+                t += '<tr><td class="labelCell" bgcolor="#FFFFFF" width="20%"><span class="paramLabel">' + label + '</span><span class="paramDesc">'+ info + '</span></td><td class="inputCell relative" bgcolor="#FFFFFF" width="74%">' + c + '</td><td style="align:center" bgcolor="#FFFFFF" >' + sd + '</td></tr>';
             });
 
         t += '</table>';

From 3b34010a833c2169ef28addfe0a31cf06337f18e Mon Sep 17 00:00:00 2001
From: Nicola Lambathakis <modxit@modx.it>
Date: Tue, 1 Nov 2016 16:20:56 +0100
Subject: [PATCH 3/3] updated styles for modules and plugins config table

---
 manager/media/style/MODxRE2/style.css | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/manager/media/style/MODxRE2/style.css b/manager/media/style/MODxRE2/style.css
index 17cf4d2b56..0bb6d94c0b 100644
--- a/manager/media/style/MODxRE2/style.css
+++ b/manager/media/style/MODxRE2/style.css
@@ -2461,6 +2461,7 @@ input#dateto:focus.DatePicker {
 
 /* normalize some td paddings */
 
+
 #tabTemplate tbody td,
 .sectionBody td,
 .sectionBody th {
@@ -2479,19 +2480,26 @@ input#dateto:focus.DatePicker {
 
 .sectionBody .displayparams th,
 .sectionBody .displayparams td {
-  padding: 2px 4px;
+  padding: 4px 4px;
 }
 
 .sectionBody .displayparams thead td,
 .sectionBody .permissiongroup thead td {
   border-top: none;
 }
-
+#displayparams table{
+  background: #fff;
+}
+#displayparams table tbody td {
+  border-bottom: 1px dotted #d3d3d3;
+}
 .sectionBody .permissiongroup input,
 .sectionBody .permissiongroups input {
   width: 200px;
 }
-
+td#displayparams > table.displayparams {
+border: none;
+}
 
 /* replace warning red color in tabs */
 
@@ -3381,4 +3389,5 @@ table.actionButtons .searchtext {
 .userprofiletable {margin-bottom:-20px;}
 
 .btn-small { padding:2px 4px !important; font-size:11px !important; }
-.relative { position:relative; }
\ No newline at end of file
+.relative { position:relative; }
+