Inform map editors not to use more than 8 players
Bug #1691339 reported by
GunChleoc
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| widelands |
Fix Released
|
Low
|
Unassigned | ||
Bug Description
We now allow 16 players for AI testing, but map editors might not be aware that it's not a good idea (no UI support, maybe also laggy games). We should show them an unobtrusive message when they assign more than 8 players, both in the players and the map creation menus.
Related branches
lp://staging/~widelands-dev/widelands/bug-1691339-editor-8-players
- Klaus Halfmann: Approve (compile, test)
-
Diff: 875 lines (+367/-258)10 files modifiedsrc/editor/CMakeLists.txt (+1/-0)
src/editor/ui_menus/player_menu.cc (+269/-217)
src/editor/ui_menus/player_menu.h (+36/-29)
src/ui_basic/button.cc (+5/-2)
src/ui_basic/button.h (+3/-1)
src/ui_basic/dropdown.cc (+45/-9)
src/ui_basic/dropdown.h (+3/-0)
src/ui_basic/panel.cc (+1/-0)
src/ui_basic/panel.h (+3/-0)
src/wlapplication.cc (+1/-0)
| Changed in widelands: | |
| assignee: | nobody → GunChleoc (gunchleoc) |
| status: | New → In Progress |
Revision history for this message
| kaputtnik (franku) wrote | #1 |
Revision history for this message
| kaputtnik (franku) wrote | #2 |
| Changed in widelands: | |
| status: | In Progress → Fix Committed |
| assignee: | GunChleoc (gunchleoc) → nobody |
Revision history for this message
| GunChleoc (gunchleoc) wrote | #3 |
| Changed in widelands: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in build20-rc1