The list of loadable maps for the editor does not list number of players

Bug #676241 reported by Hans Joachim Desserud
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Medium
Unassigned

Bug Description

Recently, the list of maps were updated to display number of players along with the name of the map, which is a really nice addition. While this change has been implemented in both single- and multi-player, the list of maps for the editor (Main menu->Editor->Load map and Load map dialog from in-editor menu) is still using the old list with just map name. I think this list should be updated as well, in order to remain consistent with the others.

Tags: editor ui
Revision history for this message
SirVer (sirver) wrote :

actually, this list should list filenames as the in-editor load list uses. Or rather, this list should not be there in the first place; it's only right to existence is because the editor reloads every bit of graphics anew when one loads a new map.

My ideal approach to solving this would be the following: unite all worlds into one. Then only load every bit of graphic only once. Now, the editor should only need to load graphics once and one does no longer need a load map from the main menu.

Changed in widelands:
status: New → Confirmed
milestone: none → build17-rc1
Revision history for this message
SirVer (sirver) wrote :

So, I put this on confirmed for now, though I think I will reword it. I play to give the editor some love in build 17.

Changed in widelands:
importance: Undecided → Medium
Revision history for this message
SirVer (sirver) wrote :

No love for the editor in b17 - at least not from me :(.

Changed in widelands:
milestone: build17-rc1 → none
Revision history for this message
SirVer (sirver) wrote :

Setting to incomplete for bug sweeping.

Changed in widelands:
status: Confirmed → Incomplete
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Based on comment #1, someone should take a look at this once the one_world branch has been merged.

Changed in widelands:
milestone: none → build19-rc1
status: Incomplete → Confirmed
tags: added: editor ui
Revision history for this message
SirVer (sirver) wrote :

Setting to incomplete for bug sweeping.

Changed in widelands:
status: Confirmed → Incomplete
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Resolved in r7249 (presumably due to the one_world branch). Main menu -> editor -> load map now sports the same list as in new game, including number of players (and also map size).

Changed in widelands:
status: Incomplete → Fix Committed
Revision history for this message
wl-zocker (wl-zocker) wrote :

I think it was rather GunChleoc's menu redesign :)
However, the list inside the editor has not been changed. Do we want it to show more information, too? (The same goes for the save map/save game dialog.)

Revision history for this message
SirVer (sirver) wrote :

I think this bug is clearly done. However I have some gripes with the state of things around the editor loading/saving:
- I opened bug 1390794 to remove the extra menu when clicking on editor.
- I opened bug 1390793 to bring back using filenames instead of map names in the editor.

GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.