From 028f6cdd3e32d23e1412c8c77a8444057284587c Mon Sep 17 00:00:00 2001 From: Nimetu Date: Sat, 18 Sep 2021 17:26:53 +0300 Subject: [PATCH 1/3] Add -ryzom-modulate-text-color to separate text from background, ryzom/ryzomcore#644 --- nel/include/nel/gui/css_style.h | 3 +++ nel/src/gui/css_style.cpp | 10 ++++++++++ nel/src/gui/group_html.cpp | 12 +++++++++--- 3 files changed, 22 insertions(+), 3 deletions(-) diff --git a/nel/include/nel/gui/css_style.h b/nel/include/nel/gui/css_style.h index 042486d17..674f0268e 100644 --- a/nel/include/nel/gui/css_style.h +++ b/nel/include/nel/gui/css_style.h @@ -63,6 +63,7 @@ namespace NLGUI Underlined=false; StrikeThrough=false; GlobalColor=false; + GlobalColorText=false; DisplayBlock=false; Width=-1; Height=-1; @@ -93,6 +94,7 @@ namespace NLGUI NLMISC::CRGBA TextColor; STextShadow TextShadow; bool GlobalColor; + bool GlobalColorText; bool Underlined; bool StrikeThrough; bool DisplayBlock; @@ -215,6 +217,7 @@ namespace NLGUI _StyleStack.push_back(Current); Current.GlobalColor = false; + // inherit GlobalColorText Current.DisplayBlock = false; Current.Width=-1; Current.Height=-1; diff --git a/nel/src/gui/css_style.cpp b/nel/src/gui/css_style.cpp index 912ff04d4..567ffe42a 100644 --- a/nel/src/gui/css_style.cpp +++ b/nel/src/gui/css_style.cpp @@ -936,6 +936,16 @@ namespace NLGUI style.GlobalColor = b; } else + if (it->first == "-ryzom-modulate-text-color") + { + bool b; + if (it->second == "inherit") + style.GlobalColorText = current.GlobalColorText; + else + if (fromString(it->second, b)) + style.GlobalColorText = b; + } + else if (it->first == "background-color") { if (it->second == "inherit") diff --git a/nel/src/gui/group_html.cpp b/nel/src/gui/group_html.cpp index 1dd28d473..0cb32f25e 100644 --- a/nel/src/gui/group_html.cpp +++ b/nel/src/gui/group_html.cpp @@ -2551,7 +2551,7 @@ namespace NLGUI (embolden == text->getEmbolden()) && (style.FontOblique == text->getOblique()) && (getLink() == text->Link) && - (style.GlobalColor == text->getModulateGlobalColor()); + (style.GlobalColorText == text->getModulateGlobalColor()); } // *************************************************************************** @@ -2588,7 +2588,10 @@ namespace NLGUI nlinfo("Text button template '%s' is missing :button or :b text element", tpl.c_str()); return; } - ctrlButton->setModulateGlobalColorAll(false); + ctrlButton->setModulateGlobalColorAll(_Style.Current.GlobalColor); + ctrlButton->setTextModulateGlobalColorNormal(_Style.Current.GlobalColorText); + ctrlButton->setTextModulateGlobalColorOver(_Style.Current.GlobalColorText); + ctrlButton->setTextModulateGlobalColorPushed(_Style.Current.GlobalColorText); // Translate the tooltip ctrlButton->setText(text); @@ -2619,7 +2622,7 @@ namespace NLGUI newLink->setText(text); newLink->setMultiLineSpace((uint)((float)(_Style.Current.FontSize)*LineSpaceFontFactor)); newLink->setMultiLine(true); - newLink->setModulateGlobalColor(_Style.Current.GlobalColor); + newLink->setModulateGlobalColor(_Style.Current.GlobalColorText); setTextStyle(newLink, _Style.Current); registerAnchor(newLink); @@ -5247,6 +5250,9 @@ namespace NLGUI if (ctrlButton) { ctrlButton->setModulateGlobalColorAll (_Style.Current.GlobalColor); + ctrlButton->setTextModulateGlobalColorNormal(_Style.Current.GlobalColorText); + ctrlButton->setTextModulateGlobalColorOver(_Style.Current.GlobalColorText); + ctrlButton->setTextModulateGlobalColorPushed(_Style.Current.GlobalColorText); // Translate the tooltip if (!tooltip.empty()) From 5c53148907435dd301c44f99ea332d20f738bd7a Mon Sep 17 00:00:00 2001 From: Nimetu Date: Sat, 18 Sep 2021 17:30:25 +0300 Subject: [PATCH 2/3] Fix downloaded background images not showing --- nel/src/gui/group_html.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nel/src/gui/group_html.cpp b/nel/src/gui/group_html.cpp index 0cb32f25e..5d7a66eed 100644 --- a/nel/src/gui/group_html.cpp +++ b/nel/src/gui/group_html.cpp @@ -350,7 +350,7 @@ namespace NLGUI void CGroupHTML::TextureDownloadCB::finish() { // tmpdest file does not exist if download skipped (ie cache was used) - if (CFile::fileExists(tmpdest) && CFile::getFileSize(tmpdest) == 0) + if (CFile::fileExists(tmpdest) && CFile::getFileSize(tmpdest) > 0) { if (CFile::fileExists(dest)) CFile::deleteFile(dest); From fc6ecc7dbd62723463410f9c9199b2f8ca6a35e7 Mon Sep 17 00:00:00 2001 From: Nimetu Date: Sat, 18 Sep 2021 17:40:41 +0300 Subject: [PATCH 3/3] Fix html,body background not using container opacity --- nel/src/gui/group_html.cpp | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/nel/src/gui/group_html.cpp b/nel/src/gui/group_html.cpp index 5d7a66eed..021a3691a 100644 --- a/nel/src/gui/group_html.cpp +++ b/nel/src/gui/group_html.cpp @@ -4120,6 +4120,22 @@ namespace NLGUI void CGroupHTML::draw () { + uint8 CurrentAlpha = 255; + // search a parent container + CInterfaceGroup *gr = getParent(); + while (gr) + { + if (gr->isGroupContainer()) + { + CGroupContainer *gc = static_cast(gr); + CurrentAlpha = gc->getCurrentContainerAlpha(); + break; + } + gr = gr->getParent(); + } + m_HtmlBackground.CurrentAlpha = CurrentAlpha; + m_BodyBackground.CurrentAlpha = CurrentAlpha; + m_HtmlBackground.draw(); m_BodyBackground.draw(); CGroupScrollText::draw ();