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:~
@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
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 +
>
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
4 matches
Mail list logo