On line 400 in the acquia marina stylesheet you have set a 24px height to .postscript-bottom-wrapper, wouldn't it be reasonable to set this to 240px to extend the background all the way down a block?

CommentFileSizeAuthor
#1 acquia_marina-1454388-missing-ie-selector-1.patch466 bytessheena_d

Comments

sheena_d’s picture

sheena_d commented
Status: Active » Needs review
StatusFileSize
new acquia_marina-1454388-missing-ie-selector-1.patch466 bytes

Hi, Thanks for catching this! It was a typo in the CSS file. That style is meant for IE6 only (since IE6 doesn't understand the min-height rule) and it was missing the .ie6 selector, causing it to override the style for all browsers.

Attached is a patch to correct the issue. If this patch fixes your issue, I will push my changes so they will show up in the next automatic 7.x-2.x-dev release.

dyardley’s picture

dyardley commented
Status: Needs review » Fixed

My bad, I should have checked that a bit further myself. Patch looks clean, style shows as expected so I'll mark as resolved.

sheena_d’s picture

sheena_d commented

We still needed to fix it in the theme, so thanks for reporting! I've pushed that commit, so it should show up in the dev release that's rolled tonight.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.