Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes minor bootstrap 3 issues with login form. #438

Merged
merged 1 commit into from
Oct 25, 2014

Conversation

cluebattery
Copy link
Contributor

Fixes the floating checkbox over the "remember me" label per the basic example provided in Bootstrap 3's documentation.

The default bootstrap CSS applies a negative left margin to the checkbox, so the box should come before the label text. Also, the containing div should be classed as a "checkbox" not "form-control." Also added the 'role="form"' to the form declaration and moved the "forgot password" link to a paragraph classed "help-block" below the password entry for consistency with Bootstrap 3 documentation.

Fixes the floating checkbox over the label per the basic example provided in Bootstrap 3's documentation. 

The default bootstrap CSS applies a negative left margin to the checkbox, so the box should come before the label text. Also, the containing div should be classed as a "checkbox" not "form-control." Also added the 'role="form"' to the form declaration and moved the "forgot password" link to a paragraph classed "help-block" below the password entry for consistency with Bootstrap 3 documentation.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 06b18ba on cluebattery:master into c1280c4 on Zizaco:master.

@Zizaco
Copy link
Owner

Zizaco commented Oct 25, 2014

Thanks for this!

Zizaco added a commit that referenced this pull request Oct 25, 2014
Fixes minor bootstrap 3 issues with login form.
@Zizaco Zizaco merged commit 64c2e57 into Zizaco:master Oct 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants