Improve password generator

[kcinay055679]

No description provided.

[kcinay055679]

Functional everything is fine.

TODO:

• Design
• Tests

[Robin481]

Looking quite nice.
However, I don't think the interface makes a lot of sense at this point.
Currently I would argue 95% of the time a user doesn't create a password in Cryptopus but rather saves an already existing password.

Therefore, the use case of going to an encryptable which already has a password and randomly generating a new one that then automatically overwrites the old one doesn't really exist in my opinion and is rather confusing.

Maybe a easy solution would be to not automatically overwrite the existing password but rather have a button that says "create new secure password" instead of always overwriting when using the slider or checking the symbol box.

[Robin481]

Looking quite nice.
However the button takes up too mache space now.
What about putting it there and making it smaller? Think that would be the best of both worlds.

[Robin481]

Looks very nice!
As discussed with @kcinay055679 a nice improvement would be to maybe not show the password generator when the field is already pre-filled. Work out what you think is a nice idea and let's discuss it :)

[kcinay055679]

I added a accordion to the password generator witch is expanded on default and auto collapses after generating a new password. But the Accordeon generated by my code isnt the same as the one from the docs. I think the reason for that behavior is that we are currently using bootstrap 4.6.

[Robin481]

@kcinay055679
Good start.
Make the dropdown smaller, give it a bottom margin and add an arrow to the dropdown that actually indicates that this is a drodopwn, something like this.

(and fix the styling errors)

[kcinay055679]

The requested arrow is default functionality according to the ember docs, but in our project the arrow is simpy not there.
i think this is because we're using bootstrap 4

outdated