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

Removed the unnecessary override and the !important from .wrap #10729

Merged
merged 2 commits into from
Oct 21, 2013
Merged

Removed the unnecessary override and the !important from .wrap #10729

merged 2 commits into from
Oct 21, 2013

Conversation

lipis
Copy link
Contributor

@lipis lipis commented Sep 19, 2013

In the sticky footer example there were some weird stuff..

@mdo
Copy link
Member

mdo commented Sep 20, 2013

I imagine these were there for cross browser compatibility. Have you tried this in all the supported browsers (IE8+, latest Safari, Chrome, and Firefox)?

@lipis
Copy link
Contributor Author

lipis commented Sep 20, 2013

Latest Safari, Chrome, Firefox on OS X works as expected..
IE8 on WinXP the result is the exactly the same.. always showing the scroll bar..
No access to IE9 or IE10... could test later today though and come back with results..

@mdo
Copy link
Member

mdo commented Oct 21, 2013

Since this doesn't change IE8 at all (which is broken as it stands now), nor IE9 and any of the other browsers, we can merge this. Thanks!

mdo added a commit that referenced this pull request Oct 21, 2013
Removed the unnecessary override and the !important from .wrap
@mdo mdo merged commit 976866d into twbs:master Oct 21, 2013
@mdo mdo mentioned this pull request Oct 21, 2013
@lipis lipis deleted the patch-2 branch October 21, 2013 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants