Add feature to show login formular as collapsed by default

[konsumate]

Use-Case

With external authentication services, such as CAS or SAML, the landing/login page shows the external service button and the login formular (user, password field). Even tho there is a "Login with CAS/SAML/xyz ..." button shown, some part of the users still feel like trying to use the username/password formular - it seems like some natural reaction to login formulars.

Solution
In order to misguide them less on the login page, there could be an optional setting to have the login formular collapsed by default in order to surpress the urge to use the username/password field ..

[lucasvanhalst]

Do you mean the 'Show form-based Login' setting (under admin-> accounts)?
I assume that setting hides the username/password login form

[geekgonecrazy]

@LVH1 That's exactly what that option does. Just verified it.

@CoreCache can you give that option a shot and see if that does the trick for you?

[konsumate]

@LVH1 and @geekgonecrazy: I don't want the login form to be omitted completely. The Admin user of the RC should still be able to use the user/pass login formular, while users authed by external services will use their Login method at the Login layout.

The Option you described above hides the username/password field completely, it is not collapsed, but "omitted" - here is the difference:

Hidden: External users use their Login Service button. The local admin user is now unable to login back into RC without fiddling in the settings manually to re-enable the login/formular

Collapsed: External users use their Login Service button. The local admin user can uncollapse the user/pass formular by a click on a small icon/plus-sign on the login layout and is able to login afterall.

[konsumate]

Also I wanted to note that "Show form-based Login" should really be called "Enable form-based login" as it has the potential to lockout admnis aswell - or at least warn users about it :)

[gregharvey]

My 2-cents, I had to Google to find this setting - I assumed it didn't exist because I couldn't see it anywhere, mainly because "form-based" didn't mean anything to me. :-)

So +1 for a minor wording change - my suggestion would be:

"Enable Rocket.Chat default login form"

That's really clear, no way anyone can possibly not understand what it means. Happy to make a PR if it will help?

[gregharvey]

Did it anyway. You can reject it if you disagree. :-)

[konsumate]

I wouldn't even call it "Rocket.Chat login form" - I guess the correct term would be "Password login form" since the internal service is also called "services.password".

BESIDE that, the commit is totally not the point of this issue. This issue is about beeing able to "collapse/display" the login form - not hiding it statically. Please read above for details.