From 299f6ef8499bb3ee4ada5d63dc71091fddcb39c6 Mon Sep 17 00:00:00 2001
From: Bent Bisballe Nyeng <deva@aasimon.org>
Date: Fri, 20 Nov 2015 20:04:39 +0100
Subject: Improve behaviour when holding the mouse down and moving it in and
 out of the checkbox.

---
 plugingui/checkbox.cc | 55 +++++++++++++++++++++++++++++++++------------------
 1 file changed, 36 insertions(+), 19 deletions(-)

(limited to 'plugingui/checkbox.cc')

diff --git a/plugingui/checkbox.cc b/plugingui/checkbox.cc
index 0d37127..60ed7a6 100644
--- a/plugingui/checkbox.cc
+++ b/plugingui/checkbox.cc
@@ -46,11 +46,16 @@ void CheckBox::buttonEvent(ButtonEvent* buttonEvent)
 {
 	if((buttonEvent->direction == Direction::up) || buttonEvent->doubleClick)
 	{
+		buttonDown = false;
 		middle = false;
-		internalSetChecked(!state);
+		if(inCheckbox)
+		{
+			internalSetChecked(!state);
+		}
 	}
 	else
 	{
+		buttonDown = true;
 		middle = true;
 	}
 
@@ -87,29 +92,21 @@ void CheckBox::repaintEvent(RepaintEvent* repaintEvent)
 
 	p.clear();
 
+	p.drawImage(0, (knob.height() - bg_on.height()) / 2, state ? bg_on : bg_off);
+
+	if(middle)
+	{
+		p.drawImage((bg_on.width() - knob.width()) / 2 + 1, 0, knob);
+		return;
+	}
+
 	if(state)
 	{
-		p.drawImage(0, (knob.height() - bg_on.height()) / 2, bg_on);
-		if(middle)
-		{
-			p.drawImage((bg_on.width() - knob.width()) / 2 + 1, 0, knob);
-		}
-		else
-		{
-			p.drawImage(bg_on.width() - 40 + 2, 0, knob);
-		}
+		p.drawImage(bg_on.width() - 40 + 2, 0, knob);
 	}
 	else
 	{
-		p.drawImage(0, (knob.height() - bg_off.height()) / 2, bg_off);
-		if(middle)
-		{
-			p.drawImage((bg_on.width() - knob.width()) / 2 + 1, 0, knob);
-		}
-		else
-		{
-			p.drawImage(0, 0, knob);
-		}
+		p.drawImage(0, 0, knob);
 	}
 }
 
@@ -123,6 +120,26 @@ void CheckBox::setChecked(bool c)
 	internalSetChecked(c);
 }
 
+void CheckBox::mouseLeaveEvent()
+{
+	inCheckbox = false;
+	if(buttonDown)
+	{
+		middle = false;
+		repaintEvent(nullptr);
+	}
+}
+
+void CheckBox::mouseEnterEvent()
+{
+	inCheckbox = true;
+	if(buttonDown)
+	{
+		middle = true;
+		repaintEvent(nullptr);
+	}
+}
+
 void CheckBox::internalSetChecked(bool checked)
 {
 	if(checked == state)
-- 
cgit v1.2.3