Allow passing ID for comment_form container and title

[rmccue]

Right now, there's a div hardcoded with #respond and a h3 hardcoded with reply-title. These make it hard for the comment form to be used on archive pages, as they assume the comment form is only ever output on single pages.

(There are other IDs output in the form, however these are controllable through id_form and id_submit)

[SergeyBiryukov]

Note that #respond is also hardcoded in comments_popup_link()​ and get_cancel_comment_reply_link()​.

get_comment_reply_link()​ and get_post_reply_link()​, however, have a respond_id argument.

[Shelob9]

I am attaching a path that adds an argument to comment_form() for the hardcoded h3 ID. It also introduces a new function, for the hardcoded #respond ID. The new function returns a filter that can be used to change the value of that ID, sanitizes it and ensures it has a hash, or not, depending on context. That way it can be used for targeting the element in JavaScript, and outputted in HTML.

[SergeyBiryukov]

Could you upload a patch without the formatting changes? It would be easier to review. Thanks!

[SergeyBiryukov]

Replying to SergeyBiryukov:

Note that #respond is also hardcoded in comments_popup_link()​ and get_cancel_comment_reply_link()​.

Related: #28314

[rachelbaker]

Patch still needs a refresh.

[rachelbaker]

@Shelob9 Interested in refreshing your patch to remove the extra formatting changes?

[Ninos Ego]

Cleaned patch withouth formating

[Ninos Ego]

I've refreshed his patch and removed the formattings. Hope it's ok now :)

[rachelbaker]

#24799 was marked as a duplicate.