On Sat, 5 Jan 2013 04:13:27 PM Michael Vehrs wrote:
>>[`Show Productivity' is coherent?]
> It's not, really. I think it should show the production due to the unit
> itself, and that due to the building, but the two should be more clearly
> distinguishable.

Agreed.  `Show Productivity' now shows the unit contribution, with the total 
described as "Unit Result" rather than "Final Result".  Then below any further 
building-level modifiers to come are listed, but no result shown.  It might be 
nice to add a `Building level modifiers' heading above that, but I am reluctant 
to make additions to the messages file when a release needs to happen.  We 
already have people filing bugs about two month old messages that remain 
untranslated.

I also added the unit production (as goods-icon-with-number) to the 
BuildingToolTip.  The layout is not totally borked as it already allowed for a 
student-of-this-teacher icon to be present, but as usual I am clumsy with 
MigLayout and may have done this stupidly.  

The BuildingToolTIp also includes a listing of the building modifiers, but only 
in debug mode as a developer convenience.

>>[Add backward compatibility mods]
> I really don't think it is worth the trouble, but there is nothing to
> stop you.

The sooner 0.10.7 is out, the sooner this is fixed, so I am happy to just put a 
prominent message in the release notes.

Cheers,
Mike Pope

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122912
_______________________________________________
Freecol-developers mailing list
Freecol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to