-
-
Notifications
You must be signed in to change notification settings - Fork 140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace button "Show" with Icon to show/hide passphrase #1821
Replace button "Show" with Icon to show/hide passphrase #1821
Conversation
This looks pretty, pretty good |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Screen.Recording.2023-11-03.at.11.17.40.AM.mov
The pin slightly shifts to the left when we tap the Eye icon to show it.
Happens only in the IOS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The delete button on the SetPassword
view should not have the "red" color; it should have the configuration like color: themeColor('delete'),
also probably add the confirmation before deleting it as well like we do in the Contacts and Nodes view
Unfortunately I cannot test iOS. If you can find a solution to this, that would be awesome!
When you have a second input for confirming, you generally don't need to show the password. It is usually either confirm by 2nd input, or show/hide.
Yeah, I think padding is something that we can improve on many places throughout the app.
Good idea, will do that! (Edit: in a different PR though) |
Sure. will try to check what's going on in the ios
Okay, sounds right. |
733e06a
to
a075187
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK
Description
This fixes #1758.
Button "Show" is replaced with eye icon to show/hide passphrase at login. Implemented same logic for PIN (was missing completely). On PIN screen eye icon won't be displayed when no digit was entered yet.
Before:
![before-pass](https://private-user-images.githubusercontent.com/77545287/280150209-ccb3fa85-18fa-4e90-aed6-c07ce22a7a04.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyMDktY2NiM2ZhODUtMThmYS00ZTkwLWFlZDYtYzA3Y2UyMmE3YTA0LlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdlMDBiNWVmMGRjYzAzOWI2ODQwMjI1ODI2Njc1NjhhZmM4MTZhYjVlYTExODc0MDQ2MzdjZDc5NWRiZWZkY2QmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.BFBbgzjHHo4554lIQH-BJq3yqWUNoHts1O-yMpsm0ao)
![before-pin](https://private-user-images.githubusercontent.com/77545287/280150212-1f73459b-4f27-4cec-a98f-3dc517889c93.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyMTItMWY3MzQ1OWItNGYyNy00Y2VjLWE5OGYtM2RjNTE3ODg5YzkzLlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTUxMGRkM2E5NjNjOTU2YjI5ZmQ4YmRlNTEwYjhkMWMxMGYxOTE1OWJmYzI0YTYwMGRhOTRhZDM3Y2Q1NDRiYmEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.S-Xlr1grt9IfEX7iUg8Toa9vK9OtJMQuYAYOLYonoaE)
After:
![after-pass-hidden](https://private-user-images.githubusercontent.com/77545287/280150289-dda6272d-bdc7-405f-9b73-bc17942ff7da.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyODktZGRhNjI3MmQtYmRjNy00MDVmLTliNzMtYmMxNzk0MmZmN2RhLlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTNiYWE1MWI3Mjg0ZWM5ZWQ1OGQyMTNkYmQzN2Y3MjFmNWNlMWY1Y2JlZmYyNTBmMzU3Yjg4MDczYmMwMTI5ZjImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.5XwqAZAQ_0eapKQbP1BgUeIdxgbH44tlVX4DELGHdBM)
![after-pass-visible](https://private-user-images.githubusercontent.com/77545287/280150292-bbfcbfec-8e49-47cc-8938-dcb2ea49cbea.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyOTItYmJmY2JmZWMtOGU0OS00N2NjLTg5MzgtZGNiMmVhNDljYmVhLlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWVjZjI4YjkwNjJhZWM0YjQ0ZmJlNzMxODJmYzI1NTYyMjdiNGRhNjA3NDEzZGUzOTU4ZjMwMGVjZjhiY2RjZGImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.-rvuqz09Q_ZwDpdpTQSK61dgddCzeP7U8Cc679ZXeec)
![after-pin-hidden](https://private-user-images.githubusercontent.com/77545287/280150293-3d1e6806-f7cb-4813-8e1d-5da917228517.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyOTMtM2QxZTY4MDYtZjdjYi00ODEzLThlMWQtNWRhOTE3MjI4NTE3LlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZkNjY1ZGY2YmMyYzZiN2UwMzA0OTIyNTQwNjlhYzMzNTczMmY4ZjkxMDY4YWEwODcyMWRjZGE5NmVhMTFhMjYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.MId4E4P30opohmthDTYWJEobD9nKGzI4vApn7yY3dLg)
![after-pin-visible](https://private-user-images.githubusercontent.com/77545287/280150294-ac5a2d94-6637-4fab-9298-81ba1d0adf77.PNG?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMjg5MjEsIm5iZiI6MTcyMjEyODYyMSwicGF0aCI6Ii83NzU0NTI4Ny8yODAxNTAyOTQtYWM1YTJkOTQtNjYzNy00ZmFiLTkyOTgtODFiYTFkMGFkZjc3LlBORz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI4VDAxMDM0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWNiNjY5YzMyZDZiMzZmYTU3NzU0YjdmNmU4YWJiYjE2M2MzMWYxZDI4MTAyZTcyZGRiYzdiM2RjNWRjYjllYWQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.iJjF-Ltqi93P1nrgdIx4Tjnp0-R91RbuYJbFVEJv5mY)
This pull request is categorized as a:
Checklist
yarn run tsc
and made sure my code compiles correctlyyarn run lint
and made sure my code didn’t contain any problematic patternsyarn run prettier
and made sure my code is formatted correctlyyarn run test
and made sure all of the tests passTesting
If you modified or added a utility file, did you add new unit tests?
I have tested this PR on the following platforms (please specify OS version and phone model/VM):
I have tested this PR with the following types of nodes (please specify node version and API version where appropriate):
Locales
Third Party Dependencies and Packages
yarn
after this PR is merged inpackage.json
andyarn.lock
have been properly updatedOther: