1 files changed, 26 insertions, 23 deletions

diff --git a/core/pim/todo/todotable.cpp b/core/pim/todo/todotable.cpp
index 877308a..52a3087 100644
--- a/core/pim/todo/todotable.cpp
+++ b/core/pim/todo/todotable.cpp
@@ -285,62 +285,59 @@ void TodoTable::slotClicked( int row, int col, int, const QPoint &pos )
 {
     if ( !cellGeometry( row, col ).contains(pos) )
 	return;
     // let's switch on the column number...
     switch ( col )
     {
         case 0: {
 		CheckItem *i = static_cast<CheckItem*>(item( row, col ));
 		if ( i ) {
 		    int x = pos.x() - columnPos( col );
 		    int y = pos.y() - rowPos( row );
 		    int w = columnWidth( col );
 		    int h = rowHeight( row );
 		    if ( i && x >= ( w - BoxSize ) / 2 && x <= ( w - BoxSize ) / 2 + BoxSize &&
 			    y >= ( h - BoxSize ) / 2 && y <= ( h - BoxSize ) / 2 + BoxSize ) {
 			i->toggle();
 		    }
 		    emit signalDoneChanged( i->isChecked() );
 		}
 	    }
             break;
         case 1:
             break;
         case 2:
-	  // may as well edit it...
-	  // menuTimer->stop();
-	  // emit signalEdit();
 	  // Show detailed view of the selected entry
 	  {
 	    menuTimer->stop();
 	    ToDoEvent *todo = todoList[static_cast<CheckItem*>(item(row, 0))];
 	    emit showDetails( *todo );
 	  }
 	  break;
         case 3: 
 	  // may as well edit it...
 	  menuTimer->stop();
-	  // emit signalEdit();
+	  emit signalEdit();
 	  break;
     }
 }
 
 void TodoTable::slotPressed( int row, int col, int, const QPoint &pos )
 {
     if ( col == 2 && cellGeometry( row, col ).contains(pos) )
 	menuTimer->start( 750, TRUE );
 }
 
 void TodoTable::slotShowMenu()
 {
     emit signalShowMenu( QCursor::pos() );
 }
 
 void TodoTable::slotCurrentChanged( int, int )
 {
     menuTimer->stop();
 }
 
 void TodoTable::internalAddEntries( QList<ToDoEvent> &list )
 {
     setNumRows( list.count() );
     int row = 0;
@@ -638,69 +635,75 @@ void TodoTable::journalFreeReplaceEntry( const ToDoEvent &todo, int row )
 	static_cast<CheckItem*>(item(row, 0))->setChecked( todo.isCompleted() );
 	static_cast<ComboItem*>(item(row, 1))->setText( QString::number(todo.priority()) );
 	item( row, 2 )->setText( strTodo );
 
  	if (showDeadl){
 	  static_cast<DueTextItem*>(item(row,3))->setToDoEvent(&todo );
  	}
 	todoList.insert( static_cast<CheckItem*>(item(row,0)), new ToDoEvent(todo) );
     }
 }
 
 void TodoTable::journalFreeRemoveEntry( int row )
 {
     CheckItem *chk;
     chk = static_cast<CheckItem*>(item(row, 0 ));
     if ( !chk )
 	return;
     todoList.remove( chk );
 
     realignTable( row );
 }
 
 void TodoTable::keyPressEvent( QKeyEvent *e )
 {
-    if ( e->key() == Key_Space || e->key() == Key_Return ) {
-	switch ( currentColumn() ) {
-	    case 0: {
-		CheckItem *i = static_cast<CheckItem*>(item(currentRow(),
-							    currentColumn()));
-		if ( i )
-		    i->toggle();
-		break;
-	    }
-	    case 1:
-		break;
-	    case 2:
-		emit signalEdit();
-	    default:
-		break;
-	}
-    } else {
-	QTable::keyPressEvent( e );
-    }
+	if ( e->key() == Key_Space || e->key() == Key_Return ) {
+		switch ( currentColumn() ) {
+		case 0: {
+			CheckItem *i = static_cast<CheckItem*>(item(currentRow(),
+								    currentColumn()));
+			if ( i )
+				i->toggle();
+			break;
+		}
+		case 1:
+			break;
+		case 2:{
+			ToDoEvent *todo = todoList[static_cast<CheckItem*>(item(currentRow(), 0))];
+			emit showDetails(*todo);
+			break;
+		}
+		case 3:
+			// Future: Let us change the dueDate directly...
+			emit signalEdit();
+		default:
+			break;
+		}		
+	} else
+		QTable::keyPressEvent( e );
 }
 
+
 QStringList TodoTable::categories()
 {
     // This is called seldom, so calling a load in here
     // should be fine.
     mCat.load( categoryFileName() );
     QStringList categoryList = mCat.labels( "Todo List" );
     return categoryList;
 }
 
 void TodoTable::slotDoFind( const QString &findString, bool caseSensitive,
 			    bool backwards, int category )
 {
     // we have to iterate through the table, this gives the illusion that
     // sorting is actually being used.
     if ( currFindRow < -1 )
 	currFindRow = currentRow() - 1;
     clearSelection( TRUE );
     int rows,
 	row;
     CheckItem *chk;
     QRegExp r( findString );
 
     r.setCaseSensitive( caseSensitive );
     rows = numRows();