Project

General

Profile

Actions

Defect #26403

closed

The second and subsequent lines of commit messages are not displayed in repository browser

Added by Go MAEDA over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Category:
SCM
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

After r16062, only the first line of a commit message is displayed in revisions list of the repository browser.

In my opinion, repository browser should not truncate commit messages because it often contain important information for developers.

I hope that this change will be undone in 3.4.2.

Redmine 3.3:

Redmine 3.4:


Files

revisions-3_3@2x.png (72.3 KB) revisions-3_3@2x.png Go MAEDA, 2017-07-10 15:39
revisions-3_4@2x.png (68.3 KB) revisions-3_4@2x.png Go MAEDA, 2017-07-10 15:39
dont-truncate-commit-message.diff (913 Bytes) dont-truncate-commit-message.diff patch Go MAEDA, 2017-07-10 15:42
hg.png (364 KB) hg.png Toshi MARUYAMA, 2017-07-19 20:03
Actions #1

Updated by Toshi MARUYAMA over 7 years ago

But your patch shows all text.
Redmine 3.3 truncates to 255 chars.
source:tags/3.3.4/app/helpers/repositories_helper.rb#L29

Actions #2

Updated by Yuichi Ishii over 7 years ago

This problem stops my team from updating to 3.4.0.
I hope this will be fixed as soon as possible.

Actions #3

Updated by Jean-Philippe Lang over 7 years ago

Yuichi Ishii wrote:

This problem stops my team from updating to 3.4.0.
I hope this will be fixed as soon as possible.

Do you really need to see more than the first line of the message in the revision list? The full commit message is still visible on the commit details view.

Actions #4

Updated by Yuichi Ishii over 7 years ago

Do you really need to see more than the first line of the message in the revision list? The full commit message is still visible on the commit details view.

Thanks for the reply. Yes, we really do.
Most of our commit messages contain more than one line, and reviewers use the repository browser in order to see recent commits roughly.

Actions #5

Updated by Yuuki NARA over 7 years ago

+1
I agree with the need for multi line display of the repository browser.
(It is also possible with option setting)

Traceability for tickets and changing source code is necessary for maintenance efficiency.

In order to investigate the outline of source code change details later .
It is necessary to use multiple line and new line , to make it easy to see.

If you force it into one line, it becomes difficult to grasp the contents

Actions #6

Updated by Jean-Philippe Lang over 7 years ago

  • Target version set to 3.4.3
Actions #7

Updated by Jean-Philippe Lang over 7 years ago

Should we truncate at 255 characters (which was the behaviour in all prior versions) or display the full commit message in the revision list?

Actions #8

Updated by Toshi MARUYAMA over 7 years ago

I think truncating at some characters is better because full text is very annoying.

https://www.mercurial-scm.org/repo/hg/shortlog/4.3-rc

Actions #9

Updated by Jean-Philippe Lang over 7 years ago

What about displaying the first line with a toggle button that would show the full commit message?

Actions #10

Updated by Yuichi Ishii over 7 years ago

What about displaying the first line with a toggle button that would show the full commit message?

Thank you for this nice idea. But the 3.3 or older versions style is better for us.
Because it helps us see outlines of commit messages easily.

Actions #11

Updated by Jean-Philippe Lang over 7 years ago

  • Status changed from New to Resolved
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

r16843 restores the same behaviour as 3.3 for displaying commit messages in the revision list.

Actions #12

Updated by Jean-Philippe Lang over 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF