Checkbox only shows if logged out
-
The plugin is great, simple and lightweight as advertised, but needs a bit of work.
My main complaint is that the subscribe checkbox only shows if the person commenting is not logged in. This is because it is hooked in using
comment_form_default_fields
which is for the ‘logged out’ form.
It seems like an arbitrary distinction to make, only showing the checkbox to logged out users, especially on blogs where users MUST be logged in to comment in the first place. To show checkbox for logged in users:
add_action( 'comment_form_logged_in_after', 'logged_in_fields' ); function logged_in_fields() { $commenter = wp_get_current_commenter(); echo implode('', cren_comment_fields(array())); }
This puts the checkbox above the comment textarea. To move it below, I used flex-box order css. Maybe there is a better hook to use to move it below?
Next big thing. The email has to have an ‘unsubscribe’ link which stops the email notification. I know it add to the plugin but it is basically a legal requirement and, on really busy comment sections, the user could become very annoyed without the ability to unsubscribe.
The next thing, which perhaps should be a setting, is that the plugin sends an email to the comment parent author even if they are replying to their own comment. Why would people reply to their own comments? Well, in blogs where comment sections are limited in depth (to say depth of 3 replies), the way to ‘reply’ to someone in the bottom depth is to reply to the parent comment which can often be oneself. A simple fix for this to go within “cren_comment_notification”:
//if replying to self dont send email if($email == $comment->comment_author_email){ return false; }
Lastly, but just an idea and probably best as a setting, you might want to check before sending mail:
IF the discussion setting “Email me whenever Anyone posts a comment” is ON
THEN don’t send the reply notification to the post author, because they will be getting an email about a new comment anyway.These things might improve the plugin while still keeping it lightweight. Cheers
- The topic ‘Checkbox only shows if logged out’ is closed to new replies.