From: "Shawn O. Pearce" <spearce@spearce•org>
To: Alex Riesen <raa.lkml@gmail•com>
Cc: Seth Falcon <sethfalcon@gmail•com>,
Junio C Hamano <junkio@cox•net>,
git@vger•kernel.org, Mark Levedahl <mdl123@verizon•net>
Subject: Re: [PATCH] Make gitk work reasonably well on Cygwin.
Date: Thu, 1 Feb 2007 14:24:32 -0500 [thread overview]
Message-ID: <20070201192432.GD18608@spearce.org> (raw)
In-Reply-To: <81b0412b0702010003t7becddf3x8fe6b36b0d84599b@mail.gmail.com>
Alex Riesen <raa.lkml@gmail•com> wrote:
> On 2/1/07, Seth Falcon <sethfalcon@gmail•com> wrote:
> >
> >> I am not in X right now and won't be able to look at it myself
> >> tonight. Could folks on MacOS comment on Mark's patch?
> >
> >I gave it a quick spin on OS X. gitk loads with a different geometry
> >than it had before, but seems to work the same. If this fixes things
> >elsewhere, it seems fine here.
> >
>
> FWIW, I confirm that. Much better. Mouse selection still does not
> work in patch pane, but broken layout was the most annoying.
The trick to getting mouse selection to work on Mac OS X on a
disabled text field (which is what that patch pane is):
bind $damn_text_widget <Button-1> [list focus $damn_text_widget]
Apparently the disabled text widgets on Mac OS X don't receive
focus unless its forced by the application. :-(
This trick doesn't harm a Tk application running on Windows, its
safe to just always setup the binding. I'm using it in git-gui
for the diff pane.
--
Shawn.
next prev parent reply other threads:[~2007-02-01 19:24 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-31 14:50 [PATCH] Make gitk work reasonably well on Cygwin Mike Nefari
2007-01-31 15:17 ` Johannes Schindelin
2007-01-31 16:00 ` Mark Levedahl
2007-01-31 16:33 ` Johannes Sixt
2007-01-31 16:44 ` Johannes Schindelin
2007-02-01 1:41 ` Mark Levedahl
2007-02-01 3:22 ` Junio C Hamano
2007-02-01 5:24 ` Seth Falcon
2007-02-01 8:03 ` Alex Riesen
2007-02-01 14:03 ` Mark Levedahl
2007-02-01 19:24 ` Shawn O. Pearce [this message]
2007-02-01 9:20 ` Johannes Schindelin
2007-02-01 13:56 ` Mark Levedahl
2007-02-01 14:05 ` Johannes Schindelin
2007-02-01 15:04 ` Mark Levedahl
2007-02-01 15:28 ` Christian MICHON
2007-02-01 15:39 ` Johannes Sixt
2007-02-01 16:00 ` Johannes Schindelin
2007-02-01 17:33 ` Mark Levedahl
2007-02-01 2:09 ` Mark Levedahl
2007-01-31 16:28 ` Johannes Sixt
2007-01-31 16:36 ` Alex Riesen
2007-01-31 16:39 ` Mark Levedahl
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=20070201192432.GD18608@spearce.org \
--to=spearce@spearce$(echo .)org \
--cc=git@vger$(echo .)kernel.org \
--cc=junkio@cox$(echo .)net \
--cc=mdl123@verizon$(echo .)net \
--cc=raa.lkml@gmail$(echo .)com \
--cc=sethfalcon@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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