[Freeciv-Dev] Re: (PR#4610) CMA frontend for Xaw

["andrearo@xxxxxxxxxxxx" <andrearo@xxxxxxxxxxxx>]

On Sat, 26 Jul 2003, Mike Kaufman wrote:
> A couple more things:
>
> o when you're moving sliders and you come upon a preset that preset should
>   highlight in the preset list.
> o the delete preset button should have a confirmation dialog.
> o I was going to say that the add_preset dialog should be parented to the
>   cma_dialog rather than top_level because it hides the city and cma dialogs
>   on popup, but it appears that it is parented correctly. strange.
> o add spaces here:       preset_lines[i]=preset_text[i];
> o go through cma_fe.c and add blank lines between code blocks where
>   necessary.
> o for extra brownie points, move the global variables into cma_dialog
>   struct. See gui-gtk. This is necessary for multiple invocations.

The patch is done, except for allowing multiple invocations.

Andreas Røsdal

xaw_cma_fe4.diff.gz
Description: xaw_cma_fe4.diff.gz