Remove the word "commented" in issues comment thread

Description

Remove the word "commented" in every entry of a comment thread of any issue. So it would just say something like the following. Look at screenshot.

Foo Bar
@foobaruser 4 months ago
I agree that this would make a great feature and let's make this comment thread longer by talking about it more.

Design considerations

In modern web / app design, the vertical comment thread or messaging flow is a common, well-understood UI element. Typically, each entry identifies a person (or a bot), a timestamp, and the content of the entry. Since this is so simple and well-understood by the common user, a good design should minimize unnecessary words and graphics, and allow people to skim and scroll quickly. So the word "commented" is just extraneous. Furthermore, it aligns with the auto-generated comments, which is good. See screenshot.

Open source community

This would be a great candidate for an up-for-grabs label.

Screenshot

@awhildy : If you agree, we can see if someone in the community wants to pick this up quickly.

@victorwu Thanks! I agree the word 'commented' isn't necessary. I'm all for removing things, and the word 'commented' is implied by the context. Let's go for it!

As an FYI, we are changing our system notes (the autogenerated messages): #19797 (closed). The goal is to clean them up, collapse long system notes, and make them generally smaller/visually quieter. The verb in system notes seems necessary, though, and I think it is alright to keep it there.

Assignee removed

Added frontend label

Removed ~19492 label

Added ~131895 label

Thanks @awhildy . Another small one I created I will close then, since the system notes one is already handling it.

This one: #24395 (moved)

Added ~123454 label

Mentioned in merge request !7464 (closed)

I fixed this with https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7464

@nmrony made a good point yesterday, are we still cool with removing this word in light of #19797 (closed)?

@nmrony + @ClemMakesApps:

I looked at #19797 (closed) several times again. I can see that the human-entered entries are purposefully designed to be visually different from the system-generated ones so that users can quickly distinguish. So I can see including commented or not including it to be both valid. I'll defer to @awhildy . (Sorry I'm not helping much =p)

Reassigned to @awhildy

Mentioned in merge request !7465 (closed)

I think I would say leave it as is since they are all structured as Name action time ago with https://gitlab.com/gitlab-org/gitlab-ce/issues/19797

My feeling was that the action is needed for the system notes because their sole goal is to call out the action, but that the action for comments is implied purely by the context of the comment right below. I figured there wasn't confusion with Name time ago because that is how system notes have existed for a while and they seem to work, and in this case, I leaned towards less is more as comments are a very established pattern.

The counter argument is obviously explicit consistency of Name action time ago. Since several people have brought up this concern, it makes since to stick with what is consistent, and keep commented. Thanks for the feedback!

Assignee removed

Status changed to closed

Closing issue since we are no longer going forward with this feature proposal