RANGER-3667 : Improve feedback in policy creation UI when resource do…

…es not exist. Signed-off-by: Nitin Galave <ni3galave@apache.org>
apache · Mar 23, 2022 · d8fc10a · d8fc10a
1 parent f66c559
commit d8fc10a
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 9 deletions.
diff --git a/security-admin/src/main/webapp/scripts/utils/XAUtils.js b/security-admin/src/main/webapp/scripts/utils/XAUtils.js
@@ -1542,12 +1542,18 @@ define(function(require) {
                             return null;
                         }else{
                             return {
-                                id : term,
+                                id : "<b><i class='text-muted-select2'>Create</i></b> " + term,
                                 text: term
                             };
                         }
                     }
                 },
+                formatResult : function(result){
+                    return result.id;
+                },
+                formatSelection : function(result){
+                    return result.text;
+                },
             };
 
         return opts;

diff --git a/security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js b/security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js
@@ -789,8 +789,8 @@ define(function(require){
 								return null;
 							}else{
 								return {
-									id : term,
-									text: term
+									id : "<b><i class='text-muted-select2'>Create</i></b> " + term,
+									text: term,
 								};
 							}
 						}
@@ -831,7 +831,7 @@ define(function(require){
 
 					},	
 					formatResult : function(result){
-						return result.text;
+						return result.id;
 					},
 					formatSelection : function(result){
 						return result.text;

diff --git a/security-admin/src/main/webapp/scripts/views/security_zone/ZoneResourceForm.js b/security-admin/src/main/webapp/scripts/views/security_zone/ZoneResourceForm.js
@@ -361,7 +361,7 @@ define(function(require) {
                                     return null;
                                 } else {
                                     return {
-                                        id: term,
+                                        id: "<b><i class='text-muted-select2'>Create</i></b> " + term,
                                         text: term
                                     };
                                 }
@@ -413,7 +413,7 @@ define(function(require) {
 
                         },
                         formatResult: function(result) {
-                            return result.text;
+                            return result.id;
                         },
                         formatSelection: function(result) {
                             return result.text;

diff --git a/security-admin/src/main/webapp/scripts/views/service/ServiceAuditFilterResources.js b/security-admin/src/main/webapp/scripts/views/service/ServiceAuditFilterResources.js
@@ -347,7 +347,7 @@ define(function(require) {
                                     return null;
                                 } else {
                                     return {
-                                        id: term,
+                                        id: "<b><i class='text-muted-select2'>Create</i></b> " + term,
                                         text: term
                                     };
                                 }
@@ -399,7 +399,7 @@ define(function(require) {
 
                         },
                         formatResult: function(result) {
-                            return result.text;
+                            return result.id;
                         },
                         formatSelection: function(result) {
                             return result.text;

diff --git a/security-admin/src/main/webapp/styles/xa.css b/security-admin/src/main/webapp/styles/xa.css
@@ -980,7 +980,7 @@ ul.tagit li.tagit-choice .tagit-close {
 }
 
 .ui-menu.ui-widget.ui-widget-content.ui-autocomplete.ui-front.tagit-autocomplete {
-  width: 50% !important;
+  width: 21% !important;
 }
 
 table.backgrid tr > td > .label {
@@ -2872,6 +2872,10 @@ div#zoneServiceAccordion table thead {
   opacity: 0.7;
 }
 
+.text-muted-select2 {
+  opacity: 0.5;
+}
+
 .center {
   text-align: center;
 }