widelands-dev team mailing list archive

["Jens Beyer \(Qcumber-some\)" <Qcumber-some@xxxxxxxxxxxxx>]

Jens Beyer (Qcumber-some) has proposed merging lp:~widelands-dev/widelands/buildicon_playercolors into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #536230 in widelands: "building icons in menu are shown without correct playercolor"
  https://bugs.launchpad.net/widelands/+bug/536230

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/buildicon_playercolors/+merge/211184

This is not a merge-request per se, but more a request to review the code, and to find some way to create the necessary media files.

With this branch merged, the game displays the building icons in player color if available.
It uses playercolor mask files (menu_pc.png next to their menu.png files). Unfortunately, the mask files for the menu.png files are not available yet.
I added a rough playercolor image (a force-resize to the correct 30x30px) for the atlantean coal mine as proof of concept (you can check it ingame, best use a different player than player1 because of the blue player color of player1...)
The game can live without player color mask files, as you see (only the atlantean coal mine is there, the other files are missing). But now we need someone who is able to create the correct playercolor mask files where neccesary. Problem is that most animation frames are not of same width and height, but the menu icons are exactly 30x30, so simply force-resizing would probably lead to some undesired display effects.
-- 
https://code.launchpad.net/~widelands-dev/widelands/buildicon_playercolors/+merge/211184
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/buildicon_playercolors into lp:widelands.

=== modified file 'src/logic/building.cc'
--- src/logic/building.cc	2014-03-04 13:24:58 +0000
+++ src/logic/building.cc	2014-03-15 19:51:27 +0000
@@ -21,6 +21,7 @@
 
 #include <cstdio>
 #include <sstream>
+#include <fstream>
 
 #include <boost/foreach.hpp>
 
@@ -30,6 +31,7 @@
 #include "graphic/font_handler.h"
 #include "graphic/font_handler1.h"
 #include "graphic/graphic.h"
+#include "graphic/image_transformations.h"
 #include "graphic/rendertarget.h"
 #include "io/filesystem/filesystem.h"
 #include "io/filesystem/layered_filesystem.h"
@@ -62,6 +64,7 @@
 	m_tribe         (_descr),
 	m_buildable     (true),
 	m_buildicon     (nullptr),
+	m_buildiconmask     (nullptr),
 	m_size          (BaseImmovable::SMALL),
 	m_mine          (false),
 	m_port          (false),
@@ -135,7 +138,9 @@
 	if (m_buildable || m_enhanced_building) {
 		//  get build icon
 		m_buildicon_fname  = directory;
+		m_buildiconmask_fname  = directory;
 		m_buildicon_fname += "/menu.png";
+		m_buildiconmask_fname += "/menu_pc.png";
 
 		//  build animation
 		if (Section * const build_s = prof.get_section("build")) {
@@ -162,7 +167,9 @@
 	} else if (m_global) {
 		//  get build icon for global buildings (for statistics window)
 		m_buildicon_fname  = directory;
+		m_buildiconmask_fname  = directory;
 		m_buildicon_fname += "/menu.png";
+		m_buildiconmask_fname += "/menu_pc.png";
 	}
 
 	{ //  parse basic animation data
@@ -239,6 +246,13 @@
 {
 	if (m_buildicon_fname.size())
 		m_buildicon = g_gr->images().get(m_buildicon_fname);
+	if (m_buildiconmask_fname.size())
+	{
+		//TODO remove this when all menu_pc.png files are available
+		std::ifstream i(m_buildiconmask_fname);
+		if (i.good())
+			m_buildiconmask = g_gr->images().get(m_buildiconmask_fname);
+	}
 }
 
 /*

=== modified file 'src/logic/building.h'
--- src/logic/building.h	2014-02-22 18:04:02 +0000
+++ src/logic/building.h	2014-03-15 19:51:27 +0000
@@ -92,6 +92,7 @@
 	 */
 	const Buildcost & returned_wares_enhanced() const {return m_return_enhanced;}
 	const Image* get_buildicon() const {return m_buildicon;}
+	const Image* get_buildiconmask() const {return m_buildiconmask;}
 	int32_t get_size() const {return m_size;}
 	bool get_ismine() const {return m_mine;}
 	bool get_isport() const {return m_port;}
@@ -144,7 +145,9 @@
 	Buildcost     m_enhance_cost;     // cost for enhancing
 	Buildcost     m_return_enhanced;   // Returned ware for dismantling an enhanced building
 	const Image*     m_buildicon;       // if buildable: picture in the build dialog
+	const Image*     m_buildiconmask;       // if buildable: picture in the build dialog
 	std::string   m_buildicon_fname; // filename for this icon
+	std::string   m_buildiconmask_fname; // filename for this icon
 	int32_t       m_size;            // size of the building
 	bool          m_mine;
 	bool          m_port;

=== modified file 'src/wui/fieldaction.cc'
--- src/wui/fieldaction.cc	2014-03-07 19:12:24 +0000
+++ src/wui/fieldaction.cc	2014-03-15 19:51:27 +0000
@@ -23,6 +23,7 @@
 #include "economy/flag.h"
 #include "economy/road.h"
 #include "graphic/graphic.h"
+#include "graphic/image_transformations.h"
 #include "i18n.h"
 #include "logic/attackable.h"
 #include "logic/cmd_queue.h"
@@ -69,7 +70,7 @@
 	boost::signals2::signal<void (Widelands::Building_Index::value_t)> buildmouseout;
 	boost::signals2::signal<void (Widelands::Building_Index::value_t)> buildmousein;
 
-	void add(Widelands::Building_Index::value_t);
+	void add(Widelands::Building_Index::value_t, Widelands::Player *);
 
 private:
 	void clickslot(int32_t idx);
@@ -102,12 +103,24 @@
 Add a new building to the list of buildable buildings
 ===============
 */
-void BuildGrid::add(Widelands::Building_Index::value_t id)
+void BuildGrid::add(Widelands::Building_Index::value_t id, Widelands::Player * player)
 {
 	const Widelands::Building_Descr & descr =
 		*m_tribe.get_building_descr(Widelands::Building_Index(id));
+	const Image * icon = descr.get_buildicon();
+	const Image * mask = descr.get_buildiconmask();
+	const Image * pcicon;
+	if (mask == nullptr)
+	{
+		pcicon = icon;
+
+	}
+	else
+	{
+		pcicon = ImageTransformations::player_colored(player->get_playercolor(), icon, mask);
+	}
 	UI::Icon_Grid::add
-		(descr.name(), descr.get_buildicon(),
+		(descr.name(), pcicon,
 		 reinterpret_cast<void *>(id),
 		 descr.descname() + "<br><font size=11>" + _("Construction costs:") + "</font><br>" +
 			waremap_to_richtext(m_tribe, descr.buildcost()));
@@ -568,7 +581,7 @@
 		}
 
 		// Add it to the grid
-		(*ppgrid)->add(id.value());
+		(*ppgrid)->add(id.value(), m_plr);
 	}
 
 	// Add all necessary tabs

=== added file 'tribes/atlanteans/coalmine/menu_pc.png'
Binary files tribes/atlanteans/coalmine/menu_pc.png	1970-01-01 00:00:00 +0000 and tribes/atlanteans/coalmine/menu_pc.png	2014-03-15 19:51:27 +0000 differ