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

Removing margins from :first-child / :last-child navbar-form #10549

Closed
wants to merge 2 commits into from
Closed

Removing margins from :first-child / :last-child navbar-form #10549

wants to merge 2 commits into from

Conversation

drewcovi
Copy link

@drewcovi drewcovi commented Sep 9, 2013

Navbar Forms that are first or last children and pulled right or left currently have left and right margins.

.navbar-nav.navbar-left, and .navbar-nav.navbar-right currently strip this margin away and allow elements within to justify to the left or right.

This update allows buttons in navbar-forms to align with other header / footer elements and when fixed to the top or bottom, also align with the elements on the page.

Navbar forms are not properly left and right aligning when first or
last children accordingly.  This leads to submit buttons out of
alignment with other navbar / header / footer elements, and all page
content when navbars are fixed.
Making sure to include compiled bootstrap css and min.css for pull
request
@mdo
Copy link
Member

mdo commented Dec 2, 2013

This is no longer necessary per #11530.

@mdo mdo closed this Dec 2, 2013
@drewcovi
Copy link
Author

drewcovi commented Dec 2, 2013

cool so this will now sit flush for left aligned forms too? thanks for the update.

@mdo
Copy link
Member

mdo commented Dec 2, 2013

@drewcovi Ah, good point—didn't look at the :first:child situations. Will take a peak. Let me know if you run into any other problems with the new changes.

@mdo
Copy link
Member

mdo commented Dec 2, 2013

@drewcovi Looks like there are some more thing to work on: http://jsbin.com/odeSiSo/5/. I'll get on them later tonight perhaps, unless you want first dibs :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants