Merge branch '31510-mask-password-field-edit' into 'master'

Show password field mask while editing service settings

Closes #31510

See merge request !11431
This commit is contained in:
Phil Hughes 2017-05-22 07:49:58 +00:00
commit 6233e56e99
3 changed files with 7 additions and 3 deletions

View File

@ -9,7 +9,7 @@
.form-group
- if type == "password" && value.present?
= form.label name, "Change #{title}", class: "control-label"
= form.label name, "Enter new #{title.downcase}", class: "control-label"
- else
= form.label name, title, class: "control-label"
.col-sm-10
@ -22,6 +22,6 @@
- elsif type == 'select'
= form.select name, options_for_select(choices, value ? value : default_choice), {}, { class: "form-control" }
- elsif type == 'password'
= form.password_field name, autocomplete: "new-password", class: 'form-control'
= form.password_field name, autocomplete: "new-password", class: "form-control"
- if help
%span.help-block= help

View File

@ -0,0 +1,4 @@
---
title: Update password field label while editing service settings
merge_request: 11431
author:

View File

@ -211,7 +211,7 @@ class Spinach::Features::ProjectServices < Spinach::FeatureSteps
end
step 'I should see empty field Change Password' do
expect(find_field('Change Password').value).to be_nil
expect(find_field('Enter new password').value).to be_nil
end
step 'I click JetBrains TeamCity CI service link' do