• 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

    • This topic was modified 7 years, 7 months ago by 21stcn.
Viewing 1 replies (of 1 total)
  • Plugin Author guhemama

    (@guhemama)

    Thanks for the detailed feedback!

    I’ve fixed the checkbox issue and I’ve also added the unsubscribe link to the email (I involuntarily delayed this implementation, lol).

    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.

    Indeed, this is a bug. I will release a correction ASAP.

    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.

    This is a good idea. I will analyse if – and how – it could be implemented.

    Regards,
    Gustavo

Viewing 1 replies (of 1 total)
  • The topic ‘Checkbox only shows if logged out’ is closed to new replies.