OpenTTD

Tasklist

FS#3185 - Choose town name before creation

Attached to Project: OpenTTD
Opened by Jose Soler (Terkhen) - Tuesday, 08 September 2009, 20:34 GMT
Last edited by Zdeněk Sojka (SmatZ) - Tuesday, 22 September 2009, 14:00 GMT
Type Patch
Category Interface
Status Closed
Assigned To No-one
Operating System All
Severity Low
Priority Normal
Reported Version trunk
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch implements one of SmatZ's ideas for the found a town feature. This feature allows to decide the name of a town before placing it, either by selecting random names or by writing it directly. It has been developed with the Found a town patch in mind, but it is useful for the scenario editor too. Check the attached screenshot to see the finished GUI.

This patch needs to be applied after the "Support for nested QueryStringBaseWindow" patch, as it makes FoundTownWindow a working nested QueryStringBaseWindow. You can find it here: http://bugs.openttd.org/task/3184
This task depends upon

Closed by  Zdeněk Sojka (SmatZ)
Tuesday, 22 September 2009, 14:00 GMT
Reason for closing:  Implemented
Additional comments about closing:  r17612
Comment by Jose Soler (Terkhen) - Tuesday, 08 September 2009, 22:07 GMT
New version that (should) correct a segmentation fault in non-windows systems. I have checked that the patch works correctly when there is no town names remaining using the attached GRF file (it only has four possible town names).
Comment by Jose Soler (Terkhen) - Thursday, 10 September 2009, 12:55 GMT
I have corrected some white spaces in empty lines, cleaned the code a bit and corrected a few coding style issues.
Comment by Peter T (Petert) - Monday, 14 September 2009, 00:53 GMT
I like it! I hope it's in trunk soon.
Comment by Jose Soler (Terkhen) - Monday, 14 September 2009, 10:27 GMT
New version, updated for recent changes in trunk. The patch has been compiled and tested in linux. The "Support for nested QueryStringBaseWindow" patch is now in trunk, so now you can try this patch without anything else.

Loading...