Xyphyn / photon

A replacement for lemmy-ui with more features, a better design, and more customizability.
https://phtn.app
GNU Affero General Public License v3.0
344 stars 38 forks source link

Add some password length info for user #301

Closed gapetto closed 4 months ago

vercel[bot] commented 4 months ago

@gapetto is attempting to deploy a commit to the Xylight's projects Team on Vercel.

A member of the Team first needs to authorize it.

Xyphyn commented 4 months ago

It'd be better if the password was just invalidated under 10 characters in the textinput, as this would get the point across while using the browser's tools. Don't worry about it- i'll add this, and will credit you in the upcoming release for bring this to my attention.

gapetto commented 1 month ago

@Xyphyn this is still an issue - I think users may not notice the popup in the right bottom corner as in current state, they'll think that something is broken and eventually they just give up signing up.

Xyphyn commented 1 month ago

I set a minlength and the popup will appear directly under the password input, and the password would be red.

gapetto commented 1 month ago

Sounds good as long as it's clearly visible. My user counter hasn't moved a lot for a long time and I just wonder how much of a factor this is.