From 4df63c7f04bec29f5a91a480655cd7d2b3a7fbb3 Mon Sep 17 00:00:00 2001 From: Kenn Jacobsen Date: Tue, 30 Apr 2019 21:19:35 +0200 Subject: [PATCH] V8: Save the slider default values on the model if no value is set (#4931) --- .../propertyeditors/slider/slider.controller.js | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/slider/slider.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/slider/slider.controller.js index 6c57cecdf1..a20289d076 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/slider/slider.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/slider/slider.controller.js @@ -26,10 +26,6 @@ }; function init() { - - // convert to array - $scope.sliderValue = $scope.model.value ? $scope.model.value.split(',') : null; - configureDefaults(); // format config to fit slider plugin @@ -38,6 +34,14 @@ const tooltips = $scope.model.config.enableRange ? [true, true] : [true]; const min = $scope.model.config.minVal ? [$scope.model.config.minVal] : [$scope.model.config.minVal]; const max = $scope.model.config.maxVal ? [$scope.model.config.maxVal] : [$scope.model.config.maxVal]; + + // set model.value to the default value if it's not set (but don't trigger setDirty, just silently update with the default value) + if (!$scope.model.value) { + $scope.model.value = start.toString(); + } + // convert to array + $scope.sliderValue = $scope.model.value ? $scope.model.value.split(',') : null; + // don't render values with decimal places if the step increment in a whole number var stepDecimalPlaces = $scope.model.config.step % 1 == 0 ? 0