You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by me...@apache.org on 2018/07/26 09:10:34 UTC
ranger git commit: RANGER-2161: Improvement in policy screen
permission item's.
Repository: ranger
Updated Branches:
refs/heads/master d352dfaab -> 2e73aaae9
RANGER-2161: Improvement in policy screen permission item's.
Signed-off-by: Mehul Parikh <me...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/2e73aaae
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/2e73aaae
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/2e73aaae
Branch: refs/heads/master
Commit: 2e73aaae97b25a316f64dbce211c6d8c800ba7c0
Parents: d352dfa
Author: ni3galave <ni...@gmail.com>
Authored: Wed Jul 25 15:13:31 2018 +0530
Committer: Mehul Parikh <me...@apache.org>
Committed: Thu Jul 26 14:38:34 2018 +0530
----------------------------------------------------------------------
.../src/main/webapp/scripts/modules/XAOverrides.js | 2 +-
security-admin/src/main/webapp/styles/xa.css | 8 +++++++-
.../templates/policies/RangerPolicyForm_tmpl.html | 12 ++++++------
3 files changed, 14 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ranger/blob/2e73aaae/security-admin/src/main/webapp/scripts/modules/XAOverrides.js
----------------------------------------------------------------------
diff --git a/security-admin/src/main/webapp/scripts/modules/XAOverrides.js b/security-admin/src/main/webapp/scripts/modules/XAOverrides.js
index 3832916..c45f74d 100644
--- a/security-admin/src/main/webapp/scripts/modules/XAOverrides.js
+++ b/security-admin/src/main/webapp/scripts/modules/XAOverrides.js
@@ -398,7 +398,7 @@
* Adds the editor to the DOM
*/
render: function() {
- var that = this, attrs = { 'name': this.key };
+ var that = this, attrs = { 'name': this.key.replace(/\./g, '_')};
if(this.schema.editorAttrs){
attrs = _.extend(attrs , this.schema.editorAttrs);
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/2e73aaae/security-admin/src/main/webapp/styles/xa.css
----------------------------------------------------------------------
diff --git a/security-admin/src/main/webapp/styles/xa.css b/security-admin/src/main/webapp/styles/xa.css
index b895331..db96ed2 100644
--- a/security-admin/src/main/webapp/styles/xa.css
+++ b/security-admin/src/main/webapp/styles/xa.css
@@ -1254,7 +1254,9 @@ ul.tabs > li > a {
margin-bottom: 18px;
margin-left: 1px;
padding: 8px 5px;
- background-color: #f5f5f5;
+ background-color: #eeeeef;
+ border-bottom: 1px solid #64a424;
+ text-shadow: 0px 0px 0px #222;
}
/* loading indicator for button */
@@ -2384,4 +2386,8 @@ textarea:read-only{
}
.validSyntax {
color: #3e910f;
+}
+.borderNone {
+ border: none;
+ padding: 0px;
}
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/ranger/blob/2e73aaae/security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html
----------------------------------------------------------------------
diff --git a/security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html b/security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html
index 9596c6f..b7666f9 100644
--- a/security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html
+++ b/security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html
@@ -53,10 +53,10 @@ language governing permissions and limitations under the License. --}}
</div>
{{#compare "Allow" "eq" conditionType}}
<div class="form-indent-right" data-js="allowExcludePerm">
- <p class="wrap-header reportSearchHeader ">Exclude from Allow
- Conditions :</p>
+ <p class="wrap-header reportSearchHeader "><i class="icon-warning-sign icon-1 text-color-red"></i>
+ Exclude from Allow Conditions :</p>
- <div class="wrap position-relative">
+ <div class="wrap position-relative borderNone">
<div class="" data-customfields="groupPermsAllowExclude">
<div class="control-group" style="margin-left: -100px;">
@@ -86,10 +86,10 @@ language governing permissions and limitations under the License. --}}
</div>
</div>
<div class="form-indent-right">
- <p class="wrap-header reportSearchHeader">Exclude from Deny
- Conditions :</p>
+ <p class="wrap-header reportSearchHeader"><i class="icon-warning-sign icon-1 text-color-red"></i>
+ Exclude from Deny Conditions :</p>
- <div class="wrap position-relative">
+ <div class="wrap position-relative borderNone">
<div class="" data-customfields="groupPermsDenyExclude">
<div class="control-group">
<label class="control-label">Exclude :</label>