http://screencast.com/t/NWFlMWFjZGYt
Must be some kind of margin issue or maybe some inner div is to wide.

Comments

eigentor’s picture

explanation: in the picture above both sidebars should be visible.

jarek foksa’s picture

Status: Active » Fixed

Fixed in RC6.

eigentor’s picture

Status: Fixed » Needs work

Somehow Jeffs Magic appears still to have flaws :)
On the book demo page, /book my sidebar (only one sidebar there) gets pushed to faraway places:
http://screencast.com/t/MTI0ZjE0

Jeff Burnz’s picture

Can you tell me exactly what your browser is (down to version - it looks like IE6 to me) and if you are in min-width or greater, what sidebars you have enabled and blocks etc, just as much as possible, and if you were resizing the viewport when this happened.

The real surprising thing there is its LTR, not RTL! Bit odd but cant be anything major.

BTW - is that a table in the sidebar? Which block is that?

Jeff Burnz’s picture

OK, for now I isolated it to the "Recent Content" block in IE6 - does that concur with your setup?

Jeff Burnz’s picture

Version: 7.x-1.0-rc4 » 7.x-1.x-dev
Status: Needs work » Needs review
StatusFileSize
new843 bytes

Dammit, i see the problem - suggestion, why don't we just push a "width: expression" at IE6 and give the same layout as everything else?

I know this will get people going, but this is what they are there for, only IE6 gets it (and lets be frank, who the hell is using IE6 these days?)

jarek foksa’s picture

We could just set fixed layout for IE6 and focus on more important issues. It's not a requirement to make the theme behave exactly the same on legacy browsers.

Jeff Burnz’s picture

OK, lets do it, better than a nasty expression, only other reliable way would be to use jQuery, but seriously we probably shouldn't bother.

Jeff Burnz’s picture

Status: Needs review » Fixed

Fixed with the suggestion in #7.

eigentor’s picture

Sorry for not replying back ;)
Yes, it was IE6, since the title of this issue was focused on IE6, I did not mention again.
Am fine with your decision: We will get it to work finally, for now it should be good to have it fixed width.

eigentor’s picture

Status: Fixed » Needs work

Er - did you check out the max with trick? Can it make it work in IE? Not giving up so quickly...

.maxwidth-960 {
max-width: 960px;
width: auto !important;
width: 960px;
}

Jeff Burnz’s picture

I dont want to give up either, I have tried that but not so much, lets keep working on it and see if we can get a CSS fix for this.

Jeff Burnz’s picture

I was able to get a min-width CSS only fix working in IE6, however it requires a shed load of extra markup to achieve.

It works by adding a border the same width as the min-width and then using negative margin to pull the content back over it.

E.g on the #page you need to add two new inner divs, i.e.

<div id="page">
  <div id="page-inner">
    <div id="page-inner-2">
       everything else here as per normal...
    </div>
  </div>
</div>

// CSS
#page-inner, 
#page-inner-2 {
  height: 0;
}

#page-inner {
  border-left: 960px solid #FBFCFD;
}

#page-inner-2 {
  margin-left: -960px;
  position: relative;
}

While it works (even in RTL), we'd have to add that extra markup to #navigation, #page and #footer (could use conditional comments to do so), so not so sure about this.

I dont really care about max-width for IE6, to me min-width is the one.

Jeff Burnz’s picture

StatusFileSize
new2.66 KB

Proof of concept patch, this actually and works, and in limited RTL testing working in there also.

After some thought I'm not terribly worried about the extra markup, Drupal7 throws enough extra markup at us already, this is in reality a very small amount of extra markup.

Jeff Burnz’s picture

Status: Needs work » Needs review
Jeff Burnz’s picture

Status: Needs review » Fixed

I committed this for now, if we revert later so be it, there maybe some discussion raised over id naming the the additional markup to support just one browser, at least now we have really good support cross browser rather than providing a somewhat degraded experience. I think fixing the width in IE6 was OK, but not ideal and we can do better, this fix achieves that but at a small cost (extra wrapper divs).

http://drupal.org/cvs?commit=370730

eigentor’s picture

The issue was still there but is fixed now, as mentioned in #802894: Modify layout to support RTL and Fluid width in all browsers (except IE6).
What did I do?
I changed this

.sidebar-first #main {
  margin-right: 26.25%;  /* LTR */
}
.sidebar-second #main {
  margin-right: 26.25%;  /* LTR */
}

They both had 25,25% before, and this

#sidebar-second {
  width: 24.25%;
  margin-left: -25.25%;  /* LTR */
}
.sidebar-first #sidebar-first {
  width: 24.25%;
  margin-left: -25.25%;  /* LTR */
}

margin-left was only 24,25% before on these.

Status: Fixed » Closed (fixed)
Issue tags: -internet-explorer

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