This is an automated email from the ASF dual-hosted git repository.
jfthomps pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/vcl.git
The following commit(s) were added to refs/heads/develop by this push:
new 08bf9514 VCL-1144 - minor improvements to Resource grouping and
mapping page
08bf9514 is described below
commit 08bf9514e36f63ac654d91fbce634b7e21c6ace5
Author: Josh Thompson <[email protected]>
AuthorDate: Tue Feb 20 16:40:39 2024 -0500
VCL-1144 - minor improvements to Resource grouping and mapping page
resource.php:
-modified groupByResourceHTML, groupByGroupHTML, mapByResGroupHTML, and
mapByMapToGroupHTML: changed width of DataGrids to match the width of their
containing tables
-modified mapByResGroupHTML and mapByMapToGroupHTML: added ContentPane for
containing message about no access for users without mapping access, previously
the no access message was poorly displayed and not in its own tab
resources.js: modified Resource.prototype.addRemItemCB: added check for
data.items.status == 'invalid'
---
web/.ht-inc/resource.php | 22 ++++++++++++++--------
web/js/resources.js | 3 +++
2 files changed, 17 insertions(+), 8 deletions(-)
diff --git a/web/.ht-inc/resource.php b/web/.ht-inc/resource.php
index f6ea570c..4ba93ec8 100644
--- a/web/.ht-inc/resource.php
+++ b/web/.ht-inc/resource.php
@@ -801,7 +801,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -830,7 +830,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -984,7 +984,7 @@ class Resource {
$h .= "selectionMode=\"extended\" sortInfo=\"1\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -1013,7 +1013,7 @@ class Resource {
$h .= "selectionMode=\"extended\" sortInfo=\"1\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -1164,7 +1164,10 @@ class Resource {
$h = '';
if(! count($groups) || ! count($mapgroups)) {
+ $h .= "<div id=\"mapbyresgroupdiv\"
dojoType=\"dijit.layout.ContentPane\" ";
+ $h .= "title=\"" . i("Map By {$this->restypename}
Group") . "\">\n";
$h .= i("You don't have access to manage any mappings
for this resource type.");
+ $h .= "</div>\n";
return $h;
}
@@ -1189,7 +1192,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -1220,7 +1223,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -1357,8 +1360,11 @@ class Resource {
$h = '';
if(! count($mapgroups) || ! count($groups)) {
+ $h .= "<div id=\"mapbymaptogroupdiv\"
dojoType=\"dijit.layout.ContentPane\" ";
+ $h .= "title=\"" . i("Map By {$this->maptypename}
Group") . "\">\n";
$h .= "You don't have access to manage any mappings for
this resource ";
$h .= "type.<br>\n";
+ $h .= "</div>\n";
return $h;
}
@@ -1383,7 +1389,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
@@ -1412,7 +1418,7 @@ class Resource {
$h .= "selectionMode=\"extended\">\n";
$h .= "<thead>\n";
$h .= "<tr>\n";
- $h .= "<th field=\"name\" width=\"160px\"></th>\n";
+ $h .= "<th field=\"name\" width=\"220px\"></th>\n";
$h .= "</tr>\n";
$h .= "</thead>\n";
$h .= "</table>\n";
diff --git a/web/js/resources.js b/web/js/resources.js
index aa0dd5f7..6a15e614 100644
--- a/web/js/resources.js
+++ b/web/js/resources.js
@@ -116,6 +116,9 @@ Resource.prototype.addRemItemCB = function(data, ioArgs) {
else if(data.items.status == 'noaccess') {
alert(_('You do not have access to the submitted resource or
group'));
}
+ else if(data.items.status == 'invalid') {
+ alert(_('Invalid value submitted'));
+ }
document.body.style.cursor = 'default';
}