GB Password Strength

I am very glad that you have recovered this extension!!! it was a very good idea from @Kylo

Add that it is important to disable the Show Password extension as they conflict with each other as you can see in the images:

I have had to disable the extension due to a very critical bug, with the extension enabled it does not allow users to log in, I have not yet been able to test if it allows registration. Here are the console errors when trying to log in:

I will try to get more details in a few hours.

Trombert After being a while the truth that I have not managed to achieve if the culprit is another extension (this being complicated with 71 extensions installed ) that conflicts, I would say no, if someone has it installed and it works would be a great favor to pass your php flarum info to rule out various extensions.

askvortsov Surprisingly it wasn't the username change request extension either, it turns out that the culprit is the password visibility toggle switch built into the extension itself, there seems to be something wrong there that causes this error.

By deactivating it, it has started to work without any problem, even enabling some extensions I thought suspicious.

DursunCan CSS broken with the Sign Up Confirm Password extension.

Thank you for the feedback, I'll take a look today.

Trombert Add that it is important to disable the Show Password extension as they conflict with each other

I will add a note for that or add something that makes them work seamlessly together, thanks for letting me know.

Trombert it turns out that the culprit is the password visibility toggle switch built into the extension itself, there seems to be something wrong there that causes this error.

And thanks for your extensive testing, I will try to fix that in the next update.

@DursunCan & @Trombert This update should fix your issues. Thanks again for your feedback.

Release v2.0.1

Thank you for reporting bugs and feedback!

Changelog:

• Fixed things so that it is actually possible to log-in or sign-up. In the previous release, the value of the password field was not saved correctly .
• Compatibility with nearata/flarum-ext-signup-confirm-password
• Conflict with therealsujitk/flarum-ext-show-password

Is there a reason why this extension amounts for almost 1 MB of the forum bundle? it's like double what core weighs...

Seems to be related to bundling the entire zxcvbn here.
Dropbox has some suggestions to improve the performance hit,I am still looking around to understand how flarum works but I think that it shouldn't be too difficult to load the library asyncronously.

wonderbeel Flarum doesn't support lazy loading, but it's always possible to load an external file "on demand". For example the Emoji Picker extension by @clarkwinkelmann does that, and I think also @Kyrne's Websocket extension.

Yup, something similar to this should work in this case.
BTW looking at the issues it seems that zxcvbn isn't mantained anymore and that there is a newer fork that supports i18n and multiple keyboard layouts, maybe it is better to make the switch?

wonderbeel Good to know, I will take a look when I have some time.

@anyModerator please add incompatible tag as it is not supported for 16 beta

[deleted] thanks for letting us know, feel free to use the flag functionality next time

Cross posting from here

I've been commissioned by @glowingblue to update this extension in the coming days ready for Flarum 1.0

Either myself or @glowingblue / @iPurpl3x will post back here once it's ready