- This topic is empty.
-
AuthorPosts
-
December 26, 2012 at 10:57 am #41603siouxfan45Participant
My #container div sets a 15px margin on the top and bottom. Unfortunately, the bottom margin does not seem to be working unless overflow:hidden is set. [Check the sidebar on the homepage for an example](http://themeforward.com/demo2/ “”).
I can’t set overflow:hidden on #container because the sub-menu will be hidden under the footer on shorter pages. For example at [http://themeforward.com/demo2/?page_id=1846](http://themeforward.com/demo2/?page_id=1846 “”) when I set overflow:hidden on container the sub-menu under ‘Categories’ becomes hidden.
My current CSS
#container {
max-width: 960px;
margin: 15px auto
}December 26, 2012 at 11:07 am #119026David_LeitchParticipantBecause you’ve floated your sidebar, the container is only growing to the height of main content. You just need to add a [clearfix](https://css-tricks.com/snippets/css/clear-fix/ “https://css-tricks.com/snippets/css/clear-fix/”) to the container element and the container will grow to the height of the sidebar and the margin-bottom will be applied.
December 26, 2012 at 11:08 am #119027SenffParticipantThe sidebar is breaking out of the container because it’s not cleared. You’ll need to use a clearfix on the container — setting “overflow:hidden;” on it would be one way but since you can’t use that, you’ll need another way (having said that though, I didn’t see any problems with the submenu even when the container DOES have
overflow:hidden;
).Try adding the following code in your CSS and then add a class “clearfix” to your container.
.clearfix:before, .clearfix:after {
content: ” “;
display:table;
}.clearfix:after {
clear:both;
}December 26, 2012 at 11:41 am #119030SenffParticipantLooks like you (still) haven’t applied the class “clearfix” to #container, so it’s hard for us to check why it’s not working then.
December 26, 2012 at 12:18 pm #119035SenffParticipantNo clearfix class anywhere in that one either.
December 26, 2012 at 1:08 pm #119042SenffParticipantYou don’t need to add another DIV at all, you really just need to add the class “clearfix” to the existing container div and make sure the code that I posted above is somewhere in your CSS file.
December 26, 2012 at 1:35 pm #119051SenffParticipantNope, don’t think it should.
-
AuthorPosts
- The forum ‘CSS’ is closed to new topics and replies.