Re: [ANNOUNCE] git/gitweb.git repository

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



--- Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Here is my impressions on patches in:
> 
> 	git://repo.or.cz/git/gitweb.git/ next
> 
> * Removal of git_blame
> 
>   I think this makes sense; we do not need two not-so-different
>   implementations, and the other one has been the default for
>   quite some time.

My impression of the running gitweb installation at said site
is that it doesn't implement 244a70e608204a515c214a11c43f3ecf7642533a.
It is very easy to verify that it is not supported.

This is a data mining feature that I use regularly.

     Luben


> 
> * Extra columns in blame
> 
>   I can see why some people especially with wide screen would
>   want this, and toggling the display for extra columns is also
>   a good idea.  But this makes me wonder if the single button at
>   the very top is easy to locate (maybe explain what [+] does?)
>   and easy to access (maybe have it at the top and at least at
>   the bottom as well?).
> 
> * Incremental blame
> 
>   It does not seem to break the blame, but at least from where I
>   sit accessing repo.or.cz this does not look incremental to me.
>   The entire browser session freezes until the blame page
>   displays in full.  My local installation behaves the same way.
> 
> 
> 
>   
> 

-
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux