patchFreeciv - Patches: patch #5027, Fort and Airstrip graphics for...

 
 
Show feedback again

patch #5027: Fort and Airstrip graphics for Amplio2/Cimpletoon/Hexemplio/Toonhex

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Sat 09 Aug 2014 10:11:14 AM UTC  
 
Category: artPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Open/Closed: Closed
Planned Release: 2.5.0, 2.6.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Thu 11 Dec 2014 09:01:19 PM UTC, SVN revision 27258:

Added Fort and Airstrip graphics to Amplio2.

Art by GriffonSpade

See patch #5027

(Browse SVN revision 27258)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Thu 11 Dec 2014 09:01:09 PM UTC, SVN revision 27257:

Added Fort and Airstrip graphics to Amplio2.

Art by GriffonSpade

See patch #5027

(Browse SVN revision 27257)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Wed 10 Dec 2014 08:48:54 AM UTC, comment #13:

>I think this is wanted to S2_5 for supporting custom ruleset, experimental edition of the civ2civ3.

Right, thank you.

David Fernandez <bardo>
Tue 09 Dec 2014 06:39:36 PM UTC, comment #12:

- Changed "trench" tag to "fort" tag as per current version of patch #5525
- Include also S2_5 version of the patch. .png is identical between the branches. I think this is wanted to S2_5 for supporting custom ruleset, experimental edition of the civ2civ3.

(file #23123, file #23124)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue 09 Dec 2014 01:43:14 AM UTC, comment #11:

Attaching my version for Amplio2, keeping Fortress and Airbase as they are but adding Airstrip and Trench.

(file #23115, file #23116)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue 09 Dec 2014 01:13:05 AM UTC, comment #10:

Renaming Airfield to Airstrip -> patch #5524.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Mon 08 Dec 2014 05:10:35 AM UTC, comment #9:

>So, do you have something to share as solution to this ticket?

I shared in the forums a moddification of Amplio tileset where I included the new base levels made by GriffonSpade:
http://forum.freeciv.org/f/viewtopic.php?f=12&t=450

In this pack, the old fortress and airbase keep the same name, and the new ones use the names fortress1 and airbase1.
The other 2 levels use the name Fort and Fort1 (instead of Trench). I'd suggest to change the name in the ruleset too.

I can create a patch for Amplio2 if you are interested. But I suppose it is better to wait for GriffonSpade opinion, in case he wants to create his own patch.

David Fernandez <bardo>
Sun 07 Dec 2014 07:10:33 PM UTC, comment #8:

> My impressions after testing them a bit (by manually pasting
> them to amplio2)


So, do you have something to share as solution to this ticket?

> I think the 4th and 8th would be perfect for civ2civ3, keeping
> the default bases as pre-fortresses.


By "default bases" you mean existing ones? Maybe we then should rename them as Trench and Airfield, and add new Fortress and Airbase.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue 21 Oct 2014 11:57:20 PM UTC, comment #7:

My impressions after testing them a bit (by manually pasting them to amplio2):

- I'm liking how the short wall looks in all terrains (5th from left to right).
- When I test walls with towers both in front and rear (7th), they overlap and it looks odd.
- I like the standard wall with no towers (6th), and the 3 towers in front with no tower at back (8th).

- The dirt airbase (1st) looks good over plain terrains, but it looks weird over hills/moutains or rivers/forests.
- The tarmac airbase with one single runway (2th) looks out of place in most terrains.
- I'm liking a lot the airbase with tower (4th).

I think the 4th and 8th would be perfect for civ2civ3, keeping the default bases as pre-fortresses.

David Fernandez <bardo>
Sun 05 Oct 2014 02:37:22 PM UTC, comment #6:

Some candidate gfx from GriffonSpade on the forum.

I think something like some of these designs will do, but haven't tested e.g. the brown airstrip for visibility against all terrains.

Jacob Nevins <jtn>
Project Administrator
Sun 10 Aug 2014 02:57:24 PM UTC, comment #5:

The english name that you think fits better is ok to me. In spanish I'd vote for "aerodromo".

About the Trench, in game it acts more as an improvised fortification than as actual trenchs. I like the idea of sandbag-line, or breastwork. But it does not protect from killstack, so I think the line/wall should not be completely closed.

David Fernandez <bardo>
Sat 09 Aug 2014 08:30:04 PM UTC, comment #4:

Either Trench should be made "hidden_by" Fortress in the ruleset, or the fact that they are visible simultaneously should be considered in gfx design.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sat 09 Aug 2014 05:37:14 PM UTC, comment #3:

> I'm afraid I have no art skills, but I like to imagine the
> trench as a squared moat around the fortress.

Mm, maybe a sandbag-lined thing?

What do you think of renaming "Airfield" to "Airstrip" in trunk civ2civ3 (in another ticket)? I think that better captures its temporary / hastily constructed nature. (We can keep the "Airfield" rule_name for compatibility with any old savegames, although we don't have to.)

Jacob Nevins <jtn>
Project Administrator
Sat 09 Aug 2014 03:23:54 PM UTC, comment #2:

>Trench is precursor to Fortress, so ideally it'd look like something you can build Fortress on top of. Not sure what exactly.

I'm afraid I have no art skills, but I like to imagine the trench as a squared moat around the fortress.
Trench was the name used in longturn games, but I suppose it could be changed to something easier to represent as gfx.

David Fernandez <bardo>
Sat 09 Aug 2014 10:13:07 AM UTC, comment #1:

Trench is precursor to Fortress, so ideally it'd look like something you can build Fortress on top of. Not sure what exactly.

Airfield is precursor to Airbase. I guess it could look like a grass airstrip as compared to Airbase's concrete/tarmac?

Jacob Nevins <jtn>
Project Administrator
Sat 09 Aug 2014 10:11:14 AM UTC, original submission:

civ2civ3 added Trench/Airfield "pre-bases" in patch #4608, so we should have graphics for them.

(If we do get such graphics, we should add them to tilesets on stable branches too, since such "pre-bases" are in wide use in Longturn-style games.)

Jacob Nevins <jtn>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #23116:  moderncities.png added by cazfi (158kB - image/png)

 

Depends on the following items: None found

Digest:
   patch dependencies.

 

Carbon-Copy List
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by bardo (Posted a comment)
  • -unavailable- added by jtn (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 11 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 16 Dec 2014 03:32:06 AM UTCcazfiStatusReady For Test=>Done
      Assigned toNone=>cazfi
      Open/ClosedOpen=>Closed
      SummaryTrench and Airfield graphics=>Fort and Airstrip graphics for Amplio2/Cimpletoon/Hexemplio/Toonhex
    Tue 09 Dec 2014 06:39:36 PM UTCcazfiAttached File-=>Added TrenchAirstripAmplio2-S2_5-2.patch, #23124
      Planned Release2.6.0=>2.5.0, 2.6.0
    Tue 09 Dec 2014 06:39:35 PM UTCcazfiAttached File-=>Added TrenchAirstripAmplio2-2.patch, #23123
    Tue 09 Dec 2014 01:43:14 AM UTCcazfiAttached File-=>Added TrenchAirstripAmplio2.patch, #23115
      Attached File-=>Added moderncities.png, #23116
      StatusNone=>Ready For Test
    Sat 09 Aug 2014 10:11:26 AM UTCjtnDependencies-=>patch #4690 is dependent
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup