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

2017-10-09 Thread SirVer
What is the status of this branch? Is it ready for reviewing? -- https://code.launchpad.net/~widelands-dev/widelands/reveal_hide_animations/+merge/327062 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1687100-reveal_fields.

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

2017-10-09 Thread kaputtnik
Gun did the review already, but reviewing by a another person can't be wrong. Before merging this branch the prerequisite branch needs reviewing and merged: https://code.launchpad.net/~widelands-dev/widelands/bug-1687100-reveal_fields -- https://code.launchpad.net/~widelands-dev/widelands/reveal

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

2017-10-09 Thread GunChleoc
Given that we're short-handed on code reviewers, I'd rather you just reviewed the other branch: https://code.launchpad.net/~widelands-dev/widelands/bug-1687100-reveal_fields/+merge/323721 It's a prerequisite for this branch, which is providing the datadir for the other branch. -- https://code.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/searching_with_haystack_whoosh into lp:widelands-website

2017-10-09 Thread GunChleoc
Added some comments and proofreading. We should definitely do some cross-browse testing on he alpha site. Diff comments: > > === modified file 'templates/navigation.html' > --- templates/navigation.html 2017-09-23 08:52:36 + > +++ templates/navigation.html 2017-10-05 21:59:22 + > @@ -59

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/translation_stats into lp:widelands

2017-10-09 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/translation_stats into lp:widelands. Commit message: Show translation stats next to the language selection menu and invite translators if a translation is incomplete, with the help of the Translate Toolkit. - Added a new utils script "

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/searching_with_haystack_whoosh into lp:widelands-website

2017-10-09 Thread kaputtnik
Thanks for proofreading :-) Searching the forum is default. > Should we also search the map hints? Maybe. I am quite unsure... we could add the map hints later. I am also not very convinced by the default highlighter included in haystack. It seems to be a bit buggy but i didn't want to write a

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/translation_stats into lp:widelands

2017-10-09 Thread bunnybot
Continuous integration builds have changed state: Travis build 2702. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/285758712. Appveyor build 2517. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_translatio