Minimap not redrawn often enough
In the Gtk3 client (S2_5 and S2_6), on a new game, if I go into the editor, select the vision tool, increase its size, and click on the map to reveal a large swath of territory, the minimap isn't updated until I right-click on the main map or minimap to scroll.
This works as I expect in the Gtk2 client.
I haven't checked if similar effects are visible in the game (the map editor being an easy way to cause large changes visible on the minimap).
#1 Updated by Christian Knoke 9 months ago
HostedRedmine.com wrote on Sep 17, 09:50 (+0000):
Issue #697658 has been reported by Jacob Nevins.
I haven't checked if similar effects are visible in the game (the map
editor being an easy way to cause large changes visible on the minimap).
This is the case (bug present in 2.6 Beta as of yesterday) and reported to
GNA months ago.
Christian Knoke * * * http://cknoke.de
- * * * * * * * * Ceterum censeo Microsoft esse dividendum.
#2 Updated by Jacob Nevins 9 months ago
- File gtk3_overview_flush.patch gtk3_overview_flush.patch added
- File gtk3_overview_flush-26-trunk.patch gtk3_overview_flush-26-trunk.patch added
- Status changed from New to In Progress
You reported it in gna bug #24082 "Map Update Bugs" among some other symptoms:
Vers. 2.5.1 Gtk3 cpmpiled from source, multiplayer, one-human game
In some cases, the map or the minimap is not updated correctly.
In all cases, the (mini)map display is corrected, if I right-click on a non-central spot of the main map (but maybe not by right-click in the minimap).
2 At the beginning of a new turn, units on minimap are not correct
at the beginning of a new turn, the minimap shows units (in red) that are not on the main map, been killed in the turn before, and are correctly not shown on the main map.
Attached patch fixes minimap issues.
It works the same way like in qt.
get_overview_window() is invoked when overview has changed and before redrawing to overview surface - which doesnt exist in gtk3.
Beacause overivew is drawn directly from gui_options.overview.window.
So making update during get_overview_window() is 100% accurate.
ML feel free to check and commit if u like.
There was some faff and I raised a more specific bug #25318 "Gtk3 client minimap doesn't update":
The Gtk3 minimap is often out of date. It only seems to update when the whole map is recentred.
- Moving units don't change the fogged areas, unit pixels, etc
- Using vision tool in editor doesn't update minimap
Seen in current (2.5.6) S2_5, and I've freshly confirmed it goes back at least as far back as 2.5.1.
chrisk previously reported this as part of bug #24082, and mir3x diagnosed the problem there (and wrote a patch?)
#3 Updated by Jacob Nevins 9 months ago
- File m-gtk3-overview-flush.patch m-gtk3-overview-flush.patch added
- Status changed from In Progress to Resolved
- Assignee set to Jacob Nevins
- Target version set to 2.5.10
This patch solves the problem for me in gui-gtk-3.0.
No sense sitting on mir3x's patch any longer; I intend to commit it.
I haven't tested gui-gtk-3.22 or -4.0, I'm just trusting that the fix is the right thing to do in those versions too.