search

registerguard / bulldog

The official 2013 registerguard.com theme.
http://registerguard.com
1 stars 1 forks source link

Ad margin #57

Closed mhulse closed 11 years ago

mhulse commented 11 years ago

Remove top/bottom margin from ad_manager.css:

.advert {
    text-align: center; /* Oldschool centering technique. */
    position: relative; /* Just in case ad decides to `position: absolute;` without a `position: relative;` parent. */
    margin: .75em auto; /* Putting `auto` here to assist `.advert`s that have a width. */
}

Reason why I'm putting in Bulldog's issue's: The spacing is needed if ad is at top of a column; I need to build a more official way of adding margin/padding to columns (perhaps as a Wiffle addon?)

mhulse commented 11 years ago

Removed top margin and kept bottom, but changed it to 10px. Might change this later. Need to complete #58. Closing for now.