From 00006863cb9b7438c1259bda9ab0772afe3f39d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Nusser?= Date: Wed, 19 Apr 2017 23:00:36 +0200 Subject: Remove two TODOs in Slider. Clamp new_value in Slider::setValue. --- plugingui/diskstreamingframecontent.cc | 2 +- plugingui/slider.cc | 10 ++++++++-- 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); -- cgit v1.2.3