← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/document_worker_program into lp:widelands

 

Did not read completely, just stumbled over 'Common Worker Properties':

I think this part is explained vice versa as other parts...

"Workers are defined with Lua functions called new_worker_type{table}. The contents of table depend on the type of worker that you are defining. The common properties shared by all workers are:"

Shouldn't be the function name (new_worker_type{table}) a header? So backlinks to this make more sense, imho. E.g. in the worker Programs:

"Worker programs are defined in the programs subtable specified in calls to tribes:new_worker_type. Each worker program..."

Currently the link to tribes:new_worker_type points to "Common Worker Properties" and one has to read the text to find the functions name.

Formatting looks good so far (testing with local website) :-)
-- 
https://code.launchpad.net/~widelands-dev/widelands/document_worker_program/+merge/329696
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/document_worker_program.


References