← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~widelands-dev/widelands/user-status-lobby into lp:widelands

 

Toni Förster has proposed merging lp:~widelands-dev/widelands/user-status-lobby into lp:widelands.

Commit message:
Properly formatted User Status overlay:
- centered and bold Heading
- proper aligned icons and text 

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/user-status-lobby/+merge/361704

The current overlay looked a little off. It is much nicer looking now IMHO.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/user-status-lobby into lp:widelands.
=== modified file 'src/ui_fsmenu/internet_lobby.cc'
--- src/ui_fsmenu/internet_lobby.cc	2018-12-13 07:24:01 +0000
+++ src/ui_fsmenu/internet_lobby.cc	2019-01-12 22:31:41 +0000
@@ -134,10 +134,12 @@
 
 	// prepare the lists
 	std::string t_tip =
-	   (boost::format("%s%s%s%s%s%s%s%s%s%s") % "<rt><p><font underline=yes>" % _("User Status") %
-	    "</font><br>" % "<img src=images/wui/overlays/roadb_yellow.png> " % _("Registered") %
-	    "<br><img src=images/wui/overlays/roadb_green.png> " % _("Administrator") %
-	    "<br><img src=images/wui/overlays/roadb_red.png> " % _("Unregistered") % "</p></rt>")
+	   (boost::format("%s%s%s%s%s%s%s%s%s%s")
+       % "<rt padding=2><p align=center spacing=3><font bold=yes underline=yes>"
+       % _("User Status") % "</font></p>"
+       % "<p valign=bottom><img src=images/wui/overlays/roadb_green.png> " % _("Administrator")
+       % "<br><img src=images/wui/overlays/roadb_yellow.png> " % _("Registered")
+       % "<br><img src=images/wui/overlays/roadb_red.png> " % _("Unregistered") % "</p></rt>")
 	      .str();
 	clientsonline_list_.add_column(22, "*", t_tip);
 	/** TRANSLATORS: Player Name */


Follow ups