LGTM but I'd like to see it in action to check how it behaves upon resizing the browser window

@strk I will add it to my server and fyi it will put the scrollbar the moment there is an overflow.

@strk https://staging.geek1011.net

Ok now I see it. The scrollbar kicks in too late so I can handle to have a view with truncated labels rather than a scrollbar.

Attaching screenshot:

btw, maybe it would be good to rename Email Addresses to Email and Two-Factor Authentication to 2FA (not sure about this)

@strk I'll rename that in another PR. I have updated my server with the bugfixes.

Better, thank you.

I just spotted this overlap in current master, dunno if your branch also fixes this (btw, needs to resolve conflicts)

@strk I fixed it in #1316

Yes @geek1011 it conflicted after the merge of the OpenID branch. Please fix the conflict and rebase.

Only templates/user/auth/signin_inner.tmpl is left.

Fixed, sorry about the delay.

@appleboy, @lunny can you please review/merge this ?

@strk @geek1011 this will conflict with #1010

@lunny you found #1010 broken while this is ready, let's merge this and let conflicts be resolved in #1010

Is this ready to review?

Yes

chrome on mac the github login button is not center.

any news?

I'm closing this PR, I will rewrite it with a better design because this is conflicted and will not look good with more than 2 providers.

@lunny #1777