only in IE8 (Flash 10.1.102.64, SVG Web r1309) the following doesn't work

rightRect.style.display='block';

see reduced testcase here

http://vis.uell.net/gsvg/displayBlock_IE8.html

Compatibility View, IE7 and native renderers all work fine.

Fixed in r1310. Tests in r1311.

This was an unreleased regression from Issue 573/r1273 and therefore will not be included in the release notes.

This is not yet working for me. I've updated the above testcase to r1317 but the problem persists in IE8 (and IE8 only)

http://vis.uell.net/gsvg/displayBlock_IE8.html

Flash 10.1.102.64 IE 8.0.6001.18702 (WinXP) IE 8.0.7600.16385 (Win7)

Fixed in r1318.

Issue 598; Reduce compression of svg.js because this is causing a failure of the test file in Issue 598. Even --compiliation_level SIMPLE_OPTIMIZATIONS is failing. This increases compressed size by over 30K. Full compression will be restored if the problem with the Google closure compiler can be determined and resolved.

I am going to have to debug this failure in compressed mode and hopefully figure out how the compression process caused this new(?) failure. :(

Compression restored and compression friendly workaround implemented in r1319.

Updated compiler and fixes for warnings in r1320.

Ups, I was of the opinion that the Google closure compiler was non-destructive. Sorry that I always tested with the closure compiled version.

Does comment #5 mean that the Google closure compiler at http://closure-compiler.appspot.com should not be used but one should use the Java version locally that you have improved?

The update to the compiler was just one I obtained by taking a fresh download and I think it is the same one you are pointing to. I decided to check it in because it seemed to work to I figured it was a good idea to use the latest version. It did not fix any specific problem that I know of.

Bug doesn't seems to be fixed IMO. Just by checking the given url with IE8 (IETester in IE8 mode in fact), bugs.

I found the issue! It seems that in compatibility view, event if the page said that it IS compatible with IE8 (by adding a meta tag), the UA string is always MSIE 7.0 but documentMode is 8. If you change the only test of isIE >= 8 to isIE >= 7, it works, so I guess that when isIE and documentMode == 8, isIE should be set to 8.