fix: tooltip opacity

nel/include/nel/gui/interface_group.h

@@ -271,7 +271,6 @@ namespace NLGUI
 
 		virtual sint32 getAlpha() const;
 		virtual void setAlpha (sint32 a);
-		virtual void setContainerAlpha (sint32 a);
 
 		/// Eval current clip coords. This is not incremental as with makeNewClip, and thus more slow. This also doesn't change the current clip window.
 		void getClip(sint32 &x, sint32 &y, sint32 &w, sint32 &h) const;

nel/src/gui/interface_group.cpp

@@ -2153,18 +2153,6 @@ namespace NLGUI
 		}
 	}
 
-	// ------------------------------------------------------------------------------------------------
-	void CInterfaceGroup::setContainerAlpha (sint32 a)
-	{
-		vector<CViewBase*>::const_iterator itv;
-		for (itv = _Views.begin(); itv != _Views.end(); itv++)
-		{
-			CViewBase *pVB = *itv;
-			nlwarning("setContainerAlpha: %s %s", pVB->getShortID().c_str(), std::to_string(pVB->getRenderLayer()).c_str());
-			pVB->setAlpha(a);
-		}
-	}
-
 	// ------------------------------------------------------------------------------------------------
 	void	CInterfaceGroup::setLeftClickHandler(const std::string &handler)
 	{

nel/src/gui/widget_manager.cpp

@@ -1780,33 +1780,40 @@ namespace NLGUI
 
 			if(groupContextHelp)
 			{	
-				groupContextHelp->setContainerAlpha(100);
 				/** If there's a modal box around, should be sure that the context help doesn't intersect it.
 				  * If this is the case, we just disable it, unless the tooltip was generated by the current modal window
 				  */
+				
+				bool draw = false;
 				if ( hasModal() )
 				{
 					CInterfaceGroup *mw = getModal().ModalWindow;
 					if (mw && mw->isIn(*groupContextHelp))
 					{
 						if (_CurCtrlContextHelp->isSonOf(mw))
-						{
+							draw = true;
-							groupContextHelp->executeLuaScriptOnDraw();
-							groupContextHelp->draw ();
-							// flush layers
-							CViewRenderer::getInstance()->flush();
-						}
 					}
 					else
-					{
+						draw = true;
-						groupContextHelp->executeLuaScriptOnDraw();
-						groupContextHelp->draw ();
-						// flush layers
-						CViewRenderer::getInstance()->flush();
-					}
 				}
 				else
+					draw = true;
+
+				if (draw) {
+					// apply opacity settings
+					const std::vector< CViewBase* > &vs = groupContextHelp->getViews();
+					// content opacity: only target views or else breaks
+					for( std::vector< CViewBase* >::const_iterator itr = vs.begin(); itr != vs.end(); ++itr )
 					{
+						CViewBase *pVB = *itr;
+						pVB->setAlpha(_ContextHelpContentAlpha);
+					}
+					// container opacity
+					CInterfaceGroup* container = dynamic_cast<CInterfaceGroup*>(groupContextHelp->findFromShortId("border"))
+					if (container)
+						container->setAlpha(_ContextHelpContainerAlpha);
+
+					// draw the tooltip
 					groupContextHelp->executeLuaScriptOnDraw();
 					groupContextHelp->draw ();
 					// flush layers