tkdiff-discuss

[Tkdiff-discuss] tkdiff 4.0b1 available

[John K. <jo...@kl...>]

The first public beta of the upcoming release 4.0 is available on
tkdiff.sf.net...  Go to the "files" section and grab a copy.  The current
release version (3.09, or 3.0.9, depending on how you look at it) is there
as well.

Thanks,
John

--
John Klassa / jo...@kl...

Please use this address for all personal correspondence.  Thanks!

Re: [Tkdiff-discuss] tkdiff 4.0b1 available

[<dan...@ta...>]

Hi!

In line 153-158 you have:

------------------------------------------------------
set w(selcolor) [lindex [.testent configure -selectbackground] 3]
set entryfont [lindex [.testent configure -font] 3]
destroy .testent
# the above results in a nearly undistinguishable darker gray for the
# selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
set w(selcolor) "#b03060"
------------------------------------------------------

I think the problem is the index 3 when setting w(selcolor):

------------------------------------------------------
set w(selcolor) [lindex [.testent configure -selectbackground] 4]
set entryfont [lindex [.testent configure -font] 3]
destroy .testent
------------------------------------------------------

=2E.. sets the value of "*selectBackground" from your .Xdefaults (or #bfd=
fff,=20
a light blue, as default) into w(selcolor).

An other solution would be:

------------------------------------------------------
set w(selcolor) [lindex [.testent configure -selectforeground] 4]
set entryfont [lindex [.testent configure -font] 3]
destroy .testent
------------------------------------------------------

In this case you get the value of "*selectForeground". Better than using =
the=20
background color for the foreground of a checkbox.

Is there a way to get the value of "*selector" from the .Xdefaults? I thi=
nk=20
this would be the "correct" value for this case.

Greetings,
    Daniel

--=20
Daniel H=F6pfl                                            targit GmbH
                                                R=FCdesheimer Stra=DFe 7
Tel: +49 (89) 517 397 - 0                           D-80686 M=FCnchen
Fax: +49 (89) 517 397 - 77                    http://www.targit.de/

Re: [Tkdiff-discuss] tkdiff 4.0b1 available

[Dorothy R. <mo...@ar...>]

Hmm.  On all the wm's I have at hand, indices 3 and 4 are the same.  Howe=
ver, I=20
see I was grabbing the color from the wrong place anyway, and I should ha=
ve got=20
it from the test menu:

*** 138,143 ****
--- 138,144 ----
   . configure -menu .native
   menu .native
   set menufont [lindex [.native configure -font] 3]
+ set w(selcolor) [lindex [.native configure -selectcolor] 3]
   destroy .native

   # Find out what the tk default is
***************
*** 150,161 ****
   destroy .testtext

   entry .testent
- set w(selcolor) [lindex [.testent configure -selectbackground] 3]
   set entryfont [lindex [.testent configure -font] 3]
   destroy .testent
- # the above results in a nearly undistinguishable darker gray for the
- # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
- set w(selcolor) "#b03060"

   # Make sure they're specified in points not pixels.  AquaTk doesn't.
   if {! [regexp -- {-} $labelfont]} {
--- 151,158 ----

Dorothy

Daniel H=F6pfl wrote:
> Hi!
>=20
> In line 153-158 you have:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 3]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> # the above results in a nearly undistinguishable darker gray for the
> # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> set w(selcolor) "#b03060"
> ------------------------------------------------------
>=20
> I think the problem is the index 3 when setting w(selcolor):
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> ... sets the value of "*selectBackground" from your .Xdefaults (or=20
> #bfdfff, a light blue, as default) into w(selcolor).
>=20
> An other solution would be:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> In this case you get the value of "*selectForeground". Better than usin=
g=20
> the background color for the foreground of a checkbox.
>=20
> Is there a way to get the value of "*selector" from the .Xdefaults? I=20
> think this would be the "correct" value for this case.
>=20
> Greetings,
>    Daniel
>=20

--=20
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=
=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
Dorothy Robinson
http://www.twobarleycorns.net

Re: [Tkdiff-discuss] tkdiff 4.0b1 available

[Dorothy R. <mo...@ar...>]

I found a mistake which caused filenames loaded from the Browse dialog - =
ie., on=20
a PC - to not withstand spaces in the filenames.  I fixed it, along with =
the=20
setcolor problem, and submitted a patch on Sourceforge.  (I'm sure the up=
load=20
"took" this time.)

On another subject - I want to allow tkdiff to do a cvs rdiff like
"cvs -d repository rdiff -r1.1 -r1.2 modulepath/filename" on a file that =
isn't=20
checked out.  Right now it won't do that, because it doesn't know from rd=
iff and=20
because it checks to see if the file given in the argument exists, which =
it=20
won't in this case.  So I thought of adding a command line option for doi=
ng an=20
rdiff, and removing the check for the existence of the file.  Something l=
ike
"tkdiff -d repository -rdiff -r1.1 -r1.2 modulepath/filename"
Objections?  Better ideas?  I can't help thinking there might be an easie=
r way.

Dorothy

Daniel H=F6pfl wrote:
> Hi!
>=20
> In line 153-158 you have:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 3]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> # the above results in a nearly undistinguishable darker gray for the
> # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> set w(selcolor) "#b03060"
> ------------------------------------------------------
>=20
> I think the problem is the index 3 when setting w(selcolor):
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> ... sets the value of "*selectBackground" from your .Xdefaults (or=20
> #bfdfff, a light blue, as default) into w(selcolor).
>=20
> An other solution would be:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> In this case you get the value of "*selectForeground". Better than usin=
g=20
> the background color for the foreground of a checkbox.
>=20
> Is there a way to get the value of "*selector" from the .Xdefaults? I=20
> think this would be the "correct" value for this case.
>=20
> Greetings,
>    Daniel
>=20

--=20
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=
=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
Dorothy Robinson
http://www.twobarleycorns.net

RE: [Tkdiff-discuss] tkdiff 4.0b1 available

[John K. <jo...@kl...>]

I applied your patch...  Everything's cool (at least under cygwin), =
except
that when you run tkdiff without a filename, it flashes and thrashes and
generally goes nuts.  If you supply a filename, all is well.  Any ideas?

Your version is 1.12 in CVS.

As for your other changes (and David's proposed changes), let's wait =
until
the dust settles on b2 and get 4.0 out the door...  After that, we can
figure out how best to add the feature you're after, in light of David's
work.  Sound okay with everybody?

Thanks, Dorothy!

John

--
John Klassa / jo...@kl...

Please use this address for all personal correspondence.  Thanks!

> -----Original Message-----
> From: John Klassa [mailto:kl...@qw...] On Behalf Of Dorothy =
Robinson
> Sent: Wednesday, May 14, 2003 2:33 AM
> To: tkd...@so...
> Subject: Re: [Tkdiff-discuss] tkdiff 4.0b1 available
>=20
> I found a mistake which caused filenames loaded from the Browse dialog =
-
> ie., on
> a PC - to not withstand spaces in the filenames.  I fixed it, along =
with
> the
> setcolor problem, and submitted a patch on Sourceforge.  (I'm sure the
> upload
> "took" this time.)
>=20
> On another subject - I want to allow tkdiff to do a cvs rdiff like
> "cvs -d repository rdiff -r1.1 -r1.2 modulepath/filename" on a file =
that
> isn't
> checked out.  Right now it won't do that, because it doesn't know from
> rdiff and
> because it checks to see if the file given in the argument exists, =
which
> it
> won't in this case.  So I thought of adding a command line option for
> doing an
> rdiff, and removing the check for the existence of the file.  =
Something
> like
> "tkdiff -d repository -rdiff -r1.1 -r1.2 modulepath/filename"
> Objections?  Better ideas?  I can't help thinking there might be an =
easier
> way.
>=20
> Dorothy
>=20
> Daniel H=F6pfl wrote:
> > Hi!
> >
> > In line 153-158 you have:
> >
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectbackground] 3]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > # the above results in a nearly undistinguishable darker gray for =
the
> > # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> > set w(selcolor) "#b03060"
> > ------------------------------------------------------
> >
> > I think the problem is the index 3 when setting w(selcolor):
> >
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >
> > ... sets the value of "*selectBackground" from your .Xdefaults (or
> > #bfdfff, a light blue, as default) into w(selcolor).
> >
> > An other solution would be:
> >
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >
> > In this case you get the value of "*selectForeground". Better than =
using
> > the background color for the foreground of a checkbox.
> >
> > Is there a way to get the value of "*selector" from the .Xdefaults? =
I
> > think this would be the "correct" value for this case.
> >
> > Greetings,
> >    Daniel
> >
>=20
> --
> =
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=
=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
> Dorothy Robinson
> http://www.twobarleycorns.net
>=20
>=20
>=20
> -------------------------------------------------------
> Enterprise Linux Forum Conference & Expo, June 4-6, 2003, Santa Clara
> The only event dedicated to issues related to Linux enterprise =
solutions
> www.enterpriselinuxforum.com
>=20
> _______________________________________________
> Tkdiff-discuss mailing list
> Tkd...@li...
> https://lists.sourceforge.net/lists/listinfo/tkdiff-discuss

Re: [Tkdiff-discuss] tkdiff 4.0b1 available

[Dorothy R. <mo...@ar...>]

Flashes and thrashes??  I can't get that to happen on my PC, either with Cygwin 
or with a plain old DOS window and the Windows version of Tk.  Both Tk's are 
8.3, though.  I had 8.4 on my PC for awhile and it did some very strange things, 
though flashing and thrashing weren't usually among them.  Or are you running Tk 
under X11 in Cygwin?  I haven't tried that yet.  (Only have a PC to do things 
that _can't_ be done on a reasonable computer ...)

dorothy

John Klassa wrote:
> I applied your patch...  Everything's cool (at least under cygwin), except
> that when you run tkdiff without a filename, it flashes and thrashes and
> generally goes nuts.  If you supply a filename, all is well.  Any ideas?

-- 
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Dorothy Robinson
http://www.twobarleycorns.net