Project

General

Profile

Actions

Feature #2221

open

gtk4: synchronize update_animation with idle tick

Added by Marko Lindqvist 2 days ago. Updated 1 day ago.

Status:
In Review
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 (1 open0 closed)

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

Actions
Actions #1

Updated by Marko Lindqvist 2 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 2 days ago

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

Updated by Caeies Caeies 1 day 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 1 day 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

Also available in: Atom PDF