Re: [PATCHv7 2/9] gitweb: uniform author info for commit and commitdiff

On Sat, Jun 27, 2009 at 8:38 PM, Junio C Hamano[off-list ref] wrote:

Jakub Narebski [off-list ref] writes:

quoted

I would use:

  Switch from form similar to the one used by 'log' view

      A U Thor [off-list ref] [date time]

  to the form used in 'commit' view

      author       A U Thor [off-list ref]
                   date time
      committer    C O Mitter [off-list ref]
                   date time

(i.e. use spaces and not tabs to align).  But this is minor
issue, not worth worrying about IMVHO.

These extra blank lines make things much easier to read, and explicit
mention of switching from WHAT to WHAT ELSE is very much appreciated.

I updated the patch accordingly.

quoted

On the other hand side IIRC 'commitdiff' uses short (one-line)
authorship info because the main point is the diff, and multi-line
author and commit info like the one used in 'commit' view takes
a bit of vertical space.

In general, we might want to make the committer information less prominent
than it currently is.

I've been thinking about this. A possible approach I considered could
be to only show the commit _date_ by default (which could be
interesting to see), and use some DHTML to show the committer on
request. But as you mention, this is not the patchset to do this kind
of work on 8-)

-- 
Giuseppe "Oblomov" Bilotta