Skip to content

Commit

Permalink
Fix a bunch of XUI errors (#2347)
Browse files Browse the repository at this point in the history
* Fix a bunch of XUI errors

* Change TRUE to true
  • Loading branch information
Ansariel committed Aug 19, 2024
1 parent 862a467 commit 169599f
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 66 deletions.
5 changes: 0 additions & 5 deletions indra/newview/llfloaterpreference.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1237,11 +1237,6 @@ void LLFloaterPreference::buildPopupLists()

void LLFloaterPreference::refreshEnabledState()
{
LLCheckBoxCtrl* ctrl_pbr = getChild<LLCheckBoxCtrl>("UsePBRShaders");

//PBR
ctrl_pbr->setEnabled(true);

// Cannot have floater active until caps have been received
getChild<LLButton>("default_creation_permissions")->setEnabled(LLStartUp::getStartupState() >= STATE_STARTED);

Expand Down
57 changes: 0 additions & 57 deletions indra/newview/llfloaterpreferencesgraphicsadvanced.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,6 @@ void LLFloaterPreferenceGraphicsAdvanced::refresh()
updateSliderText(getChild<LLSliderCtrl>("TerrainMeshDetail", true), getChild<LLTextBox>("TerrainMeshDetailText", true));
updateSliderText(getChild<LLSliderCtrl>("RenderPostProcess", true), getChild<LLTextBox>("PostProcessText", true));
updateSliderText(getChild<LLSliderCtrl>("SkyMeshDetail", true), getChild<LLTextBox>("SkyMeshDetailText", true));
updateSliderText(getChild<LLSliderCtrl>("TerrainDetail", true), getChild<LLTextBox>("TerrainDetailText", true));
LLAvatarComplexityControls::setIndirectControls();
setMaxNonImpostorsText(
gSavedSettings.getU32("RenderAvatarMaxNonImpostors"),
Expand Down Expand Up @@ -268,10 +267,6 @@ void LLFloaterPreferenceGraphicsAdvanced::setMaxNonImpostorsText(U32 value, LLTe

void LLFloaterPreferenceGraphicsAdvanced::disableUnavailableSettings()
{
LLComboBox* ctrl_reflections = getChild<LLComboBox>("Reflections");
LLTextBox* reflections_text = getChild<LLTextBox>("ReflectionsText");
LLCheckBoxCtrl* ctrl_wind_light = getChild<LLCheckBoxCtrl>("WindLightUseAtmosShaders");
LLCheckBoxCtrl* ctrl_deferred = getChild<LLCheckBoxCtrl>("UseLightShaders");
LLComboBox* ctrl_shadows = getChild<LLComboBox>("ShadowDetail");
LLTextBox* shadows_text = getChild<LLTextBox>("RenderShadowDetailText");
LLCheckBoxCtrl* ctrl_ssao = getChild<LLCheckBoxCtrl>("UseSSAO");
Expand All @@ -282,9 +277,6 @@ void LLFloaterPreferenceGraphicsAdvanced::disableUnavailableSettings()
// disabled windlight
if (!LLFeatureManager::getInstance()->isFeatureAvailable("WindLightUseAtmosShaders"))
{
ctrl_wind_light->setEnabled(false);
ctrl_wind_light->setValue(false);

sky->setEnabled(false);
sky_text->setEnabled(false);

Expand All @@ -298,9 +290,6 @@ void LLFloaterPreferenceGraphicsAdvanced::disableUnavailableSettings()

ctrl_dof->setEnabled(false);
ctrl_dof->setValue(false);

ctrl_deferred->setEnabled(false);
ctrl_deferred->setValue(false);
}

// disabled deferred
Expand All @@ -315,9 +304,6 @@ void LLFloaterPreferenceGraphicsAdvanced::disableUnavailableSettings()

ctrl_dof->setEnabled(false);
ctrl_dof->setValue(false);

ctrl_deferred->setEnabled(false);
ctrl_deferred->setValue(false);
}

// disabled deferred SSAO
Expand All @@ -338,51 +324,13 @@ void LLFloaterPreferenceGraphicsAdvanced::disableUnavailableSettings()

void LLFloaterPreferenceGraphicsAdvanced::refreshEnabledState()
{
LLComboBox* ctrl_reflections = getChild<LLComboBox>("Reflections");
LLTextBox* reflections_text = getChild<LLTextBox>("ReflectionsText");

// Reflections
bool reflections = LLCubeMap::sUseCubeMaps;
ctrl_reflections->setEnabled(reflections);
reflections_text->setEnabled(reflections);

// Bump & Shiny
LLCheckBoxCtrl* bumpshiny_ctrl = getChild<LLCheckBoxCtrl>("BumpShiny");
bool bumpshiny = LLCubeMap::sUseCubeMaps && LLFeatureManager::getInstance()->isFeatureAvailable("RenderObjectBump");
bumpshiny_ctrl->setEnabled(bumpshiny);

// Vertex Shaders, Global Shader Enable
// SL-12594 Basic shaders are always enabled. DJH TODO clean up now-orphaned state handling code
LLSliderCtrl* terrain_detail = getChild<LLSliderCtrl>("TerrainDetail"); // can be linked with control var
LLTextBox* terrain_text = getChild<LLTextBox>("TerrainDetailText");

terrain_detail->setEnabled(false);
terrain_text->setEnabled(false);

// WindLight
//LLCheckBoxCtrl* ctrl_wind_light = getChild<LLCheckBoxCtrl>("WindLightUseAtmosShaders");
//ctrl_wind_light->setEnabled(true);
LLSliderCtrl* sky = getChild<LLSliderCtrl>("SkyMeshDetail");
LLTextBox* sky_text = getChild<LLTextBox>("SkyMeshDetailText");
sky->setEnabled(true);
sky_text->setEnabled(true);

bool enabled = true;
#if 0 // deferred always on now
//Deferred/SSAO/Shadows
LLCheckBoxCtrl* ctrl_deferred = getChild<LLCheckBoxCtrl>("UseLightShaders");

enabled = LLFeatureManager::getInstance()->isFeatureAvailable("RenderDeferred") &&
bumpshiny_ctrl && bumpshiny_ctrl->get() &&
ctrl_wind_light->get();

ctrl_deferred->setEnabled(enabled);
#endif

LLCheckBoxCtrl* ctrl_pbr = getChild<LLCheckBoxCtrl>("UsePBRShaders");

//PBR
ctrl_pbr->setEnabled(true);

LLCheckBoxCtrl* ctrl_ssao = getChild<LLCheckBoxCtrl>("UseSSAO");
LLCheckBoxCtrl* ctrl_dof = getChild<LLCheckBoxCtrl>("UseDoF");
Expand Down Expand Up @@ -414,11 +362,6 @@ void LLFloaterPreferenceGraphicsAdvanced::refreshEnabledState()
getChildView("texture compression")->setEnabled(false);
}

// if no windlight shaders, turn off nighttime brightness, gamma, and fog distance
LLUICtrl* gamma_ctrl = getChild<LLUICtrl>("gamma");
gamma_ctrl->setEnabled(!gPipeline.canUseWindLightShaders());
getChildView("(brightness, lower is brighter)")->setEnabled(!gPipeline.canUseWindLightShaders());
getChildView("fog")->setEnabled(!gPipeline.canUseWindLightShaders());
getChildView("antialiasing restart")->setVisible(!LLFeatureManager::getInstance()->isFeatureAvailable("RenderDeferred"));

// now turn off any features that are unavailable
Expand Down
2 changes: 1 addition & 1 deletion indra/newview/llfloatersidepanelcontainer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ LLFloaterSidePanelContainer::~LLFloaterSidePanelContainer()
bool LLFloaterSidePanelContainer::postBuild()
{
mMainPanel = getChild<LLPanel>(sMainPanelName);
return TRUE;
return true;
}

void LLFloaterSidePanelContainer::onOpen(const LLSD& key)
Expand Down
2 changes: 2 additions & 0 deletions indra/newview/skins/default/xui/en/floater_fast_timers.xml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,9 @@
name="scroll_vert"
orientation="vertical"
step_size="16"
doc_pos="0"
doc_size="3000"
page_size="0"
/>
</layout_panel>
<layout_panel name="timers_panel"
Expand Down
1 change: 0 additions & 1 deletion indra/newview/skins/default/xui/en/panel_login.xml
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,6 @@
control_name="RememberPassword"
follows="left|top"
font="SansSerifMedium"
text_color="EmphasisColor"
height="24"
left="408"
bottom_delta="0"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,6 @@
<color_swatch
can_apply_immediately="true"
color="0 0 0 1"
control_name="NameTagBackground"
follows="left|top"
height="24"
label_height="0"
Expand Down
1 change: 0 additions & 1 deletion indra/newview/skins/default/xui/en/widgets/slider_bar.xml
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?xml version="1.0" encoding="utf-8" standalone="yes"?>
<!--All horizontal sliders are configured to have no highlighted track. See EXT-5939. -->
<slider_bar follows="left|top"
track_color="SliderTrackColor"
thumb_outline_color="SliderThumbOutlineColor"
thumb_center_color="SliderThumbCenterColor"
thumb_image="SliderThumb_Off"
Expand Down

0 comments on commit 169599f

Please sign in to comment.