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

2016-02-05 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/messagebox into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/messagebox/+merge/285023 -- Your team Widelands Developers is subscribed to branch lp:~

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

2016-02-05 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/messagebox/+merge/285023 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/messagebox. ___ Mailing list: https://launchpad.net/~widelands-dev Post t

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

2016-02-05 Thread SirVer
Review: Approve Add a tracking bug for the Ok/Cancel button placement? One nit, otherwise lgtm. Diff comments: > === modified file 'src/ui_basic/messagebox.cc' > --- src/ui_basic/messagebox.cc2016-01-28 05:24:34 + > +++ src/ui_basic/messagebox.cc2016-02-04 09:08:39 + >

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

2016-02-04 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/messagebox into lp:widelands. Commit message: Refactored WLMessageBox. It now no longer depends on the old font renderer, and scrollbars only appear when they are needed. Requested reviews: Widelands Developers (widelands-dev) For mo