Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3798 closed enhancement (fixed)

[PATCH] Add map preview for 39 maps

Reported by: Yordan Grigorov Owned by: elexis
Priority: Should Have Milestone: Alpha 20
Component: Maps Keywords: patch
Cc: Patch:

Description (last modified by Yordan Grigorov)

Added a map preview for the following maps:

Skirmish:

Barcania (3) Corithian Isthmus (4) Skirmish Demo Two seas (6)

Random:

Mainland New RMS test Sahel Survival of the Fittest Wall Demo

Scenario:

Base Temperate Map Bridge Demo Campaign Map - Test Combat Demo Combat Demo (Huge) Eire and Albion Fishing Demo Flight Demo Flight Demo 2 Gorge Introductory Tutorial Miletus Multiplayer Demo Pathfinding Terrain Demo Reservoir Resource Demo Road Decals Demo Sandbox - The Carthaginians Sandbox - The Iberians Ship Formations Ships Demo Strategic Campaign Proof of Concept Territory Demo The Persian Gates Trading Demo Tropical Island Units Demo Walls WallTest We Are Legion

This is my first contribution to this game and I have never used trac before so I might have done something wrong

The only maps that are not demos from this list are: Mainland Sahel (Random) Survival of the Fittest

Some of those demos really tricked me into thinking they were real maps though (Barcania, Tropical Island, Persian gates, etc.) CC-BY-SA 3.0.

Attachments (5)

maps.diff (53.0 KB ) - added by Yordan Grigorov 8 years ago.
mappreview1.zip (1.5 MB ) - added by Yordan Grigorov 8 years ago.
mappreview2.zip (1.5 MB ) - added by Yordan Grigorov 8 years ago.
mappreview3.zip (1.9 MB ) - added by Yordan Grigorov 8 years ago.
mappreview4.zip (1.5 MB ) - added by Yordan Grigorov 8 years ago.

Change History (13)

by Yordan Grigorov, 8 years ago

Attachment: maps.diff added

by Yordan Grigorov, 8 years ago

Attachment: mappreview1.zip added

by Yordan Grigorov, 8 years ago

Attachment: mappreview2.zip added

by Yordan Grigorov, 8 years ago

Attachment: mappreview3.zip added

by Yordan Grigorov, 8 years ago

Attachment: mappreview4.zip added

comment:1 by Niek, 8 years ago

I think you can skip all these demo maps...

in reply to:  1 comment:2 by Yordan Grigorov, 8 years ago

Replying to niektb:

I think you can skip all these demo maps...

Of course. I had fun doing it though.

in reply to:  1 comment:3 by Yordan Grigorov, 8 years ago

Replying to niektb:

I think you can skip all these demo maps...

Do you mean that I should remove their previews, or just that the effort was unnecessary?

in reply to:  1 ; comment:4 by elexis, 8 years ago

Component: Art & AnimationMaps

Replying to niektb:

I think you can skip all these demo maps...

I would agree that it is not a requirement for authors of demo maps to add a preview image. But if we have some of them, why not include them to show the user what he can expect of that demo map.

Looks good to me, we should add it in my opinion.

in reply to:  4 comment:5 by Yordan Grigorov, 8 years ago

Replying to elexis:

Replying to niektb:

I think you can skip all these demo maps...

I would agree that it is not a requirement for authors of demo maps to add a preview image. But if we have some of them, why not include them to show the user what he can expect of that demo map.

Looks good to me, we should add it in my opinion.

Furthermore, not all of these maps are demos: Barcania, Mainland, Sahel, Survival of the Fittest, Persian Gates (not sure about Tropical Island, it looks capable though) are normal maps I think. I can't confirm right now but I will do this later when I am on a PC.

comment:6 by Yordan Grigorov, 8 years ago

Description: modified (diff)

comment:7 by elexis, 8 years ago

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 17847:

Map previews for 3 random maps and some selected demo maps. Patch by yoreei, fixes #3798.

comment:8 by elexis, 8 years ago

Keywords: review removed

Thanks

Note: See TracTickets for help on using tickets.