From b046413723f32b46dbd36dc231d7ddfbf259f4ec Mon Sep 17 00:00:00 2001 From: Tonya Souther Date: Sun, 16 Feb 2014 13:24:10 -0600 Subject: [PATCH] FIRE-11911: Disable synchronize materials if planar mapping is enabled, since that breaks materials settings in weird ways. --- indra/newview/llpanelface.cpp | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/indra/newview/llpanelface.cpp b/indra/newview/llpanelface.cpp index 1a48e22892..efc524101c 100755 --- a/indra/newview/llpanelface.cpp +++ b/indra/newview/llpanelface.cpp @@ -1136,7 +1136,10 @@ void LLPanelFace::updateUI() mCtrlBumpyScaleU->setTentative(LLSD(norm_scale_tentative)); // FIRE-11407 - Materials alignment - getChildView("checkbox maps sync")->setEnabled(editable && (specmap_id.notNull() || normmap_id.notNull())); + // FIRE-11911 - Synchronize materials doens't work with planar textures + // Disable the checkbox if planar textures are in use + getChildView("checkbox maps sync")->setEnabled(editable && (specmap_id.notNull() || normmap_id.notNull()) && !align_planar); + // FIRE-11911 // } @@ -2637,6 +2640,14 @@ void LLPanelFace::onClickMapsSync(LLUICtrl* ctrl, void *userdata) //static void LLPanelFace::alignMaterialsProperties(LLPanelFace* self) { + // FIRE-11911: Synchronize materials doesn't work with planar textures + // Don't even try to do the alignment if we wind up here and planar is enabled. + if ((bool)self->childGetValue("checkbox planar align").asBoolean()) + { + return; + } + // FIRE-11911 + //LLPanelFace *self = (LLPanelFace*) userdata; llassert_always(self);