Password Strength Indicator

Littlegolden

Kylo It seems that translated text cannot be detected. . .

Kylo

`0.1.1`

Add Chinese & Japanese translations.

DursunCan

after beta 12 is released

#  php composer.phar why-not flarum/core v0.1.0-beta.12
the-turk/flarum-password-strength  v0.1.1          requires          flarum/core (>=0.1.0-beta.10 <0.1.0-beta.12)

Kylo

DursunCan

`0.1.2`

Beta 12 compatibility.

Kylo

`0.1.3`

Add Indonesian translations.
Update version constraints for Flarum 0.1.0-beta.13.

Kylo

Update time!

`1.0.0`

Indicator

I practically rewritten and redesigned the whole thing.

Load zxcvbn only when it is necessary (via CDN).
Now we have three labels as 'Weak', 'Could be stronger' and 'Strong' instead of five. Also you can't disable these labels from the settings modal anymore.
- Strength score is 0/4 or 1/4 -> Weak
- Strength score is 2/4 or 3/4 -> Could be stronger
- Strength score is 4/4 -> Strong
Add password visibility toggle option.
- Okay I know we already have Show Password for this. But I just wanted to do something that'll look good together with the strength indicator.
Remove "Change input's background color with password strength score" option.
Run prettier for all JS files.

fakruzaruret

Kylo perfect

matteocontrini

Kylo congratulations, it's awesome!

EDIT: could you perhaps make the "Password" placeholder text centered? It doesn't look very good when username and email are centered. (Especially because I've simply set "Password" as the placeholder text.)

Kylo

matteocontrini Thanks!

Actually I just decided to remove custom placeholders. I think it'll be more customizable this way.

`1.0.1`

Remove custom placeholders.

Kylo

`1.0.2`

Placeholders weren't fully centered thanks to padding-right: 50px; value that comes with the eye icon. Fixed this by adding padding-left: 50px;. I also removed the unnecessary !important property on paddings so RTL readers can relocate the icon to the left and use right aligned placeholder.

Known issues:

Password visibility toggle conflicts with the therealsujitk/flarum-ext-show-password

Ralkage

Kylo Known issues:

Password visibility toggle conflicts with the therealsujitk/flarum-ext-show-password

You can add "conflict" to your composer.json for this project mentioning this package if you'd like 🙂

Kylo

Ralkage they can disable the eye toggler that comes with this extension and use the Show Password extension's checkbox toggler with the strength indicator 👨‍💻

Kylo

Can someone test this on Firefox mobile? When you switch the password visibility multiple times while you typing it's going nuts and begin to type random things (at some point you can't even remove them). I've no idea what the hell is going on there (the console gives me nothing), need someone to confirm it so I can dig deeper. Maybe it's my setup or phone or a browser bug (seems fine on Chrome mobile) idk...

Ralkage

Kylo iOS or Android?

Kylo

Ralkage Android

Umbe

Kylo Hi, when you release an update for the new flarum version? Thanks

iPurpl3x

Umbe I am also looking forward to an update for beta.15. If @Kylo doesn't do it, I will fork the repo and update it in the coming weeks.

DursunCan

iPurpl3x can read this answer
https://discuss.flarum.org/d/22779-diff-for-flarum/132

iPurpl3x

DursunCan OK good to know, then I will take over as soon as I have time to do it.