Remove two TODOs in Slider. Clamp new_value in Slider::setValue.

2 files changed, 9 insertions, 3 deletions

diff --git a/plugingui/diskstreamingframecontent.cc b/plugingui/diskstreamingframecontent.cc
index f2536f7..75666c2 100644
--- a/plugingui/diskstreamingframecontent.cc
+++ b/plugingui/diskstreamingframecontent.cc
@@ -42,7 +42,7 @@ DiskstreamingframeContent::DiskstreamingframeContent(Widget* parent,
 	label_text.setText("Cache limit (max memory usage):");
 	label_text.setAlignment(TextAlignment::center);
 
-	button.setText("Apply"); // TODO connect
+	button.setText("Apply");
 
 	label_size.setText("0 MB");
 	label_size.setAlignment(TextAlignment::center);
diff --git a/plugingui/slider.cc b/plugingui/slider.cc
index b558468..41ffb01 100644
--- a/plugingui/slider.cc
+++ b/plugingui/slider.cc
@@ -45,9 +45,15 @@ Slider::Slider(Widget* parent) : Widget(parent)
 
 void Slider::setValue(float new_value)
 {
-	// TODO make sure that we get values in range [0, 1]
-
 	current_value = new_value;
+	if (current_value < 0.)
+	{
+		current_value = 0.;
+	}
+	else if (current_value > 1.0) {
+		current_value = 1.0;
+	}
+
 	redraw();
 	clickNotifier();
 	valueChangedNotifier(current_value);