I haven't mange to find the file causing it, but with firebug I've found that this line of code:
<div id="modalBackdrop" style="z-index: 1000; top: 0pt; height: 568px; width: 1663px;"/>
may break some themes (it breaks mine). Easiest solution would be to remove the height and width, which (tested) wouldn't effect the module...

CommentFileSizeAuthor
#2 scrn.jpg107.13 KBNeoID
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

Status: Needs review » Active

That height and width are set by javascript which is setting the modal to 80% of the viewable area. How does this break themes?

(Also, 'needs' review means there is a patch attached).

NeoID’s picture

FileSize
107.13 KB

Sorry for setting the wrong status...
On my site the page becomes very long and my footer ends before the page (note the space after the blue footer). It's much longer then on the screenshot and if I remove the height using firebug the theme looks good again... Is setting the height really important or can it (or can I) remove it?

merlinofchaos’s picture

Wow, that's really strange to get that result. That totally doesn't match the height and width that are being set, either. I don't know why this would be.

You can't remove the height setting, like I said, it sets it to 80% of the current viewport width. It shouldn't change the rest of the...hm. You know, I think if you're using alpha4 you're probably not even getting the dynamicly sized modal. I only JUST added that. That's even weirder.

merlinofchaos’s picture

Ahh, the modal backdrop is what darkens everything that isn't the modal. It normally doesn't have any interaction problems with the rest of a theme, so this might indicate there's something wrong with your HTML or CSS that this is hilighting.

merlinofchaos’s picture

Status: Active » Fixed

Somewhere along the way certain css properties for the modal backdrop had gotten lost and I never noticed. So it was putting it in totally the wrong place. Today's checkin should fix this. By placing it position: absolute it should not do that to your theme anymore.

Status: Fixed » Closed (fixed)

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