IE8.js and IE7.js might break certain @media types for stylesheets

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Check the attached file - it shows the model to reproduce the problem.

What is the expected output? What do you see instead?

When that page is rendered in Firefox or Safari I get 3 columns as it is 
supposed to be rendered for 'screen' media. 
If I open the same page in IE6 or IE7 I see empty red background as the styles 
for media="handheld, screen and (max-
device-width: 480px)" declare. This type of media definition is absolutely 
valid from both HTML and CSS points of view 
and should be fired up for 'screen' only in case the device width is 480px 
maximum.

What version of the product are you using? On what operating system?

2.0(beta3). I am using 2 separate Windows XP virtual machines in Parallels. So 
both IE6 and IE7 are full (not standalone) 
browsers.

Please provide any additional information below.

 - If I remove styles with media="handheld, screen and (max-device-width: 480px)" IE6 and IE7 both render proper 
columns. 
 - If I leave the styles with that media, but remove conditional comment with IE8.js instead, IE6 and IE7 both render page 
properly with columns. So, unless we don't enable IE8.js, both IE6 and IE7 
accept that media type and treat it as it should 
be treated.

So, I suspect that something in javascript (IE8.js) triggers or somehow 
influences @media for styles that makes IE think 
that those styles are for normal screen.

Original issue reported on code.google.com by denys.mi...@gmail.com on 24 Sep 2009 at 7:51

Attachments:

• index.html

[GoogleCodeExporter]

I got your email. I'll look into this. It's looks like the max-device-width is 
being 
ignored.

Original comment by dean.edw...@gmail.com on 26 Sep 2009 at 8:49

• Changed state: Accepted

[GoogleCodeExporter]

Thanks a lot for accepting this Dean. I appreciate your help

Original comment by denys.mi...@gmail.com on 26 Sep 2009 at 6:13

[GoogleCodeExporter]

this might be related to the issue that i'm having, in which @media 
declarations, and
media=".." properties are being ignored. Take a gander at error 205, cheers :)

Original comment by xenonthe...@gmail.com on 17 Oct 2009 at 11:42

[GoogleCodeExporter]

Yeah, would be great to see some progress on this, as we're determining whether 
to
ship the next version of Plone with ie8.js — but can't really do it if it'll 
break
our mobile phone support.

I would help if I could — unfortunately JS is not my strong suit. :|

Original comment by l...@gmail.com on 17 Oct 2009 at 8:52

[GoogleCodeExporter]

Any updates on this? 

We're getting close to shipping the next major release of the Plone CMS 
(currently in
alpha), and would really like to include ie8.js as our solution to support older
browsers.

Thanks!

(sorry for the poking, we appreciate all the hard work that has already gone 
into
this, and it's only this one bug that is blocking us at the moment)

Original comment by l...@gmail.com on 6 Nov 2009 at 7:25

[GoogleCodeExporter]

Any progress here? :)

Original comment by l...@gmail.com on 17 Dec 2009 at 7:59

[GoogleCodeExporter]

This is fixed in version 2.1 (forthcoming).

Original comment by dean.edw...@gmail.com on 11 Feb 2010 at 3:06

• Changed state: Fixed

[GoogleCodeExporter]

Thanks a lot for fixing this Dean! Looking forward to see version 2.1 out. Then 
we could include it into Plone. 
How long you think it will take for 2.1 to go public?

Original comment by denys.mi...@gmail.com on 12 Feb 2010 at 2:54

[GoogleCodeExporter]

I will try to fix as many bugs as I can over the weekend. But the plan is to 
release 
whatever I have early next week. I will stick a beta flag on it and then fix 
any 
regressions as they come up. I'm not adding any new functionality, just 
bug-fixing.

Original comment by dean.edw...@gmail.com on 12 Feb 2010 at 3:58