See my inline comment.
Diff comments:
> === modified file 'src/logic/map_objects/tribes/dismantlesite.cc'
> --- src/logic/map_objects/tribes/dismantlesite.cc 2016-02-18 18:27:52
> +0000
> +++ src/logic/map_objects/tribes/dismantlesite.cc 2016-03-13 09:28:07
> +0000
> @@ -93,7 +94,8 @@
> void DismantleSite::update_statistics_string(std::string* s)
> {
> unsigned int percent = (get_built_per64k() * 100) >> 16;
> - *s = (boost::format(_("%u%% dismantled")) % percent).str();
> + *s = (boost::format("<font color=%s>%s</font>") %
> UI_FONT_CLR_DARK.hex_value() %
> + (boost::format(_("%u%% dismantled")) %
> percent).str()).str();
Uhm, should this not be like:
boost::format("<font color=\"%s\">_(%u%% dismantled)</font>")) %
UI_FONT_CLR_DARK.hex_value() % percent").str();
Or will this clash with the translation of _(...)?
Im not not sure about this boost::format but this ".str()).str()" is strange
duplicate?
As I am kind of colorblind I will not comment on the color actally produced :-)
> }
>
> /*
--
https://code.launchpad.net/~widelands-dev/widelands/dismantlesite-color/+merge/288866
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/dismantlesite-color into lp:widelands.
_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help : https://help.launchpad.net/ListHelp
