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

Defer javascript not working in IE9 or IE11 #982

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 6 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Apr 6, 2015

What steps will reproduce the problem?
1. http://buyakilt.com/chmac_ie9.php
2. Load in IE9 http://www.webpagetest.org/result/140909_CN_1KG/
3. Load in Chrome http://www.webpagetest.org/result/140909_VQ_1KR/

What is the expected output? What do you see instead?
In Chrome, or IE10+ the placeholder h1 header is replaced by the actual header 
via jQuery's .load(), but in IE9 this doesn't happen.

What version of the product are you using (please check X-Mod-Pagespeed header)?
1.8.31.4-4056

On what operating system?
Ubuntu 12.04.5 LTS

Which version of Apache?
dpkg -l | grep apache
ii  apache2                          2.2.22-1ubuntu1.7                   Apache 
HTTP Server metapackage

Which MPM?
ii  apache2-mpm-prefork              2.2.22-1ubuntu1.7                   Apache 
HTTP Server - traditional non-threaded model

URL of broken page:
http://buyakilt.com/chmac_ie9.php

It could be something in the javascript that's failing, but if we disable 
defer_javascript the header loads as expected.

* The page is something of a messy example, it's using old HTML and new CSS, so 
it looks a bit wonky.

Original issue reported on code.google.com by callum.m...@gmail.com on 9 Sep 2014 at 1:32

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by jmara...@google.com on 24 Sep 2014 at 9:21

  • Changed state: Accepted
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

I plan to blacklist ie9 from this filter.

Original comment by jmara...@google.com on 25 Sep 2014 at 5:57

  • Changed state: Started
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Summary was: Defer javascript not working in IE9

At the moment we have no evidence it has a problem on IE10 for whatever reason.

Original comment by jmara...@google.com on 26 Sep 2014 at 12:22

  • Changed title: Defer javascript not working in IE9 or IE11
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

IE11 blacklisted from defer_javascript in 
https://code.google.com/p/modpagespeed/source/detail?r=4263

IE9 blacklisted from defer_javascript 
inhttps://code.google.com/p/modpagespeed/source/detail?r=4266

Original comment by jmara...@google.com on 26 Sep 2014 at 12:23

  • Changed state: Fixed
  • Added labels: Milestone-v32, rel
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Related to #987, we'll have to fix the Varnish snippets to reflect this change. 
Created #996.

Is blacklisting for IE 9 and IE 11 the only solution here? Feels like we're 
excluding a lot of users (>16% of all our users). Is there a longer term plan 
to fix the issues with IE9 and IE11, or are they a lost cause?

Original comment by callum.m...@gmail.com on 2 Oct 2014 at 12:19

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Downstream caching: very good point and thanks for entering the bug.

IE 9:  probably we won't address this as the usage count will diminish over time

IE 11: we ought to investigate this further, yeah.  We found one site that 
didn't work with IE11 and did with Chrome, but maybe that's not sufficient 
reason to blacklist, and IE11 users will grow over time.

Original comment by jmara...@google.com on 2 Oct 2014 at 12:41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment