new worker: chief-blacksmith

Bug #951382 reported by Angelo Locritani
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Expired
Wishlist
Unassigned

Bug Description

Based on discussion/suggestion provided by Astuur in http://wl.widelands.org/forum/topic/993/?page=1#post-7344
a new worker can be added: chief-blacksmith

In linked branch, this worker has been added.

Feel free to test it and give any feedback.

for Astuur and for everyone else having problems using bzr, I'm attaching modified files - unzip them in tribes folder

Revision history for this message
Angelo Locritani (alocritani) wrote :

icon for new worker is there, but worker still has master-blacksmith's walking images.

Revision history for this message
Angelo Locritani (alocritani) wrote :

updated version (fixes wrong position of icon)

Revision history for this message
Astuur (wolfsteinmetz) wrote :

Thank you, Angelo - very thoughtful action :)
I'll test it ASAP

Revision history for this message
Nasenbaer (nasenbaer) wrote :

Looks good for me and I vote for inclusion :).
However as this might again influence balance a bit, I think we should maybe wait until the release of Build17, and push it afterwards to trunk, so we have enough time to see, whether we should change anything else for balancing reasons.

Changed in widelands:
status: New → Confirmed
Revision history for this message
SirVer (sirver) wrote :

Piano, Piano. I have yet to Review the Therad in the forum, but I want to have my say in this issue. My first feeling is against a we worker - but I have yet to read the discussion. I'll speak up next week, but this should not go into b17 for sure.

Revision history for this message
SirVer (sirver) wrote :

I have written a comment into the thread in the forum. As I see this as a balance issue, I suggest to keep the discussion in the forum . Just for the record: I have strong feelings against this 'fix' and I am against it.

Changed in widelands:
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for widelands because there has been no activity for 60 days.]

Changed in widelands:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.