Update of patch #4936 (project freeciv):
Planned Release: 2.5.0, 2.6.0, 3..0.0 => 2.5.0, 2.6.0, 3.0.0
_______________________________________________________
Follow-up Comment #8:
Attached a new version with a stab at government help.
It incorporates and munges bardo's patch #5691.
I've tried to add text indicating particular strengths and weaknesses, drawing
on bardo's descriptions of his intentions in patch #5592.
I'm still a bit displeased with the result, especially the redundancy with
autogenerated help. But it'll do.
The patch also tweaks some other items, and addresses some (but not all) of
the comments so far. Not yet a commit candidate
Tweaks:
> I) README
> In patch #5764 I introduced some changes to the city defense
> table that seems to collide with this patch (sorry)
Yes, I saw those. Not reconciled yet, still to do.
> - You fixed some capital letters, but I'm afraid it'd require
> some more fixes in order to get an uniform use of capital
> letters.
That's OK, I'm content to fix the issues I notice and leave some in.
> I tried to use them for key words that have an special meaning
> in the game, but I'm not sure about the expected use in
> freeciv help texts.
We're not entirely consistent, but tend to use them as you say.
> 2) Units
> I think the FIXME in the transport is no longer needed.
> We fixed it so sea units can not perform other
> transformations in land.
Transports can still clean pollution in coastal cities, and faster than
Engineers.
I've updated the FIXME comment, but I don't think there's anything we can do
about it in S2_5.
> 3) Terrains
> About the fixme in Fortress and Airbase:
Deleted.
Other tweaks:
* In Marco Polo, noted the benefit to research if you're behind.
* Comments in effects.ruleset from the notes I made while doing this.
* Some minor rewordings.
(file #23773)
_______________________________________________________
Additional Item Attachment:
File name: S2_5-civ2civ3-doc-wip-bis.patch Size:83 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4936>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
