-
Notifications
You must be signed in to change notification settings - Fork 0
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
Internet Explorer support #1
Comments
Thanks @Kristinita I just tested the polyfill out on IE 11.0.9600.16663 and it seems to be working fine. The demo page however is broken, and should be fixed, but it is just failing to load the polyfill. Once, loaded, everything works as expected. Are you seeing issues when using the polyfill, or viewing the demo page? |
@uniqname, I use the Details-Polyfill to my articles, example: http://kristinita.ru/Gingerinas/Korvers. Minimal example:
Thanks. |
Thanks for the reduced test case! You're test case is having the same problem as my demo page. The error is not in the polyfill, but the test for whether the polyfill should be loaded. See this pen for an example on how to test and conditionally load the polyfill. I will get the demo page updated to reflect this same approach. |
Thanks, great work! One more request. As disclosed spoilers appear in the Chrome and Opera, browsers that support normal tag
Result in Firefox and Internet Explorer, don't support this tags:
Could you make result in browsers that supports the tags Thanks. |
@uniqname, actually. For example, how users see detailed spoilers in your page https://codepen.io/uniqname/pen/akZbjX in Chrome: And in Internet Explorer, when using Details-Polyfill. The results slightly different, but it would be nice if they were identical. Thanks. |
Yeah, I debated whether or not to match current implementation or spec. I settled on spec which does does not indicate anything about presentation. I also chose that route to make styling them easier since Chrome's default styling is quite painful to override. It's not too difficult to emulate Chrome's implementation if you need it.
|
@uniqname , it's not worked for me. My code: https://gist.github.com/Kristinita/63564d7e1a69a11fdfccac618a7b94dd How I see result in Internet Explorer: How I see result in Chrome: In browsers which support the Thanks. |
@uniqname , actually. Thanks. |
Of the browsers that do not support tags, your script good supports Firefox, Edge and Opera Mini, but Internet Explorer (newest version IE 11.0.10240.17071 for me) is not supported. Could you introduce the tag support for IE?
Thank you for a great script!
The text was updated successfully, but these errors were encountered: