Index: binaries/data/mods/mod/gui/gui.rnc =================================================================== --- binaries/data/mods/mod/gui/gui.rnc +++ binaries/data/mods/mod/gui/gui.rnc @@ -103,6 +103,7 @@ attribute sprite2_pressed { text }?& attribute sprite_selectarea { text }?& attribute square_side { xsd:decimal }?& + attribute step_size positive-decimal }?& attribute textcolor { ccolor }?& attribute textcolor_disabled { ccolor }?& attribute textcolor_over { ccolor }?& Index: binaries/data/mods/mod/gui/gui.rng =================================================================== --- binaries/data/mods/mod/gui/gui.rng +++ binaries/data/mods/mod/gui/gui.rng @@ -410,6 +410,11 @@ + + + + + Index: binaries/data/mods/public/gui/common/gamedescription.js =================================================================== --- binaries/data/mods/public/gui/common/gamedescription.js +++ binaries/data/mods/public/gui/common/gamedescription.js @@ -196,31 +196,25 @@ let title = translateVictoryCondition(victoryCondition.Name); if (victoryCondition.Name == "wonder") - { - let wonderDuration = Math.round(g_GameAttributes.settings.WonderDuration); title = sprintf( translatePluralWithContext( "victory condition", "Wonder (%(min)s minute)", "Wonder (%(min)s minutes)", - wonderDuration + g_GameAttributes.settings.WonderDuration ), - { "min": wonderDuration }); - } + { "min": g_GameAttributes.settings.WonderDuration }); let isCaptureTheRelic = victoryCondition.Name == "capture_the_relic"; if (isCaptureTheRelic) - { - let relicDuration = Math.round(g_GameAttributes.settings.RelicDuration); title = sprintf( translatePluralWithContext( "victory condition", "Capture the Relic (%(min)s minute)", "Capture the Relic (%(min)s minutes)", - relicDuration + g_GameAttributes.settings.RelicDuration ), - { "min": relicDuration }); - } + { "min": g_GameAttributes.settings.RelicDuration }); titles.push({ "label": title, @@ -230,7 +224,7 @@ if (isCaptureTheRelic) titles.push({ "label": translate("Relic Count"), - "value": Math.round(g_GameAttributes.settings.RelicCount) + "value": g_GameAttributes.settings.RelicCount }); if (victoryCondition.Name == "regicide") @@ -275,17 +269,16 @@ "value": translate("If one player wins, his or her allies win too. If one group of allies remains, they win.") }); - let ceasefire = Math.round(g_GameAttributes.settings.Ceasefire); titles.push({ "label": translate("Ceasefire"), "value": - ceasefire == 0 ? + g_GameAttributes.settings.Ceasefire == 0 ? translate("disabled") : sprintf(translatePlural( "For the first minute, other players will stay neutral.", "For the first %(min)s minutes, other players will stay neutral.", - ceasefire), - { "min": ceasefire }) + g_GameAttributes.settings.Ceasefire), + { "min": g_GameAttributes.settings.Ceasefire }) }); if (g_GameAttributes.map == "random") Index: binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/GameSettingControlSlider.js =================================================================== --- binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/GameSettingControlSlider.js +++ binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/GameSettingControlSlider.js @@ -19,6 +19,8 @@ if (this.MaxValue !== undefined) this.slider.max_value = this.MaxValue; + + this.slider.step_size = this.StepSize; } setControl(gameSettingControlManager) @@ -75,5 +77,4 @@ } } -GameSettingControlSlider.prototype.UnknownValue = - translateWithContext("settings value", "Unknown"); +GameSettingControlSlider.prototype.StepSize = 1; Index: binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/Ceasefire.js =================================================================== --- binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/Ceasefire.js +++ binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/Ceasefire.js @@ -34,11 +34,10 @@ onGameAttributesBatchChange() { - let value = Math.round(g_GameAttributes.settings.Ceasefire); + let value = g_GameAttributes.settings.Ceasefire; this.sprintfValue.minutes = value; - this.setSelectedValue( - g_GameAttributes.settings.Ceasefire, + value, value == 0 ? this.NoCeasefireCaption : sprintf(this.CeasefireCaption(value), this.sprintfValue)); @@ -50,11 +49,6 @@ this.gameSettingsControl.updateGameAttributes(); this.gameSettingsControl.setNetworkGameAttributes(); } - - onGameAttributesFinalize() - { - g_GameAttributes.settings.Ceasefire = Math.round(g_GameAttributes.settings.Ceasefire); - } }; GameSettingControls.Ceasefire.prototype.TitleCaption = Index: binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicCount.js =================================================================== --- binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicCount.js +++ binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicCount.js @@ -60,10 +60,10 @@ if (this.available) { - let value = Math.round(g_GameAttributes.settings.RelicCount); + let value = g_GameAttributes.settings.RelicCount; this.sprintfValue.number = value; this.setSelectedValue( - g_GameAttributes.settings.RelicCount, + value, value == 0 ? this.InstantVictory : sprintf(this.CaptionRelicCount(value), this.sprintfValue)); } } @@ -74,12 +74,6 @@ this.gameSettingsControl.updateGameAttributes(); this.gameSettingsControl.setNetworkGameAttributes(); } - - onGameAttributesFinalize() - { - if (this.available) - g_GameAttributes.settings.RelicCount = Math.round(g_GameAttributes.settings.RelicCount); - } }; GameSettingControls.RelicCount.prototype.TitleCaption = Index: binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicDuration.js =================================================================== --- binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicDuration.js +++ binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/RelicDuration.js @@ -60,10 +60,10 @@ if (this.available) { - let value = Math.round(g_GameAttributes.settings.RelicDuration); + let value = g_GameAttributes.settings.RelicDuration; this.sprintfValue.min = value; this.setSelectedValue( - g_GameAttributes.settings.RelicDuration, + value, value == 0 ? this.InstantVictory : sprintf(this.CaptionVictoryTime(value), this.sprintfValue)); } } @@ -74,12 +74,6 @@ this.gameSettingsControl.updateGameAttributes(); this.gameSettingsControl.setNetworkGameAttributes(); } - - onGameAttributesFinalize() - { - if (this.available) - g_GameAttributes.settings.RelicDuration = Math.round(g_GameAttributes.settings.RelicDuration); - } }; GameSettingControls.RelicDuration.prototype.TitleCaption = Index: binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/SeaLevelRiseTime.js =================================================================== --- binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/SeaLevelRiseTime.js +++ binaries/data/mods/public/gui/gamesetup/Pages/GameSetupPage/GameSettings/Single/Sliders/SeaLevelRiseTime.js @@ -47,11 +47,10 @@ if (!this.values) return; - let value = Math.round(g_GameAttributes.settings.SeaLevelRiseTime); + let value = g_GameAttributes.settings.SeaLevelRiseTime; this.sprintfValue.minutes = value; - this.setSelectedValue( - g_GameAttributes.settings.SeaLevelRiseTime, + value, sprintf(this.SeaLevelRiseTimeCaption(value), this.sprintfValue)); } @@ -61,12 +60,6 @@ this.gameSettingsControl.updateGameAttributes(); this.gameSettingsControl.setNetworkGameAttributes(); } - - onGameAttributesFinalize() - { - if (this.values) - g_GameAttributes.settings.SeaLevelRiseTime = Math.round(g_GameAttributes.settings.SeaLevelRiseTime); - } }; GameSettingControls.SeaLevelRiseTime.prototype.TitleCaption = Index: binaries/data/mods/public/gui/options/options.js =================================================================== --- binaries/data/mods/public/gui/options/options.js +++ binaries/data/mods/public/gui/options/options.js @@ -157,6 +157,7 @@ "initGUI": (option, control) => { control.max_value = option.max; control.min_value = option.min; + control.step_size = option.step; }, "tooltip": (value, option) => sprintf(translateWithContext("slider number", "Value: %(val)s (min: %(min)s, max: %(max)s)"), { Index: binaries/data/mods/public/gui/options/options.json =================================================================== --- binaries/data/mods/public/gui/options/options.json +++ binaries/data/mods/public/gui/options/options.json @@ -123,7 +123,8 @@ "tooltip": "Number of shader effects. REQUIRES GAME RESTART", "config": "materialmgr.quality", "min": 0, - "max": 10 + "max": 10, + "step": 1 }, { "type": "boolean", @@ -242,7 +243,8 @@ "tooltip": "To save CPU workload, throttle render frequency in all menus. Set to maximum to disable throttling.", "config": "adaptivefps.menu", "min": 20, - "max": 100 + "max": 100, + "step": 1 }, { "type": "slider", @@ -250,7 +252,8 @@ "tooltip": "To save CPU workload, throttle render frequency in running games. Set to maximum to disable throttling.", "config": "adaptivefps.session", "min": 20, - "max": 100 + "max": 100, + "step": 1 } ] }, @@ -265,7 +268,8 @@ "config": "sound.mastergain", "function": "SetMasterGain", "min": 0, - "max": 2 + "max": 2, + "step": 0.01 }, { "type": "slider", @@ -274,7 +278,8 @@ "config": "sound.musicgain", "function": "SetMusicGain", "min": 0, - "max": 2 + "max": 2, + "step": 0.01 }, { "type": "slider", @@ -283,7 +288,8 @@ "config": "sound.ambientgain", "function": "SetAmbientGain", "min": 0, - "max": 2 + "max": 2, + "step": 0.01 }, { "type": "slider", @@ -292,7 +298,8 @@ "config": "sound.actiongain", "function": "SetActionGain", "min": 0, - "max": 2 + "max": 2, + "step": 0.01 }, { "type": "slider", @@ -301,7 +308,8 @@ "config": "sound.uigain", "function": "SetUIGain", "min": 0, - "max": 2 + "max": 2, + "step": 0.01 }, { "type": "boolean", @@ -427,7 +435,8 @@ "tooltip": "The wounded unit hotkey considers the selected units as wounded if their health percentage falls below this number.", "config": "gui.session.woundedunithotkeythreshold", "min": 0, - "max": 100 + "max": 100, + "step": 1 }, { "type": "number", @@ -435,7 +444,8 @@ "tooltip": "Number of units trained per batch by default.", "config": "gui.session.batchtrainingsize", "min": 1, - "max": 20 + "max": 20, + "step": 1 }, { "type": "slider", @@ -443,7 +453,8 @@ "tooltip": "Number of times you have to scroll to increase/decrease the batchsize by 1.", "config": "gui.session.scrollbatchratio", "min": 0.1, - "max": 30 + "max": 30, + "step": 0.1 }, { "type": "boolean", Index: source/gui/ObjectTypes/CSlider.h =================================================================== --- source/gui/ObjectTypes/CSlider.h +++ source/gui/ObjectTypes/CSlider.h @@ -52,11 +52,6 @@ CRect GetButtonRect() const; - /** - * @return ratio between the value of the slider and its actual size in the GUI - */ - float GetSliderRatio() const; - void IncrementallyChangeValue(const float value); // Settings @@ -67,9 +62,7 @@ CGUISpriteInstance m_Sprite; CGUISpriteInstance m_SpriteBar; float m_Value; - -private: - CPos m_Mouse; + float m_StepSize; }; #endif // INCLUDED_CSLIDER Index: source/gui/ObjectTypes/CSlider.cpp =================================================================== --- source/gui/ObjectTypes/CSlider.cpp +++ source/gui/ObjectTypes/CSlider.cpp @@ -31,6 +31,7 @@ m_CellID(), m_MaxValue(), m_MinValue(), + m_StepSize(), m_Sprite(), m_SpriteBar(), m_Value() @@ -39,6 +40,7 @@ RegisterSetting("cell_id", m_CellID); RegisterSetting("max_value", m_MaxValue); RegisterSetting("min_value", m_MinValue); + RegisterSetting("step_size", m_StepSize); RegisterSetting("sprite", m_Sprite); RegisterSetting("sprite_bar", m_SpriteBar); RegisterSetting("value", m_Value); @@ -56,11 +58,6 @@ IGUIButtonBehavior::ResetStates(); } -float CSlider::GetSliderRatio() const -{ - return (m_MaxValue - m_MinValue) / (m_CachedActualSize.GetWidth() - m_ButtonSide); -} - void CSlider::IncrementallyChangeValue(const float difference) { m_Value = Clamp(m_Value + difference, m_MinValue, m_MaxValue); @@ -83,14 +80,14 @@ { if (m_Pressed) break; - IncrementallyChangeValue(-0.01f); + IncrementallyChangeValue(-m_StepSize); break; } case GUIM_MOUSE_WHEEL_UP: { if (m_Pressed) break; - IncrementallyChangeValue(0.01f); + IncrementallyChangeValue(+m_StepSize); break; } case GUIM_MOUSE_PRESS_LEFT: @@ -99,8 +96,11 @@ { if (m_Pressed) { - m_Mouse = m_pGUI.GetMousePos(); - IncrementallyChangeValue((m_Mouse.x - GetButtonRect().CenterPoint().x) * GetSliderRatio()); + const float ratio = (m_MaxValue - m_MinValue) / (m_CachedActualSize.GetWidth() - m_ButtonSide); + const float value = ((m_pGUI.GetMousePos().x - m_CachedActualSize.left) * ratio); + const float valueStep = value - fmod(value , m_StepSize); + m_Value = Clamp(valueStep, m_MinValue, m_MaxValue); + UpdateValue(); } break; } @@ -127,8 +127,8 @@ CRect CSlider::GetButtonRect() const { - float ratio = m_MaxValue > m_MinValue ? (m_Value - m_MinValue) / (m_MaxValue - m_MinValue) : 0.0f; - float x = m_CachedActualSize.left + ratio * (m_CachedActualSize.GetWidth() - m_ButtonSide); - float y = m_CachedActualSize.top + (m_CachedActualSize.GetHeight() - m_ButtonSide) / 2.0; + const float ratio = m_MaxValue > m_MinValue ? (m_Value - m_MinValue) / (m_MaxValue - m_MinValue) : 0.0f; + const float x = m_CachedActualSize.left + ratio * (m_CachedActualSize.GetWidth() - m_ButtonSide); + const float y = m_CachedActualSize.top + (m_CachedActualSize.GetHeight() - m_ButtonSide) / 2.0; return CRect(x, y, x + m_ButtonSide, y + m_ButtonSide); }