You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ni...@apache.org on 2022/03/23 06:56:05 UTC

[ranger] branch ranger-2.3 updated: RANGER-3667 : Improve feedback in policy creation UI when resource does not exist.

This is an automated email from the ASF dual-hosted git repository.

ni3galave pushed a commit to branch ranger-2.3
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/ranger-2.3 by this push:
     new bbd8427  RANGER-3667 : Improve feedback in policy creation UI when resource does not exist.
bbd8427 is described below

commit bbd8427e4d16334ef605b918781aad64f466744f
Author: Dhaval.Rajpara <dh...@gmail.com>
AuthorDate: Mon Mar 21 17:04:28 2022 +0530

    RANGER-3667 : Improve feedback in policy creation UI when resource does not exist.
    
    Signed-off-by: Nitin Galave <ni...@apache.org>
---
 security-admin/src/main/webapp/scripts/utils/XAUtils.js           | 8 +++++++-
 .../src/main/webapp/scripts/views/policies/RangerPolicyForm.js    | 6 +++---
 .../main/webapp/scripts/views/security_zone/ZoneResourceForm.js   | 4 ++--
 .../webapp/scripts/views/service/ServiceAuditFilterResources.js   | 4 ++--
 security-admin/src/main/webapp/styles/xa.css                      | 6 +++++-
 5 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/security-admin/src/main/webapp/scripts/utils/XAUtils.js b/security-admin/src/main/webapp/scripts/utils/XAUtils.js
index fdebfdb..699871f 100644
--- a/security-admin/src/main/webapp/scripts/utils/XAUtils.js
+++ b/security-admin/src/main/webapp/scripts/utils/XAUtils.js
@@ -1541,12 +1541,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
index 5b3940a..d4452a8 100644
--- 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
index 68092fd..7242d7b 100644
--- 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
index 2cfdfd9..115486e 100644
--- 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
index ef3347b..ff99fb1 100644
--- 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;
 }