Skip to:
Content

Opened 3 days ago

Last modified 3 days ago

#2971 new defect

Threaded reply form JS does not work

Reported by: r-a-y Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version: trunk
Component: Appearance - Theme Compatibility Keywords: has-patch
Cc:

Description

Turn on threaded replies.

Next, visit a forum thread with at least two posts.

When you click on the "Reply" link for the second post, the following JS error occurs:

Not enough arguments to Node.insertBefore.

Attached patch fixes this by switching reply.parentNode.insertBefore(respond); to reply.parentNode.appendChild(respond);

Now the threaded reply form correctly shows up just beneath the forum post.

Tested on 2.5.10 and 2.6-trunk.

Attachments (1)

2971.01.patch (403 bytes) - added by r-a-y 3 days ago.

Download all attachments as: .zip

Change History (2)

@r-a-y
3 days ago

#1 @r-a-y
3 days ago

If the reply form should show up before the forum post, let me know and I'll repatch.

Note: See TracTickets for help on using tickets.