Skip to content

Commit

Permalink
Merge pull request #3982 from omgitsmiles/WV-316-mobile-sign-in-page-…
Browse files Browse the repository at this point in the history
…workflow-discrepancy-in-mobile-apps-based-on-user-data-correct-vs-incorrect-data

WV-316 added commented out code back to Cordova, and added prop check into in isMobileScreenSize [TEAM REVIEW]
  • Loading branch information
DaleMcGrew committed Aug 15, 2024
2 parents 841c47b + d19ce99 commit 622dc88
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions src/js/components/Settings/VoterPhoneVerificationEntry.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -303,12 +303,15 @@ class VoterPhoneVerificationEntry extends Component {
if (cancelShouldCloseModal) {
this.closeSignInModalLocal();
} else if (isCordova()) {
if (this.props.showAllSignInOptions) {
// WV-316: seperated Cordovoa and Mobile screen cancel logic, Mobile only shows all sign in options on cancel.
}
// WV-316: seperated Cordovoa and Mobile screen cancel logic, Mobile only shows all sign in options on cancel.
// if (this.props.showAllSignInOptions) {
// this.props.showAllSignInOptions();
// }
} else if (isMobileScreenSize()) {
this.props.showAllSignInOptions();
}
if (this.props.showAllSignInOptions) {
this.props.showAllSignInOptions();
};
};
};

onFocus = () => {
Expand Down

0 comments on commit 622dc88

Please sign in to comment.