expectancy (@expectancy)

Forum Replies Created

Viewing 1 replies (of 1 total)
  • @expectancy

    Participant

    For anyone that hasn’t solved this yet, it’s actually a very easy and quick fix!

    In the W3 spec (http://www.w3.org/TR/dom/), the “insertBefore” method requires 2 arguments. The second can be null, but it is still required. It looks like IE doesn’t require the second argument (which is why it works in IE as is) while Chrome, Firefox, and Safari do.

    bbpress/templates/default/js/reply.js – line 18
    Added a null argument as the second argument on “insertBefore”.

    Before Fix: reply.parentNode.insertBefore(respond);
    After Fix: reply.parentNode.insertBefore(respond, null);

    
    reply.parentNode.insertBefore(respond, null);
    if ( post && postId )
    	post.value = postId;
    parent.value = parentId;
    cancel.style.display = '';
    
Viewing 1 replies (of 1 total)