Closed (fixed)
Project:
Comment Notify
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Mar 2010 at 21:06 UTC
Updated:
3 Aug 2011 at 19:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
gregglesYes, that can definitely be improved. Can you suggest a specific change? That makes it easier to discuss and fix.
Comment #2
WildBill commentedI used String Overrides to change the following...
From:
Receive node follow-up notification e-mailsTo:
<strong>Receive <i>content</i> follow-up notification e-mails</strong>The
<strong>tags were added to match the other comment notify header that immediately follows. This might be unique to my theme. The<i>tags were added to emphasize the difference in wording between this option and the one that follows (which is about<i>comments</i>)I also changed...
From:
Check this box to receive an e-mail notification for follow-ups on your nodes (pages, forum topics, etc). You can not disable notifications for individual threads.To:
Check this box to automatically receive e-mail notification of comments on content you've created (articles, forum topics, polls, etc). You cannot disable this on a post-by-post basis.Of course the mention of articles, forum topics, and polls is unique to my site. But I do like the phrasing "notification of comments on content you've created". Seems very clear and specific to me.
Just my suggestion. :)
Comment #3
doublejosh commentedThis is a need for me as well. Certainly want to remove the word from node from users learning curve.
Could even just change it to:
The words "(pages, forum topics, etc)" is bad depending on what modules and content types the site is used for.
If you want to get fancy you could include the first 2-3 content types available to the user.
Comment #4
doublejosh commentedHere's the patch to do what's described above.
Comment #6
doublejosh commentedHere's another go at it via a clone rather than a drush and git init...
Comment #7
doublejosh commentedwhop same name
Comment #9
doublejosh commentedIs it just the version that's causing them to fail?
Comment #10
gregglesThanks, doublejosh - now committed http://drupalcode.org/project/comment_notify.git/commit/43cccaf
and 7.x http://drupalcode.org/project/comment_notify.git/commit/d389b5a
I'm not sure why the tests were failing. It also didn't apply - I just manually applied it since the change was trivial.
Comment #12
doublejosh commented@greggles Thanks! Feels good to have a commit in this project.