[Freeciv-Dev] Re: (PR#2858)

["Jason Short via RT" <rt@xxxxxxxxxxxxxx>]

Rafa³ Bursig via RT wrote:
> This patch fix some problems with redrawing GUI widgets in SDLClient.
> 
> All widgets are part of map and must be redraw after redraw of map,
> lastest unification of draw code make this redraw imposible inside gui code.
> 
> This function replace "show_city_descriptions()" in
> "update_map_canvas_visible()" and all clients use it like wraper to
> "show_city_descriptions()" funct.

As it is, this just pushes deeper the current unbuffered 
city-description algorithms...

One possible workaround is to redraw the map widgets on top of each 
show_city_desc and flush_mapcanvas.  But this could be very ugly for 
show_city_desc.

Another alternative would be to remove show_city_desc from 
redraw_map_widgets.  Then the only addition is a single call to 
redraw_map_widgets after the city descriptions are redrawn.  Would this 
work?

Ideally, thee city descriptions should be drawn onto a buffer of their 
own and flushed appropriatly - then the flush function could easily 
redraw the widgets.  But this is some ways off.

jason