git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Elijah Newren <newren@gmail.com>
To: Paul Mackerras <paulus@samba.org>
Cc: Git Mailing List <git@vger.kernel.org>,
	Alexander Gavrilov <angavrilov@gmail.com>
Subject: Re: [RFC with a proof of concept PATCH] Add Bonsai-like query options  to gitk
Date: Mon, 23 Mar 2009 11:56:59 -0600	[thread overview]
Message-ID: <51419b2c0903231056h4eef8d6cgef42f38bb21db1d5@mail.gmail.com> (raw)
In-Reply-To: <51419b2c0903231054g621787b6i8537b8e0d5bf121d@mail.gmail.com>

(Sorry, resending my own email since I accidentally left of the cc's.)

On Mon, Mar 23, 2009 at 4:07 AM, Paul Mackerras <paulus@samba.org> wrote:
> Elijah Newren <newren@gmail.com> writes:
>
>> (If you like screenshots, I've got some before & after ones up at
>> http://www.gnome.org/~newren/temp/gitk/gitk-edit-view.html, but
>> remember that this is just a proof of concept and call for comments.)
>
> Looks reasonable... have you developed the patch any further?

Yes, I have.  There was a cosmetic tweak or two, but the main change
was fixing the view remembering stuff that I noted I broke in the
original patch.  I've also showed this modified dialog in two
different git presentations, to dozens of new and potential users of
git, and it seems to have been well received[1].  I'll send a
follow-up patch shortly (I'd copy it inline, but I'm afraid gmail will
screw it up).

Elijah


[1] For one of the two groups, a lack of a Bonsai replacement actually
derailed their conversion to svn about a year ago.  This group was
very interested in git after my presentation and seemed satisfied with
gitk (with my proposed changes) as a replacement for Bonsai.  There
was one minor question that did come up each time -- one person in
each presentation asked why it is that when you specify --author=smith
(at the command line or via the equivalent box in the gui), you also
see commits authored by e.g. "jones" in gitk; however, once I
mentioned the different colored dots in gitk and the fact that it's
showing the commits that the patches are relative to, there was no
more issue and everyone was happy.  I'm guessing it'd be slightly more
intuitive to them to have gitk filter out these parent commits from
the graph unless they also meet the search criteria, but it's a pretty
minor issue.

  parent reply	other threads:[~2009-03-23 17:58 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-09 19:42 [RFC with a proof of concept PATCH] Add Bonsai-like query options to gitk newren
2009-03-23 10:07 ` Paul Mackerras
     [not found]   ` <51419b2c0903231054g621787b6i8537b8e0d5bf121d@mail.gmail.com>
2009-03-23 17:56     ` Elijah Newren [this message]
2009-03-23 17:57   ` [PATCH] gitk: Make more options easily accessible from Edit View dialog newren

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=51419b2c0903231056h4eef8d6cgef42f38bb21db1d5@mail.gmail.com \
    --to=newren@gmail.com \
    --cc=angavrilov@gmail.com \
    --cc=git@vger.kernel.org \
    --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).