Fix #423 dealing with strike through and DueDate...

1 files changed, 15 insertions, 5 deletions

diff --git a/core/pim/todo/tableview.cpp b/core/pim/todo/tableview.cpp
index 743deb7..ded2dbd 100644
--- a/core/pim/todo/tableview.cpp
+++ b/core/pim/todo/tableview.cpp
@@ -2,3 +2,3 @@
                =.            This file is part of the OPIE Project
-             .=l.            Copyright (c)  2002 <>
+             .=l.            Copyright (c)  2002 <zecke>
            .>+-=
@@ -272,2 +272,3 @@ void TableView::slotPressed(int row, int col, int,
 
+    qWarning("pressed row %d  col %d   x:%d+y:%d", row,col,point.x(),point.y() );
     m_prevP = point;
@@ -498,2 +499,10 @@ void TableView::timerEvent( QTimerEvent* ev ) {
  * 8 we mark the underlying todo as completed and do a repaint
+ *
+ * BUG: When clicking on the Due column and it's scrollable
+ * the todo is marked as completed...
+ * REASON: QTable is doing auto scrolling which leads to a move
+ * in the x coordinate and this way it's able to pass the
+ * m_completeStrokeWidth criteria
+ * WORKAROUND: strike through needs to strike through the same
+ * row and two columns!
  */
@@ -501,5 +510,7 @@ void TableView::contentsMouseReleaseEvent( QMouseEvent* e) {
     int row = rowAt(m_prevP.y());
-    if ( row == rowAt( e->y() ) && row != -1 ) {
-        if ( ::abs( m_prevP.x() - e->x() ) >=  m_completeStrokeWidth ) {
-            qWarning("current row %d", row );
+    int colOld = columnAt(m_prevP.x() );
+    int colNew = columnAt(e->x() );
+    qWarning("colNew: %d colOld: %d", colNew, colOld );
+    if ( row == rowAt( e->y() ) && row != -1 &&
+         colOld != colNew ) {
             OTodo todo = sorted()[row];
@@ -509,3 +520,2 @@ void TableView::contentsMouseReleaseEvent( QMouseEvent* e) {
             return;
-        }
     }