expectancy (@expectancy)

Forum Replies Created

Viewing 1 replies (of 1 total)
  • In reply to: Nested / Threaded Replies not working in Chrome

    expectancy
    Participant

    @expectancy

    7 years, 9 months ago

    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)