Added: New way to add icons in Contextual Menus

code/nel/include/nel/gui/group_menu.h

@@ -31,7 +31,7 @@ namespace NLGUI
 	class CViewBitmap;
 	class CGroupList;
 	class CGroupMenu;
-
+	class CGroupSubMenu;
 
 	/**
 	 * CViewTextMenu is an element of a sub menu
@@ -50,6 +50,7 @@ namespace NLGUI
 			_Checked = false;
 			_Checkable = false;
 			_CheckBox = NULL;
+			_ParentMenu = NULL;
 			Over = false;
 		}
 
@@ -60,6 +61,8 @@ namespace NLGUI
 		bool getCheckable() const { return _Checkable; }
 		void setCheckable(bool c);
 		void setCheckBox(CViewBitmap *checkBox) { _CheckBox = checkBox; }
+		void setParentMenu(CGroupSubMenu *parentMenu) { _ParentMenu = parentMenu; };
+		void setActive (bool g);
 		CViewBitmap * getCheckBox() const { return _CheckBox; }
 		bool getFormatted () const { return getMultiLine (); }
 
@@ -86,6 +89,7 @@ namespace NLGUI
 		bool		_Grayed;
 		bool		_Checked;
 		bool		_Checkable;
+		CGroupSubMenu *_ParentMenu;
 	};
 
 	/**

code/nel/src/gui/group_menu.cpp

@@ -114,6 +114,19 @@ namespace NLGUI
 		OldShadowColorGrayed.A = OldColorGrayed.A = (uint8)a;
 	}
 
+	// ------------------------------------------------------------------------------------------------
+	void CViewTextMenu::setActive (bool state)
+	{
+		if (_ParentMenu)
+			_ParentMenu->setActive(state);
+
+		if (_Active != state)
+		{
+			_Active = state;
+			invalidateCoords();
+		}
+	}
+
 	// ------------------------------------------------------------------------------------------------
 	// CGroupSubMenu
 	// ------------------------------------------------------------------------------------------------
@@ -290,6 +303,9 @@ namespace NLGUI
 				if (cond)	strCond = (const char*)cond;
 				CXMLAutoPtr params((const char*) xmlGetProp (cur,  (xmlChar*)"params"));
 				if (params)	strParams = (const char*)params;
+				CXMLAutoPtr icon((const char*) xmlGetProp (cur,  (xmlChar*)"icon"));
+				if (icon)
+					strTexture = (const char*)icon;
 				CXMLAutoPtr strCheckable((const char*) xmlGetProp (cur,  (xmlChar*)"checkable"));
 				bool bCheckable = false;
 				if (strCheckable)	bCheckable = convertBool (strCheckable);
@@ -393,11 +409,11 @@ namespace NLGUI
 		pVB->setSerializable( false );
 		pVB->setParent (this);
 		pVB->setParentPos (parentPos);
-		pVB->setParentPosRef (Hotspot_ML);
-		pVB->setPosRef (Hotspot_MR);
+		pVB->setParentPosRef (Hotspot_BL);
+		pVB->setPosRef (Hotspot_BR);
 		pVB->setTexture(texture);
 		pVB->setModulateGlobalColor(false);
-		pVB->setX (-2);
+		pVB->setX (MENU_WIDGET_X);
 		addView (pVB);
 		return pVB;
 	}
@@ -639,12 +655,14 @@ namespace NLGUI
 
 		// *** Update Text Positions
 	//	sint32 currX = 0;
+		sint32 maxTextW = 0;
 		for(k = 0; k < _Lines.size(); ++k)
 		{
 			if (_Lines[k].ViewText)
 			{
 				// Setup Y
 				_Lines[k].ViewText->setY(_Lines[k].TextDY);
+				maxTextW = max(maxTextW, _Lines[k].ViewText->getW());
 			}
 		}
 
@@ -658,10 +676,12 @@ namespace NLGUI
 
 		// *** Setup SubMenus and CheckBoxes Positions
 		sint32 maxViewW = 0;
+
 		for (i = 1; i < _Views.size(); ++i)
 		{
 			CViewBitmap *pVB = dynamic_cast<CViewBitmap *>(_Views[i]);
 			if (pVB == NULL) continue;
+
 			if (pVB->getId() == ID_MENU_SUBMENU)
 			{
 				// Look for the next line of the menu that contains a sub menu
@@ -1250,6 +1270,7 @@ namespace NLGUI
 		_GroupList->addChild (pV);
 
 		CViewBitmap *checkBox = NULL;
+		CViewBitmap *icon = NULL;
 
 		if (checkable)
 		{
@@ -1258,15 +1279,6 @@ namespace NLGUI
 			pV->setCheckBox(checkBox);
 		}
 
-		CViewBitmap *icon = NULL;
-		if (!texture.empty())
-		{	
-			if (_GroupList->getNumChildren() == 1)
-				pV->setX(20);
-			icon = createIcon(pV, texture);
-		}
-		
-
 		tmp.ViewText = pV;
 		tmp.Separator = NULL;
 		tmp.AHName = ah;
@@ -1280,12 +1292,39 @@ namespace NLGUI
 			tmp.Id = id;
 
 		pV->setId(_GroupMenu->getId()+":"+tmp.Id);
+		
+		{
+			typedef std::pair<std::string, std::string> TTmplParams;
+			std::vector<TTmplParams> vparams;
+			uint lineIndex = _Lines.size()-1;
+			vparams.push_back(TTmplParams("id", toString("icon%d", lineIndex)));
+			vparams.push_back(TTmplParams("sizeref", ""));
+			vparams.push_back(TTmplParams("icon_texture", texture.c_str()));
+			//vparams.push_back(TTmplParams("icon_color", options.ColorNormal.toString()));
+			string lineId = toString("%s:icon%d", _GroupMenu->getId().c_str(), lineIndex);
+			
+			CInterfaceGroup *pUGLeft = CWidgetManager::getInstance()->getParser()->createGroupInstance("menu_row_icon", lineId, vparams);
+			if (pUGLeft)
+			{
+				tmp.UserGroupLeft = pUGLeft;
+				tmp.UserGroupLeftOwnership = true;
+				addGroup(pUGLeft);
+				pUGLeft->setParent(this);
+				pUGLeft->setParentPos(this);
+				pUGLeft->setParentPosRef (Hotspot_BL);
+				pUGLeft->setPosRef (Hotspot_BL);
+				pUGLeft->setX(LEFT_MENU_WIDGET_X);
+			}
+		}
+
 
 		_Lines.push_back (tmp);
 
+
 		// Add an empty sub menu by default
 		_SubMenus.push_back (NULL);
 
+
 		_GroupMenu->invalidateCoords();
 
 		return pV;