Move Unlock Button on Pin Screen

vachan · October 17, 2020, 2:30am

The buttons will be re positioned in the next update. The pull request is open on Github and it should be merged pretty soon. For more info:

github.com/bitwarden/mobile

reposition buttons in Log In and Unlock pages

bitwarden:master ← bitwarden:feature-improve-login-ui

opened 10:22PM - 17 Sep 20 UTC

eliykat

+74 -15

## Description I have repositioned the buttons in the Log In and Unlock (i.e. V…erify Master Password) pages as discussed in [this Community Forums thread](https://community.bitwarden.com/t/reposition-buttons-in-verify-master-password-and-log-in-screens-on-mobile/14326). The intention of this is to improve the positioning of primary buttons and the UX flow as the user moves through these pages. *Log In page:* move Log In button to primary location below password entry, move Get Your Master Password Hint to the More Options menu. [Screenshot](https://user-images.githubusercontent.com/31796059/93534423-6561d400-f988-11ea-803c-237a11d79e4c.PNG) *Unlock page (Verify Master Password):* swap position of Unlock and Log Out buttons. [Screenshot](https://user-images.githubusercontent.com/31796059/93534441-6e52a580-f988-11ea-82a2-bddc5bf2f909.PNG) ## Changes not implemented In the Community Forums thread, we discussed putting an icon on the Unlock button and changing the theming of primary buttons (i.e. to use the Bitwarden colour scheme rather than the current grey). However, this was more complicated than anticipated to implement properly and to get it looking good across all devices. I also think that any change to button theming should be implemented consistently across the app, as these buttons appear in other places that were not the subject of this contribution. I still think this needs to be changed, but it should be a separate contribution so that it can be properly considered and implemented consistently across the app. ## Testing I have tested this in iOS and Android and it all works well subject to the following. It appears that the Get Your Master Password Hint button is hidden when launching the login flow a certain way in iOS. See the changes to CredentialProviderViewController.cs and LoadingViewController.cs. I have updated these references as required but I was unable to test them because of issues with my iOS development environment. (Briefly, I don't have a provisioning profile or Apple Developer account, so I had to disable custom entitlements to get this to build, which broke certain features of the app such as autofill.) I recommend someone does a quick test of this login flow before accepting the PR.