New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
trunk JS error in IE8 (maybe other IE as well) #2295
Comments
(In [4401]) refs #2295 IE <9 can't handle new Date('yy-mm-dd') use datepickers |
Thanks Steve good stuff! It is working in IE8 now as well, why do you say it doesn't work in IE<9 ? |
Couldn't test it in IE 8 right now. In IE 7 the layout is broken |
Attachment: |
selecting date ranges work in current month, but if I change months I get:
|
That sounds like a problem with the pie. Will take a look at this later. |
Also on the Widgets page I see a "Expected ; at the end of" with a garbage line (WHY does this debugger does NOT tell the right line, even with developer tools grrrr) |
After updating to trunk and the new version of CSS PIE, the problem is still there so it is a different one.. The JS bug appears:
The debugger won't give the backtrace or give the proper line number, I'm not sure how to go to debug this? |
Note that now the date range works in IE8, but there is this JS error (not sure if it breaks anything?) |
(In [4412]) refs #2295 fixes layout problem in IE7 |
(In [4414]) refs #2295 fixes manual input of a date in ie <9 |
matt, I can't reproduce your error in IE8 or IE7. Did that error occur with merged assets enabled or disabled? |
I get the error on IE 8.0.6001 (I think latest version?) with both merged assets enabled or disabled, on the Dashboard page and on the Widgets page. I dont get it on API/All Websites/Email reports pages so probably a problem in the menu or datatable code. |
(In [4422]) refs #2295 no need to load ieonly.css in ie9, cause pie not needed here |
Well, tried to reproduce it on another system now, but its working here aswell. No JS-Errors ouccred while switching dates. While testing I sometimes had the issue that the pie seemed to break the complete layout in IE8. Do you have that problem, too? I think we should consider removing the pie again in the future, as now IE9 supports rounded corners. Btw is the latest tag already available on the demo? Maybe I could reproduce the error there. |
Thanks Steve I pushed 1.3rc1 to the demo and it is working fine, excellent! |
as much as I hate IE we must fix this!
Calendar doesn't show the days, simply the first row of the calendar... if anyone is keen to debug this please do :)
The text was updated successfully, but these errors were encountered: