Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Default Comment Reply Notification Template #77

Closed
raamdev opened this issue Mar 25, 2015 · 11 comments
Closed

Improve Default Comment Reply Notification Template #77

raamdev opened this issue Mar 25, 2015 · 11 comments
Labels
Milestone

Comments

@raamdev
Copy link
Contributor

@raamdev raamdev commented Mar 25, 2015

The current default Comment Mail "New comment" notification email feels way too busy and there are several things I feel could greatly improve it.


by @jaswsinc See Next Actions / Project Outline: https://quire.io/w/kx2VYsoJkIs4L6P3pfFuVDVg/Understanding_Commen


Here's an example of the current default new comment reply notification email:

As seen from GMail

2015-03-24_21-43-32

As seen from an iPhone 5

img_4958

  • A) The small font size in the author quote is too small, especially on mobile devices. It also doesn't need to be part of a list item. It's unnecessary to mention the author name, since the recipient of the email is the author--it's redundant. Showing and linking the comment ID is also unnecessary (it means nothing to the person receiving the email and there's already a link to their comment in the "New Reply to 'your Comment' on..." section right above).

    The clip here is also weird: it shows the beginning and the end of the comment, which makes you quickly feel lost reading your own comment. It would be better to just give a clip of the beginning of the comment, which would be enough to remind the author of the comment they left. I would also increase the default length of the clip here to 300 characters. Anywhere there is a clip, it should be [...] instead of just ..., as the latter can easily be mistaken for an ellipsis added by the commenter.

  • B) The main comment reply text doesn't need to be a list item. The default clip here of 200 characters is way too short. The default should really be 1000+ characters, if not something even higher. The whole point of this notification email is to see this content (the reply comment), so this is the most important content in the entire email. I would be comfortable setting this to 5000 characters by default.

  • C) I would get rid of the "Create New Subscription" section in the comment reply notification email. It doesn't seem like the right context here... "create a new subscription to what?" The only place people will be thinking about creating a comment subscription is when they're on the site viewing comments. I would also move the "Our Mailing Address" to the bottom of "Contact Info" and get rid of the tables that currently make the structure of this section. On mobile devices, tables totally fail (see mobile screenshot above).

Revised comment reply notification email mockup (GMail)

2015-03-24_22-15-13

@raamdev raamdev added the enhancement label Mar 25, 2015
@raamdev raamdev added this to the Initial Public Release milestone Mar 25, 2015
@jaswrks
Copy link

@jaswrks jaswrks commented May 29, 2015

Thank you. I will work on this during the soft launch and try to improve before the official launch.

@kristineds
Copy link
Contributor

@kristineds kristineds commented Jun 15, 2015

@jaswsinc Do you need help with this? :)

@jaswrks
Copy link

@jaswrks jaswrks commented Jun 16, 2015

Do you need help with this? :)

Yes, and I will do my best to outline a game plan for this soon. For the moment though, I don't have enough information together in order to expose what needs to happen here. Will work on it! :-)

@kristineds
Copy link
Contributor

@kristineds kristineds commented Jun 16, 2015

Will work on it! :-)

Cool. Looking forward to it. :)

@jaswrks
Copy link

@jaswrks jaswrks commented Jun 17, 2015

@kristineds @raamdev I posted an outline of how to complete the work on (A). Please see: https://quire.io/w/kx2VYsoJkIs4L6P3pfFuVDVg/Understanding_Commen... if you'd like to learn more about how you can help on this issue.

@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Jun 17, 2015

@kristineds @jaswsinc I'm going to assign this issue to myself and work on completing this.

@raamdev raamdev self-assigned this Jun 17, 2015
@raamdev raamdev modified the milestones: Initial Public Release, Next Release Jul 15, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 10, 2015
- Also moves the <hr /> tag into the foreach loop to use as a separator for multiple comments.

See wpsharks/comment-mail#77
raamdev added a commit that referenced this issue Dec 10, 2015
- Also moves the <hr /> tag into the foreach loop to use as a separator for multiple comments.

See #77
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 10, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
- Remove Serif font; mixed font styles increase strain on reader
- Remove max-width attributes; unnecessary and prohibits responsiveness
- Remove margin-bottom:0 from paragraph tags; not visually appealing
- Remove margin-left:1em from paragraph tags; not visually appealing
- Remove references to Comment ID#; confusing and irrelevant for users
- Replace <br /> tags with <p> tags; improves visual appearance
- Add bold font to all "In response to..." messages; improves consistency
- Improve message wording in various places for clarity and consistency

See wpsharks/comment-mail#77
raamdev added a commit that referenced this issue Dec 11, 2015
- Remove Serif font; mixed font styles increase strain on reader
- Remove max-width attributes; unnecessary and prohibits responsiveness
- Remove margin-bottom:0 from paragraph tags; not visually appealing
- Remove margin-left:1em from paragraph tags; not visually appealing
- Remove references to Comment ID#; confusing and irrelevant for users
- Replace <br /> tags with <p> tags; improves visual appearance
- Add bold font to all "In response to..." messages; improves consistency
- Improve message wording in various places for clarity and consistency

See #77
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
- If the content length is shorter than the clip length, it makes no sense to display an ellipsis.

See wpsharks/comment-mail#77
raamdev added a commit that referenced this issue Dec 11, 2015
- If the content length is shorter than the clip length, it makes no sense to display an ellipsis.

See #77
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 11, 2015
If we want the header to be mobile-ready across all of the major email
  clients, we cannot use tables or floats, so the logo just doesn't fit.

See wpsharks/comment-mail#77
raamdev added a commit that referenced this issue Dec 11, 2015
If we want the header to be mobile-ready across all of the major email
  clients, we cannot use tables or floats, so the logo just doesn't fit.

See #77
raamdev added a commit to wpsharks/comment-mail-pro that referenced this issue Dec 12, 2015
 When the notification is a reply to "Your Comment", the text that follows
 the title is "In response to your comment:", with 'your comment' linked
 to the comment. There is no need to include a second link in the title.

See wpsharks/comment-mail#77
raamdev added a commit that referenced this issue Dec 12, 2015
 When the notification is a reply to "Your Comment", the text that follows
 the title is "In response to your comment:", with 'your comment' linked
 to the comment. There is no need to include a second link in the title.

See #77
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Dec 12, 2015

New Screenshots

As an update to the screenshots I posted at the top, here is how things look on the desktop (Gmail) and on mobile (an iPhone 6s) after the changes I committed in #174 (Lite) and wpsharks/comment-mail-pro#35 (Pro):

Gmail (desktop)

2015-12-12_18-08-53

iPhone 6s (mobile)

2015-12-12_18-05-55

2015-12-12_18-06-25

@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Dec 12, 2015

Next Lite Release Changelog:

  • Enhancement: Many improvements the default email templates. The templates are now fully responsive and look much better on mobile devices. The wording in several places was also cleaned up for clarity and consistency and any references to an ID# (which is generally useless for readers) have been removed. The default templates also use a single font style now instead of mixing font styles and the email templates no longer use tables (this improves responsiveness). If you've modified the default Comment Mail email templates on your site, your changes will not be overwritten by updating, however if you've modified the templates and you want to get these changes, you'll need to reset to the default email templates and make your changes again (you can do this by emptying the email template fields completely and saving your options). See Issue #77.
  • Bug Fix: Fixed an email template bug that was forcing an ellipsis to show even when the length of the comment content was shorter than the max length. See this commit for details.
  • Bug Fix: Fixed an email template bug that was not linking the "add reply" link directly to the comment form for replying to the comment (it was just linking to the comment itself). See this commit for details.
  • Enhancement: Ellipses (...) in email templates are now surrounded by square brackets (i.e., [...]) to avoid confusion about whether or not the ellipsis was part of the comment content. See Issue #77.
  • Enhancement: In the email templates, the "Add Reply" links have been changed to "Reply" and when the reply is in response to someone and we have the authors name, the link includes the authors name (e.g., "Reply to "). See Issue #77.
  • Enhancement: Updated email templates to use \WP_Comment instead of \stdClass where appropriate. See Issue #77.
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Dec 12, 2015

Next Pro Release Changelog:

  • Enhancement: Many improvements the default email templates. The templates are now fully responsive and look much better on mobile devices. The wording in several places was also cleaned up for clarity and consistency and any references to an ID# (which is generally useless for readers) have been removed. The default templates also use a single font style now instead of mixing font styles and the email templates no longer use tables (this improves responsiveness). If you've modified the default Comment Mail email templates on your site, your changes will not be overwritten by updating, however if you've modified the templates and you want to get these changes, you'll need to reset to the default email templates and make your changes again (you can do this by emptying the email template fields completely and saving your options). See Issue #77.
  • Bug Fix: Fixed an email template bug that was forcing an ellipsis to show even when the length of the comment content was shorter than the max length. See this commit for details.
  • Bug Fix: Fixed an email template bug that was not linking the "add reply" link directly to the comment form for replying to the comment (it was just linking to the comment itself). See this commit for details.
  • Enhancement: Ellipses (...) in email templates are now surrounded by square brackets (i.e., [...]) to avoid confusion about whether or not the ellipsis was part of the comment content. See Issue #77.
  • Enhancement: In the email templates, the "Add Reply" links have been changed to "Reply" and when the reply is in response to someone and we have the authors name, the link includes the authors name (e.g., "Reply to "). See Issue #77.
  • Enhancement: Updated email templates to use \WP_Comment instead of \stdClass where appropriate. See Issue #77.
@raamdev raamdev closed this Dec 12, 2015
@jaswrks
Copy link

@jaswrks jaswrks commented Dec 15, 2015

👍 👍 👍 👍 👍 👍

@wpsharks wpsharks locked and limited conversation to collaborators Dec 24, 2015
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Dec 24, 2015

Comment Mail v151224 has been released and includes changes from this GitHub Issue: See the v151224 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#77).

@raamdev raamdev removed their assignment Apr 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.