Project

General

Profile

Actions

Feature #2221

closed

gtk4: synchronize update_animation with idle tick

Added by Marko Lindqvist 23 days ago. Updated 17 days ago.

Status:
Closed
Priority:
Normal
Category:
gtk4-client
Target version:
Start date:
09/06/2026
Due date:
% Done:

0%

Estimated time:

Description

Split from #2186 : One of the patches by caeies in PR https://github.com/freeciv/freeciv/pull/90 is
"gtk4: synchronize update_animation with idle tick.
in GTK4 you should not g_idle_add yourself during idle loop execution."


Files


Related issues 1 (0 open — 1 closed)

Blocks Bug #2222: gtk4: fix proposal for resize stuffClosedMarko Lindqvist09/06/2026

Actions
Actions #1

Updated by Marko Lindqvist 23 days ago

Patch in PR:
- Function header for animation_tick_cb() has not been updated from animation_idle_cb() one - still speaks of idle callback
- Indentation is off (See doc/CodingStyle)

Actions #2

Updated by Marko Lindqvist 23 days ago

  • Blocks Bug #2222: gtk4: fix proposal for resize stuff added
Actions #3

Updated by Caeies Caeies 22 days ago

I created a new PR as suggested : https://github.com/freeciv/freeciv/pull/93

Indentation should be ok.
Comment too (if it is suitable).

Thanks.

Actions #4

Updated by Marko Lindqvist 22 days ago

Patch from the PR applies to S3_3 and S3_2. Port for the main branch (just applying identical change also to gui-gtk-5.0) attached here.

Actions #5

Updated by Marko Lindqvist 17 days ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF