diff options
| author | Fatih Acet <acetfatih@gmail.com> | 2016-11-29 05:52:39 +0000 |
|---|---|---|
| committer | Fatih Acet <acetfatih@gmail.com> | 2016-11-29 05:52:39 +0000 |
| commit | e0a46540c3ab4774fd0c39ba5f28ca8909ce76b6 (patch) | |
| tree | f058fd729261230046365d7a40f98d8a8f6f03f3 /app/views | |
| parent | 25c1256736db8a9b1fc72885eb273552416851e8 (diff) | |
| parent | 59fa98dd8462fa2f7865828275b5e80e362e4a6e (diff) | |
| download | gitlab-ce-e0a46540c3ab4774fd0c39ba5f28ca8909ce76b6.tar.gz | |
Merge branch 'hide-project-variables' into 'master'
Hide project variables values by default
Add a button to reveal/hide the values to help prevent accidental disclosure of sensitive information from wandering on a page.
Closes #21358
See merge request !7731
Diffstat (limited to 'app/views')
| -rw-r--r-- | app/views/projects/variables/_table.html.haml | 4 | ||||
| -rw-r--r-- | app/views/projects/variables/index.html.haml | 1 |
2 files changed, 3 insertions, 2 deletions
diff --git a/app/views/projects/variables/_table.html.haml b/app/views/projects/variables/_table.html.haml index 07cee86ba4c..c7cebf45160 100644 --- a/app/views/projects/variables/_table.html.haml +++ b/app/views/projects/variables/_table.html.haml @@ -12,8 +12,8 @@ - @project.variables.order_key_asc.each do |variable| - if variable.id? %tr - %td= variable.key - %td= variable.value + %td.variable-key= variable.key + %td.variable-value{ "data-value" => variable.value }****** %td = link_to namespace_project_variable_path(@project.namespace, @project, variable), class: "btn btn-transparent btn-variable-edit" do %span.sr-only diff --git a/app/views/projects/variables/index.html.haml b/app/views/projects/variables/index.html.haml index 09bb54600af..39303700131 100644 --- a/app/views/projects/variables/index.html.haml +++ b/app/views/projects/variables/index.html.haml @@ -15,3 +15,4 @@ No variables found, add one with the form above. - else = render "table" + %button.btn.btn-info.js-btn-toggle-reveal-values{"data-status" => 'hidden'} Reveal Values |