Sidebar pushed under content in node view (IE6)

Hi, Michael. Thanks so much for testing D7 in IE! We certainly need that help. :)

Is it possible for you to post screenshots of what you're seeing? That might make it easier for someone to debug and fix.

Looking into it.

Now we have a bartik project, moving there.

Or we could do http://ie6update.com/ in core? ;-)

I cannot replicate this in IE6. Is anyone else able to?

Confirmed. This is only happening on nodes with the 640px wide images in node content. Between the image width, column widths, margins and padding, there appears to be 5px too much for IE6 in the main column, which is causing the sidebar to drop. I would probably recommend floating the sidebar right instead of left, removing the margin-right on this column, and reducing the padding-right on #content from 20px to 15px, but I'm not sure if that is in line with other width/spacing considerations that have been already made in this theme.

Related: #721630: content widths do not add up

I'm assigning this back to unassigned. Anyone feel free to take a look.

My first inclination is to declare

div {
overflow:hidden;
}

in layout.css.

This would solve this issue and any others. The bigger problem here is when someone uploads an image that is greater than 640 ... it seems that letting the theme bork itself over such an innocent mistake would be counter-intuitive. The blame would be upon the end user "oh, look at that, i made my image too big" rather than "OH MY GOD I BROKE THE SITE" <-- not realizing that, well, really your image is just too big.

This happens every time i ingest a tumbler feed into a panels pane. A solution there is to resize the image to the block width ... but that remains in appropriate here.

Thoughts?

Not sure about something so pervasive as div {overflow: hidden;} - thinking of modules like Nice Menus, Superfish etc - anything that does dynamic menus could be borked by this (I know the Superfish menu module will be).

At a pinch we could use :

#content {
  overflow: hidden;
}

Put this in ie.css, however it will apply to IE7 which does not need it, so I suggest if we're going to use this we need an IE6 specific stylesheet.

AFAIK there is no other workaround for the "expanding box bug" in IE6.

OK, so rolled a patch following my own advice.

This patch adds an ie6.css file with one lousy declaration in it (see #9).

Frankly, if it were up to me I would probably use the underscore hack and jam it in ie.css...

we need to fix this before officially submitting Bartik for consideration

bump for testers :)

PS - Un-assigning johnvsc because anyone with IE can do this.

I will be be the nice guy ;) ==> xp mode ==> testing

can't reproduce this :s. Don't know if thats a good thing :D

Screenshot!

Discussed with jacine, retesting this...
Maybe need a background picture

Ok after a second test round I could reproduce error.
I applied the patch and the original issue is gone.
But the image I added got rescaled in a strange way.

(See screenshot)

It didn't get re-scaled, it go cut off - overflow: hidden;

For such an ancient browser and a very difficult bug to deal with this has to be acceptable, there's only so much hand holding a theme can do for end users, if they are silly enough to put in content that is too big and want to support IE6.

I agree, stupid IE6

Dear Microsoft,

IE6 needs to die.

Love Bleen

P.S. RTBC++ (I saw the same behavior as aspilicious)

Committed! thanks ;)