It's not clear if if this is a bug in Marinelli or a bug in Image Picker. See http://drupal.org/node/1085084 for details.

CommentFileSizeAuthor
#3 views.png74.32 KBroy smith
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

axon’s picture

same problem

komal.savla’s picture

Hi,

This is not a bug in the theme. Actually what happen is When the imagepicker module is enabled, inside the image picker fieldset there is an iframe attached, and the iframe contains html and the body tag.

And in the themes css file there is css given for the body as following

body{
	background: #17293d url(../img/backgrounds/body.gif);
}

So the image pickers body tag is picking up this style hence the blue background appears inside it.

We can override it by adding the following style in themes css file to remove the background

body#imagepicker {
  background: none;
}

roy smith’s picture

FileSize
74.32 KB

I just installed Views 7.x-3.0-alpha1. It has the same problem (see attached file views.png). It looks like this is a somewhat generic problem, and special-casing the css for each conflicting module (body#imagepicker, body#views, etc) isn't going to be a viable solution.

(Hmmm, I think this particular one may be a bad interaction with Advanced Help, not with Views, but the basic point still holds; a generic solution is required, not a spot-fix for each affected module).

Lioz’s picture

Status: Active » Needs review

Hi, maybe we can load the background using a dedicated body class

for example in html.tpl.php

	<?php /* we use this to avoid conditional css loading */ ?>
	<!--[if lt IE 7 ]> <body class="marinelli ie6 <?php print $classes; ?>"> <![endif]-->
	<!--[if IE 7 ]>    <body class="marinelli ie7 <?php print $classes; ?>"> <![endif]-->
	<!--[if IE 8 ]>    <body class="marinelli ie8 <?php print $classes; ?>"> <![endif]-->
	<!--[if IE 9 ]>    <body class="marinelli ie9 <?php print $classes; ?>"> <![endif]-->
	<!--[if gt IE 9]>  <body class="marinelli <?php print $classes; ?>"> <![endif]-->
	<!--[if !IE]><!--> <body class="marinelli <?php print $classes; ?>"> <!--<![endif]-->

and then in layout.css

/*	Look & Feel
-------------------------------------------------------------- */
body.marinelli{
	background: #17293d url(../img/backgrounds/body.gif);
}

let me know if this solves the problem.
this solution should also work with ck editor

roy smith’s picture

Yes, that fixed the problem with both ImagePicker and Views. Thanks!

Lioz’s picture

Assigned: Unassigned » Lioz
Status: Needs review » Fixed

great!
pushed in the next release!

Status: Fixed » Closed (fixed)

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