[GitHub] wicket pull request #253: WICKET-6498 deferred javascript

[klopfdreh]

Github user klopfdreh commented on a diff in the pull request:

    https://github.com/apache/wicket/pull/253#discussion_r159287015
  
    --- Diff: 
wicket-core/src/main/java/org/apache/wicket/markup/head/OnLoadHeaderItem.java 
---
    @@ -86,8 +86,8 @@ public void render(Response response)
                CharSequence js = getJavaScript();
                if (Strings.isEmpty(js) == false)
                {
    -                   JavaScriptUtils.writeJavaScript(response, 
"Wicket.Event.add(window, \"load\", " +
    -                           "function(event) { " + js + ";});");
    +                   JavaScriptUtils.writeJavaScript(response,
    +                           "var f = function() {" + js + ";};\nif 
('loaded' === document.readyState) f(); else window.addEventListener('load', 
f);");
    --- End diff --
    
    Here I suggest to change it to „complete“


---