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 <jftho...@ncsu.edu> 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'; }