abolish automatic resizing of minimap #4689
Comments
Forgon uploaded file bash script that finds all maps inside given files and directories and write their names next to their dimensions to the file map_sizes.csv |
Forgon uploaded file list of 680 different maps sorted by their width |
Forgon uploaded file |
Forgon uploaded file comparison of radar zoom settings 14, 16 and 32 (GIMP file with layers) |
Forgon uploaded file illustration of radar rotation shifting the center of minimaps, with red lines and a dot outlining the squared minimap area and its center |
Forgon uploaded file illustration of radar rotation shifting the center of minimaps, with red lines and a dot outlining the squared minimap area and its center (GIMP file with layers) |
Forgon uploaded file comparison of radar size with rotateRadar set to true and false for the map HugeassRebuild |
Forgon uploaded file comparison of radar size with rotateRadar set to true and false for the map HugeassRebuild (GIMP file with layers) |
Forgon uploaded file patch file created with |
Forgon uploaded file patch file created with |
Forgon uploaded file patch file created with |
Forgon2100 <forgon2100@...> changed status from |
Forgon2100 <forgon2100@...> changed owner from `` to |
Forgon2100 <forgon2100@...> changed resolution from `` to |
Forgon2100 <forgon2100@...> committed [0752] In Warzone2100/warzone2100@0752c2b:
|
Forgon2100 <forgon2100@...> commented In Warzone2100/warzone2100@af9a96f:
|
Forgon2100 <forgon2100@...> commented In Warzone2100/warzone2100@d3410d0:
|
Per commented On sk-rush, the minimap is now a useless, tiny postage stamp. How is this a good thing? sk-rush is the default map. If it doesn't look good on that map, it is broken. |
Forgon changed _comment0 which not transferred by tractive |
Forgon commented Replying to Warzone2100/old-trac-import#4689 (comment:4):
This could easily be fixed by increasing the default minimap size from 16 to 32. I would have liked to do that anyway, because I increase radar size immediately at the start of almost every game. Would you be okay with that change? |
Per commented Let's try that. We should always aim to have good defaults, since most players never change options. |
Forgon edited the issue description |
Forgon commented Replying to Warzone2100/old-trac-import#4689 (comment:6):
Done, with #4699. |
resolution_fixed
type_patch (an actual patch, not a request for one)
| by ForgonThis patch series abolishes automatic resizing of the minimap, which is desirable for a configuration option to change the default radar size, to be introduced with #4629.
Radar size ranges from 8 to 64 with a default value of 16 (see './src/radar.h').
A routine that assumed a value < 4 has been removed.
Maps with width < 150 tiles like Sk-Rush no longer start with a resized radar of 14 (if the screen is <= 640 pixels wide) or 32, which will likely be noticed by users (see attached image [raw-attachment:minimap_size_comparison.png]).
Almost every single one of the popular 8-player NTW maps is 140 tiles wide and will thus start with a smaller radar size. Among 680 different maps whose size was measured with a bash script (view [raw-attachment:list_map_sizes] for the script and [raw-attachment:map_sizes.csv] for the results), 36% were affected overall.
When rotateRadar was enabled, all minimaps were rescaled as though their larger side (either width or height) was 128 tiles long.
A map 143x159 tiles large (the average dimensions of 680 maps examined) would thus appear with a sidelength reduced by about ~ 20%. Note that maps cannot be larger than 256 tiles or smaller than 2 tiles on either side, although tiny maps are rare: the smallest popular multiplayer map, HugeassRebuild, is 21x21 tiles in size and thus appeared with a sidelength about 6-times larger with rotating radar than fixed radar (see attached image [raw-attachment:radar_rotation_enlarging_minimap.png]).
Rotating radar also shifted the center of the minimap so that it fit into a square of its largest sidelength (see attached image [raw-attachment:radar_rotation_centering_minimap.png]).
This cost valuable screen space: Out of 680 maps, 45% were higher than wide and would thus move to the left, 14% were wider than high and shifted upwards, while 40% had equal dimensions and thereby unaffected.
Lastly, the radar size in tutorials has also increased since a routine shrinking it was ignored when converting them to JavaScript (ticket #4641) and removed from the game altogether with 76ac44c84b7f89a860c743202bb002586d137e63.
Issue migrated from trac:4689 at 2022-04-16 12:58:32 -0700
The text was updated successfully, but these errors were encountered: