From db1c8ddee47c0ed53946aee0f8d1a6abc088dcb4 Mon Sep 17 00:00:00 2001
From: laserpants <johannes@isomorphic.co>
Date: Fri, 25 Mar 2016 23:37:59 +0300
Subject: [PATCH] simplify code

---
 components/slider.cpp | 12 +++++++-----
 components/slider.h   |  5 +++++
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/components/slider.cpp b/components/slider.cpp
index 64e1062..8243e40 100644
--- a/components/slider.cpp
+++ b/components/slider.cpp
@@ -44,7 +44,7 @@ void Handle::mousePressEvent(QMouseEvent *event)
 
 void Handle::mouseMoveEvent(QMouseEvent *event)
 {
-    setPosition(_offset + event->globalPos());
+    setRelativePosition(event->globalPos());
 }
 
 Slider::Slider(QWidget *parent)
@@ -85,9 +85,12 @@ void Slider::mousePressEvent(QMouseEvent *event)
 {
     const QSize s = _handle->sizeHint();
     const QPoint p = event->pos();
-    const QPoint newPos = p - QPoint(s.width()/2, s.height()/2);
 
-    if (Qt::Horizontal == _orientation ? touchesRail(p.y(), height()/2) : touchesRail(p.x(), width()/2)) {
+    if (Qt::Horizontal == _orientation
+            ? touchesRail(p.y(), height()/2)
+            : touchesRail(p.x(), width()/2))
+    {
+        const QPoint newPos = p - QPoint(s.width()/2, s.height()/2);
         _handle->setPosition(newPos);
         _handle->setOffset(newPos - event->globalPos());
         _drag = true;
@@ -100,7 +103,7 @@ void Slider::mousePressEvent(QMouseEvent *event)
 void Slider::mouseMoveEvent(QMouseEvent *event)
 {
     if (_drag) {
-        _handle->setPosition(_handle->offset() + event->globalPos());
+        _handle->setRelativePosition(event->globalPos());
     }
     QWidget::mouseMoveEvent(event);
 }
@@ -116,4 +119,3 @@ bool Slider::touchesRail(int p, int x) const
 {
     return (p >= x-2 && p < x+2);
 }
-
diff --git a/components/slider.h b/components/slider.h
index 4dad01b..9be0250 100644
--- a/components/slider.h
+++ b/components/slider.h
@@ -15,10 +15,15 @@ public:
     ~Handle();
 
     inline QSize sizeHint() const { return QSize(16, 16); }
+
+    inline void setRelativePosition(const QPoint &pos) { setPosition(_offset + pos); }
+
     inline void setPosition(const QPoint &pos) { _position = pos; refreshGeometry(); }
     inline const QPoint &position() const { return _position; }
+
     inline void setOffset(const QPoint &offset) { _offset = offset; update(); }
     inline const QPoint &offset() const { return _offset; }
+
     void refreshGeometry();
 
 protected: