Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 5639756

Browse files
author
vikasrohit
committedNov 30, 2015
SUP-1262, Clicking the hide/show password button should work on mobile
-- Implemented required behaviour by removing focusOn directive as it is causing either an unpredictable chain of events or not allowing the model change event to be fired on manual update of model in onBlur event.
1 parent d0e9eb9 commit 5639756

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed
 

‎app/directives/account/toggle-password/toggle-password.directive.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@
4040
if (relatedTarget.attr('type') === 'checkbox' && relatedTarget.attr('id') === 'currentPasswordCheckbox') {
4141
vm.currentPasswordFocus = true;
4242
vm.currentPasswordPlaceholder = '';
43+
currentPasswordInput.focus();
4344
} else {
4445
// If you are blurring from the password input and clicking anywhere but the checkbox
4546
vm.currentPasswordFocus = false;

‎app/directives/account/toggle-password/toggle-password.jade

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ input#current-password-input(
22
ng-model="vm.currentPassword",
33
ng-model-options="{allowInvalid: true}",
44

5-
focus-on="focusOnCurrentPasswordInput",
65
ng-focus="vm.onCPFocus($event)",
76
ng-blur="vm.onCPBlur($event)",
87

0 commit comments

Comments
 (0)
Failed to load comments.