-
Notifications
You must be signed in to change notification settings - Fork 59
Feedback on v3 #115
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
Comments
None of the problems mentioned above is more important than #104 though! |
Thanks for the feedback! It does sound that we made the right call to accept #103 and #98, since you also want both of them. To give you some extra details:
|
Thank you for your answers. Please close this issue whenever you want. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here is some feedback on browserpass v3:
I'm very happy that you guys keep on maintaining browserpass. It's a very useful extension. Thank you so much.
I really like the idea of the domain badge to filter the entries. It both brings security and is very easy to get rid of.
I also like that the entries are sorted based on usage and that I can just press RET to activate the first one.
I don't see the benefit of having the number of matching entries always visible as a badge on the extension's icon. It just doesn't bring me any value and it clutters the tool bar. (Add ability to turn off badge on icon #103)
On Firefox, most icons are just black with transparent background. This makes the browserpass icon stand out quite a bit for, in my opinion, no particular reason.
The popup is the only one of all my extensions to be dark. It's not really a problem, but I like consistency :-). (Add light theme #98)
The currently selected entry has a background color which is not different enough from the normal background for me to quickly see what entry is active. My vision is not really good so it might just be me.
The icon to go to the url attached to an entry is gone. I didn't use it often, but it was sometimes very useful.
I also lack support for OTP but that is taken care of separately (#76).
The text was updated successfully, but these errors were encountered: