bugBattle for Wesnoth - Bugs: bug #5821, Moving unit_height_adjust from...

Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #5821: Moving unit_height_adjust from [terrain] to [tile]

Submitted by:  Lari Nieminen <zookeeper>
Submitted on:  Wed Apr 19 22:03:10 2006  
Category: Feature RequestSeverity: 2 - Minor
Priority: 5 - NormalItem Group: WML
Status: NonePrivacy: Public
Assigned to: Lari Nieminen <zookeeper>Open/Closed: Open
Release: trunkOperating System: all

Fri Jan 5 19:00:09 2007, comment #1:

Since my terrain branch has a letter for each keep, the most important reason for this FR no longer exists. I just discussed it with Zookeeper on IRC, but it will also be a wish to have after the merge of my terrain branch.

Mark de Wever <mordante>
Project Member
Wed Apr 19 22:03:10 2006, original submission:

Currently the unit height adjustment on a terrain has to be the same on every hex that's of the same terrain. This makes for example the height adjustment on keeps difficult, since normal castle keeps need to have much higher height than encampment keeps - but both are of the same terrain, so either one is bound to look silly. Since the amount of height adjustment should obviously be determined by the graphics on a hex, not the terrain itself, it would be logical to move the 'unit_height_adjust' key from [terrain] to [tile]. Actually, the 'submerge' key should get the same treatment, for the same reasons.

So, this is how it should work:

If a [tile] has the 'unit_height_adjust' key present, that value will be added to the value given in the [terrain] tag, but of course just for the one hex that the [tile] refers to. The same for the 'submerge' key. So both keys should still be accepted inside [terrain], both for simplicity and in case the terrain is "uniform" in height and needs either none or only few / small tile-specific adjustments.

I have the WML means to apply the different adjustments for the different keep types if this is implemented, and for example for mountains, too, if such a thing is desired.

Lari Nieminen <zookeeper>
Project AdministratorIn charge of this item.


No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by sapient (Updated the item)
  • -unavailable- added by mordante (Posted a comment)
  • -unavailable- added by zookeeper (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.


    Error: not logged in



    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri Mar 18 17:41:02 2016zookeeperAssigned tomordante=>zookeeper
    Sat Mar 3 03:03:32 2007sapientAssigned tosapient=>mordante
    Thu Aug 31 18:15:10 2006zookeeperAssigned todarthfool=>sapient
    Sat Apr 22 14:19:24 2006zookeeperAssigned toNone=>darthfool
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup