This is the mail archive of the xconq7@sources.redhat.com mailing list for the Xconq project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Civ game


>  | > It is a awful patch, but this patch will make some strings clickable.
>  |
>  | That works pretty good...except that from "Copper Melting", clicking on
>  | "The Furnace" gets you to "The Merchant" ...
>  |
>  | 	Steve
>
>I have fixed it and added a 'tree widget' in the help dialog.
>this patch requires BWiget-1.4.1 <http://sourceforge.net/projects/tcllib/>.

This is a great addition to xconq! Thank you for the work you put into
this. The clickable link code now works without problems. However, there
are still some problems with the tree widget:

First, BWidget distorts all listboxes by halving the widths of the scroll
bars (this is under Mac and Windows - I haven't checked Linux yet). Could
we disable or exclude the listbox code in BWidget and still keep tree
widget support?

Second, it seems that the tree widget only works for a few advances right
now. Most advances fail to display a tree, and if you click on them the
tree widget just switches to the next advance in the list.

Third, is the new file advance-needed-to-research.txt really necessary? It
should be possible to extract this info from the
aa_advance_needed_to_research table. If advances.g is changed in the future
without updating this other file as well, things will get messed up.

Fourth, since BWidget is not part of the standard tcltk package, making
xconq dependent on it will reduce the number of people who can install and
run xconq. One possible solution, if BWidget really is needed for the tree
widget, could of course be to include BWidget (or the tree widget part of
it) in the xconq for tcltk packages, just like libcurses is included in the
cconq (xconq for curses) package.

I therefore suggest that you check in your clickable link code (the first
patch that you posted updated with the "set cnt $cnt+1" fixes) as soon as
possible. For the tree widget, perhaps you could take a look at the above
problems and see to what extent thay can be dealt with?

Hans

Hans Ronne

hronne@pp.sbbs.se



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]