When clicking the "add a comment" button on a post, you a redirected to a page with title "Reply to comment". It doesn't matter if you are the first to comment on the post, let alone that you are not at all replying to another comment (you are replying to the post itself).

"Reply to comment" is just not fit for al the cases it is used for...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

brianV’s picture

Title: usability suggestion: "Reply to comment" » Usability: Make comment text consistent
Version: 6.4 » 7.x-dev
Issue tags: +Usability
FileSize
2.68 KB

In 7.x, this discrepancy still exists.

The default comment link is titles 'Add new comment', while the comment form page is titled 'Reply to comment' and the actual comment form is titled 'Reply'

These three elements need to be made consistent with one another.

I would propose 'Add a new comment' as a good replacement for all three. I've attached a patch to implement that; of course, alternate suggestions are encouraged.

Also tagging for Usablity.

brianV’s picture

Status: Active » Needs review
catch’s picture

Looks good to me, also aligns well with 'Add new content' - is there a problem using 'Add new comment' everything though and skipping the 'a'?

brianV’s picture

No problem. Just 'Add a new comment' makes better grammatical sense then 'Add new comment'.

If the concensus is that it should be changed, I will.

Dries’s picture

Status: Needs review » Needs work

"Add a new comment" doesn't flow as fast as "Add new comment". Can we standardize on "Add new comment" first, and then debate the "a" as a follow-up -- can be done in the same issue.

brianV’s picture

Status: Needs work » Needs review
FileSize
1.57 KB

New patch with 'Add new comment' across the board.

Dries’s picture

Committed this patch. Let's keep discussing the 'a' now.

brianV’s picture

I don't know how much discussion is needed, since there isn't a 'right' way and a 'wrong' way.

'Add new comment' does indeed flow better as you pointed out. 'Add a new comment' seems a little more 'correct'.

However, it's not a big enough deal to make it worth patching. The purpose is clearly communicated in both strings, so I am happy enough just leaving it as it is.

catch’s picture

Status: Needs review » Fixed

Since BrianV is happy with the change as is I'm going to mark this to fixed.

Shai’s picture

Woot, woot! I filed this issue in Nov. '07 (http://drupal.org/node/189659) and a discussion went back and forth with different possibilities that didn't coalesce before code froze for 6.... 42 posts in all. But here, just 9 posts and three days, and voila -- a commit for 7! Sometimes the world is just ready for something, no hassle necessary :)

Thanks for all the great work brianV, Dries, and catch.

Shai

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.