From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 27 Nov 2009 18:29:27 GMT
Subject: Re: [RFC/PATCH] gitweb: Make linking to actions requiring JavaScript a feature
Message-ID: <7veinjn7nc.fsf@alter.siamese.dyndns.org>
In-Reply-To: <200911271641.40947.jnareb@gmail.com>

Jakub Narebski <jnareb@gmail.com> writes:

> Would turning
>
>   "blame"
>
> link ito pair of links
>
>   "blame (incremental)"
>
> be a good solution?  I'm trying to come up with good naming for extra link
> to 'blame_incremental' action...

I had to step back a bit and ask myself what we are trying to achieve
here.  When the current blame and incremental one are both working
perfectly and well, will there be a reason for the end users to choose
between them when they click?

My answer is no.  If the incremental one gives nicer user experience in
all cases, it will be shown without the non-incremental one; if the
incremental one makes the server or network load too heavy, a site owner
may decide to show only the non-incremental one.

That makes my addLinks suggestion a change that would help _only_ while we
are working kinks out of the incremental one.

Let's not waste too much effort doing that.  Sorry for suggesting.

Letting the site owner choose if the site wants to set the "incremental if
possible" boolean would be more than adequate, I think.