Skip to:
Content

Opened 8 years ago

Closed 8 years ago

Last modified 3 years ago

#3247 closed enhancement (fixed)

Add additional fields to comment_form()

Reported by: hnla Owned by:
Milestone: 1.5 Priority: normal
Severity: Version: 1.5
Component: Templates Keywords:
Cc: hnla

Description

At the moment the post comment_form() is running on defaults and missing 'email will not be made public' if comments are set to allow for non logged in users and there are no 'allowed tags' notes.

Patch corrects these two items for comments.php.

Not sure if these items are wanted for bp-default and approach to adding these although along lines described in WP codex might need scrutiny and adjustment.

Attachments (1)

#3247-add-fields-to-coment-form.patch (1.2 KB) - added by hnla 8 years ago.
Add allowed tags & 'email will not be pub...' fields to comment form

Download all attachments as: .zip

Change History (6)

@hnla
8 years ago

Add allowed tags & 'email will not be pub...' fields to comment form

#1 @DJPaul
8 years ago

  • Keywords comment_form allowed tags comment form fields removed
  • Milestone changed from Awaiting Review to 1.3

#2 @DJPaul
8 years ago

It looks like these were removed intentionally when the comment form was updated. However, taking a look at this, they're probably worth putting in.

#3 @djpaul
8 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4422]) Show standard email note on comment form for unauthenticated users. Fixes #3247, props hnla

#4 @hnla
8 years ago

I'm guessing the idea was to keep things simple while allowing devs to add back in what was required through filters, I can see the reasoning but it does feel a little harder to deal with the comment form now. I think they're best put in as generally you are dealing with users that need those notes on allowed tags and the email notice is important for confidence even though unlikely to often appear with a BP site.

#5 @DJPaul
3 years ago

  • Component changed from Appearance - Template Parts to Templates
Note: See TracTickets for help on using tickets.