Code review comment for lp:~widelands-dev/widelands/irc_users

Revision history for this message
GunChleoc (gunchleoc) wrote :

I think a note is a good idea.

Don't worry about the colors - I already have a WIP branch where I have implemented fullscreen switching, so it would make more sense to implement it there to avoid merge conflicts, or wait until that branch has been merged. We could also open a bug for it.

Code LGTM, not tested.

review: Approve

« Back to merge proposal