The latest CVS version has a bug when used with IE7. All blocks that are configured for the right side are shown on the left, on top of the left side blocks.

..- Wonderland

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Frando’s picture

Priority: Critical » Normal

Puh. Had no chance yet to test with IE7. Don't know if I will have soon ...

If anyone could nail it down I'd be thankful.

I'd been playing with IE6 hacks and IE5.5 hacks through the whole developement process of bluebreeze. And now the same again for IE7.. Why, MS, WHY do you do this? ;-)

wonderland’s picture

It looked fine before the clean-up. As IE7 is delivered via auto-update, a lot of people will start using it. I will try to find the bug, but given my lack of CSS knowledge, it might not be too successful.

..- Wonderland

AmirTheSeventh’s picture

FileSize
294.16 KB

Screenshot attached!

AmirTheSeventh’s picture

FileSize
121.78 KB

BlueBreeze Fix Screenshot attached

AmirTheSeventh’s picture

The theme version I installed, doesn't have this issue on IE6SP2 and IE7. I looked at theme's files to find out date of release, but couldn't find it. file date stamps are 21/11/2006 probebly the day i installed it. the only problem my version has is here http://drupal.org/node/100449 my corrent version is just perfect with IE6 and IE7.

wonderland’s picture

Yes, it only happens with the latest CVS version (November 26th). Unfortunately the update is quite big so it's not too easy to find the reason.

..- Wonderland

angurio’s picture

i had this problem on a live site, that im using with bluebreeze (fixed version) - so had to fix urgently. (site is itsourmovie.com)

im not a css guru, so you may regard this as a hack, but it works:

im my page.tpl.php i add this conditional comment to force any browser greater than ie6 to see my special css file:


  <?php print $styles ?>
  <!--[if gt IE 6]>
<link rel="stylesheet" type="text/css" href="../iom_files/ie7andlater.css"/>
<![endif]-->
  <?php print $scripts ?>

in this css file i put this:


/* CSS Document */

#sidebar-right	{
/*  float: right; */
  margin-left: -22%; 
  width: 154px;
  padding: 5px 8px 20px 8px;
 /* background-color:#00FF66; */
/* border: dotted;
 border-color:#00FF00; */
}

basically took out the float right, reduced the margin left, and if i remember tweaked the box width to make sure the bar had enough space. hope this helps!

Frando’s picture

I'll try to find a computer running IE7 and then I'll try to fix this.
@angurio: So is the fix working good? You have a percentage left-margin, doesn't the layout change/break when resizing?

Grinin’s picture

I'm having a similar issue but its not specific to browser?!

I upgraded my Drupal site from 4.7 to 5.0 rc1, thus i downloaded the 5.x theme fore Bluebreeze (which I love by the way).... I uploaded everything except the logo (I modified mine).

When I was in the administrative area of drupal, everything was in order... everything looked fine, etc. When I clicked to go back to the main page... all hell broke loose. The leftside... was missing all blocks. and the blocks on the right started after all of the text/content loaded.... meaning... the right side blocks start where the page should have ended.... it was really weird...

so I went back to 4.7 while i tried to figure this out.... problem is, I don't know where to start.

4.7 works fine, except it blows the width of the site a little off and thus creates a horizontal scroll bar which is completely unnecessary. Check out http://ChrisLLorca.com to see what I'm talking about.

Any help would be great.

-Chris

jungle_jim01’s picture

I'm having the same problem (right sidebar displaying on top of left) with bluebreeze fixed version in IE7. I applied the hack shown above and it successfully moved the rightsidebar to the right. However, there is still a gap on the right hand side (approx 50px) which forces the sidebar to overlap some of the content area.

The website is: http://www.guernseykayaking.com

Any help would be really appreciated.

Cheers. Happy New Year!

AmirTheSeventh’s picture

Status: Active » Needs review

Guys! I found the bug:


* html #sidebar-right	{
  margin-left: -150px;

around line 50 in style.css must be


html #sidebar-right	{
  margin-left: -150px;

seems the star creating the problem!

jungle_jim01’s picture

awesome! that did the job...thanks for your help

wonderland’s picture

That does not work for the regular (non-fixed) version of the theme as the style.css does not contain these lines. Adding

html #sidebar-right	{
  margin-left: -150px;
}

to the style.css does help with the problem. The right sidebar is shown on the right but at the same time is moved down quite a bit (in all browsers).

..- Wonderland

thecarlhall’s picture

for the non-fixed version, look in layout.css rather than style.css.

This code:

* html #sidebar-right	{
  margin-left: -190px;
}

Should become:

html #sidebar-right	{
  margin-left: -190px;
}

This seems to work great for IE7 without pushing the right side column down the page.

wonderland’s picture

Status: Needs review » Reviewed & tested by the community

Yep, that does the trick :-) Thanks everybody for the help. I checked and this patches in #11 and #14 work fine with IE7, Firefox and Safari (i.e. it fixes IE7 without breaking other browers). Ready to be commited I would say :-)

..- Wonderland

Frando’s picture

Status: Reviewed & tested by the community » Fixed

Commited. Thanks everyone!

budda’s picture

Great fix. Any chance this can be committed to the 4.7 branch of the theme too?

stardal’s picture

Wow, thanks. I had decided on this one for one of my sites and was quite disappointed when I saw the error in IE7. It's funny how the smallest things can cause the biggest issues.

birdmanx35’s picture

Version: master » 4.7.x-1.x-dev
Priority: Normal » Critical
Status: Fixed » Active

These small fixes did not work in either the layout.css for the fluid, or the style.css for the fixed for 4.7.5. A new version for the 4.7.x branch should be released which has this minor change made. This could at the most take you ten minutes.

wonderland’s picture

Priority: Critical » Minor

If you read this issue, you'll find that several people reported that the fix in #11 and #14 works fine (yes, that includes 4.7.x). And it should "at the most take you ten minutes" to make that little change to your files. This issue is fixed. It might not yet be commited for 4.7, but re-opening and setting the priority to critical is just ridiculous. An issue is not critical because YOU PERSONALLY want a fix right here and right now. And it is definitely not critical if an easy fix exist that can be applied without any programming knowledge.

Just my 0.2 € of course...

..- Wonderland

birdmanx35’s picture

Right, apologies are due.
However, it is very true that upon carefully following the instructions, the problem has not been fixed. However, let's just leave the blame to MS, shall we? :)

Again, sorry, thanks.

wonderland’s picture

Ok, let's try to find out why it is not working for you :-)

- Did you use the latest version of the theme?
- Is there an online version of your installation that we can check?
- Could you publish your changed files somewhere, so somebody can double-check?

I don't have a 4.7.5 running at the moment as I switched to 5.0 so I can't test directly.

..- Wonderland

birdmanx35’s picture

FileSize
1.74 KB

I am using the newest version, yes.

No, I can't show you because it's on the localhost :(

Uploading the layout.css for the fluid version.

birdmanx35’s picture

Wow, you'd think after a good ten years of using a computer you'd remember to save the file, wouldn't you! Well, my bad folks, didn't save the file. It works like a charm now.

That said, we should still get a fixed 4.7.5 theme up :)

Thanks for all your help, it's a great theme!

birdmanx35’s picture

FileSize
124.31 KB

Well, in order to help the cause, I figured I'd make the new 4.7 version. I edited the /bluebreeze/layout.css, and the /bluebreeze/bluebreeze-fixed/style.css to take out the IE7 incompatible error. I've uploaded it with this.

birdmanx35’s picture

Status: Active » Closed (fixed)

This is closed, sorry it was opened in the first place. For those of you who are having trouble, either download the fixed version of the theme above, or follow the instructions above.

AmirTheSeventh’s picture

Status: Closed (fixed) » Needs review

Another way of fixing the issue by "diovi", seems this one don't break IE5:

this is my css code (im using bluebreeze-fixed):

#sidebar-left, #sidebar-right {
  width: 138px;
  padding: 0 6px 0 6px;
  overflow: hidden;
}

#sidebar-right {
  .margin-left: -150px; /* IE7 Fix */
}

* html #sidebar-right {
  margin-left: -150px;  /* IE6 Fix*/
}

**note the period before "margin-left" in the IE7 fix, don't forget it**

look here:
http://drupal.org/node/112401

Frando’s picture

Status: Needs review » Closed (duplicate)

Commited http://drupal.org/node/112401.
Marking this as duplicate.