When a user posts a comment, there isn't a message that's displayed to them, they're just brought to where their comment was posted. It would be nice if they got a confirmation message at the top of the node as well.

This is a major issue when the user is able to post comments, but not view them (related issue), since they get absolutely no feedback for their posting. They are just brought back to the original node.

CommentFileSizeAuthor
#5 187061.patch691 bytesRobLoach
#4 187061.patch846 bytesRobLoach
#2 comment_message.patch689 bytesRobLoach
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Version: 6.x-dev » 7.x-dev

Moving to version 7.

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented
Version: 7.x-dev » 6.x-dev
Status: Active » Needs review
FileSize
comment_message.patch689 bytes

With this patch, when a user posts a message, they're presented with a message letting them know their comment has been posted.

This is beginning to be an issue on Drupal.org itself with the new caching system. When a user posts a comment, their comment doesn't display and they get no feedback until a minute or two later when their comment finally appears on the site. I think this is something that should be present in Drupal 6.x...

keith.smith’s picture

keith.smith CreditAttribution: keith.smith commented
Version: 6.x-dev » 7.x-dev
Component: comment.module » documentation

This would be nice, but would break the 6.x string freeze. Bumping to 7.x.

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented
Component: documentation » comment.module
FileSize
187061.patch846 bytes

Rerolled to HEAD and tested.

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented
FileSize
187061.patch691 bytes

Forgot the line breaks.

Pasqualle’s picture

Pasqualle
Location 🇭🇺 Budapest
CreditAttribution: Pasqualle commented

Isn't it the basic example of trigger module? I am not sure if we really need this message.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

Feedback is good. If we have messages for nodes, we should have them for comments too. So +1 from me.

jpoesen’s picture

jpoesen CreditAttribution: jpoesen commented

Applied against HEAD and tested. Works as advertised.

Pasqualle’s picture

Pasqualle
Location 🇭🇺 Budapest
CreditAttribution: Pasqualle commented
Status: Needs review » Reviewed & tested by the community
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Category: feature » bug

Still applies with offset. Not giving feedback from user interactions is a bug.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

If we want to make this a trigger, we should do so in a follow-up patch.

Thanks.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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