git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Heiko Voigt <hvoigt@hvoigt.net>
To: Paul Mackerras <paulus@samba.org>,
	Nicolas Cornu <ncornu@aldebaran-robotics.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH v2] gitk: Add a horizontal scrollbar for commit history
Date: Sat, 02 Nov 2013 10:40:45 +0100	[thread overview]
Message-ID: <5274C89D.10103@hvoigt.net> (raw)
In-Reply-To: <20131031090526.GA9164@iris.ozlabs.ibm.com>

Hi,

Am 31.10.2013 10:05, schrieb Paul Mackerras:
> On Wed, Oct 30, 2013 at 01:47:08PM +0100, Nicolas Cornu wrote:
>> This is useful on all our repos, every times, as we put a tag per day.
>> If the HEAD didn't move during 150 days, we got 150 tags.
>
> Here is a patch that I did some time ago but have never pushed out.
> Do you think it is an improvement when using gitk on a repo with lots
> of tags?
>
> Paul.
>
> [PATCH] gitk: Tag display improvements
>
> When a commit has many tags, the tag icons in the graph display can
> easily become so wide as to push the commit message off the right-hand
> edge of the graph display pane.  This changes the display so that if
> there are more than 3 tags or they would take up more than a quarter
> of the width of the pane, we instead display a single tag icon with
> a legend inside it like "4 tags...".  If the user clicks on the tag
> icon, gitk then displays all the tags in the diff display pane.
>
> Signed-off-by: Paul Mackerras <paulus@samba.org>

Yes please. I have not tried it but the description sounds great. Will 
try to give it a testdrive next week.

Cheers Heiko

  reply	other threads:[~2013-11-02  9:49 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-30 10:58 [PATCH v2] gitk: Add a horizontal scrollbar for commit history Nicolas Cornu
2013-10-30 12:01 ` Johannes Sixt
2013-10-30 12:47   ` Nicolas Cornu
2013-10-30 14:46     ` Marc Branchaud
2013-10-30 14:49       ` Nicolas Cornu
2013-10-30 15:04         ` Marc Branchaud
2013-10-31  9:05     ` Paul Mackerras
2013-11-02  9:40       ` Heiko Voigt [this message]
2013-11-04 15:50       ` Marc Branchaud

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5274C89D.10103@hvoigt.net \
    --to=hvoigt@hvoigt.net \
    --cc=git@vger.kernel.org \
    --cc=ncornu@aldebaran-robotics.com \
    --cc=paulus@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).