Closed MichaelMartin closed 12 years ago
MichaelMartin
commented
12 years ago Hi Antonio, Thank you for your hint. We know that, but one of our project partners have exactly that requirement. Its not such a big deal to get it running on IE7 (at least on the functional site)..... hopefully ;-)
All the Best Michael
demarant
commented
12 years ago Hi, I understand...i was surprised when i saw it. Amazing requirement ...wow... are they living in the past? ;)
Bye Antonio
Michael Martin notifications@github.com wrote:
Hi Antonio, Thank you for your hint. We know that, but one of our project partners have exactly that requirement. Its not such a big deal to get it running on IE7 (at least on the functional site)..... hopefully ;-)
All the Best Michael
— Reply to this email directly or view it on GitHubhttps://github.com/AKSW/cubeviz.ontowiki/issues/56#issuecomment-9811470.
k00ni
commented
12 years ago I can't reproduce this error! Please look at my configuration for IE8 in IE7 mode here.
k00ni
commented
12 years ago Ok, Internet Explorer Compatibility View is crap. After tested this in a Virtual Box with real IE7, i could reproduce this issue.
k00ni
commented
12 years ago This issue is because of false coma's in ChartConfig.js! @MichaelMartin please be more careful during change ChartConfig, because modern browsers are able to handle dirty javascript such as false coma's, Internet Explorer fucks up and nothing works. And its a pain in the ass to debug with IE7!
Commit 3a6155657088e474067169caa1abf6fbe2baabc2 fixes this issue
MichaelMartin
commented
12 years ago Ok, right now it works!
Hi,
Are you sure you need to support IE7? Is not worth it I think. That browser version is almost dead...very little browser market share. Below 1%...declining fast...you can check statistics on several sites.
Just a tip :)
Antonio
Michael Martin notifications@github.com wrote:
— Reply to this email directly or view it on GitHubhttps://github.com/AKSW/cubeviz.ontowiki/issues/56.