Redmine

LGTM.

1. move the "Added by Jane Doe 5 months ago" line above the ID, Parent and Child information and add the bottom dotted border
2. show the ID, Parent and Child with bullet points and indented to right in the same way how the issue attributes are shown in the journals
3. show user avatar for commit author.

In this way, the commit message will have the same structure like the journal and it will be more easily for the user to found the information.

@Go MAEDA, if you agree with these proposals, I can update your patch or if it is not clear enough, I can make a print-screen with my suggestions.

Marius BALTEANU wrote:

For consistency, I would like to suggest three new changes to this patch:

1. move the "Added by Jane Doe 5 months ago" line above the ID, Parent and Child information and add the bottom dotted border
2. show the ID, Parent and Child with bullet points and indented to right in the same way how the issue attributes are shown in the journals
3. show user avatar for commit author.

It would be great (most interesting thing for me is showing user's icon) but I think you would be better to create another issue because these features are much bigger improvements than my 3 lines patch.

I have added the patch to #23146.

Unfortunately, it is not compatible with your patch because I changed from table to ul lists and the rules from this patch applies to table. Please take a look on my changes and if you like them, maybe you replace the current ticket with that one or let me know how do you want to proceed.

Also, I made a mistake in my previous comment, the user avatar was already displayed in that page, but before the revision number instead of user.