On Sat, Jun 27, 2009 at 8:38 PM, Junio C Hamano<gitster@xxxxxxxxx> wrote: > Jakub Narebski <jnareb@xxxxxxxxx> writes: > >> I would use: >> >> Switch from form similar to the one used by 'log' view >> >> A U Thor <email@xxxxxxxxxxx> [date time] >> >> to the form used in 'commit' view >> >> author A U Thor <email@xxxxxxxxxxx> >> date time >> committer C O Mitter <other.email@xxxxxxxxxxx> >> 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. >> 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html