4 files changed, 46 insertions, 246 deletions

diff --git a/bin/kdepim/WhatsNew.txt b/bin/kdepim/WhatsNew.txt
index 44bf20b..03895fd 100644
--- a/bin/kdepim/WhatsNew.txt
+++ b/bin/kdepim/WhatsNew.txt
@@ -2,6 +2,10 @@ Info about the changes in new versions of KDE-Pim/Pi
 
 ********** VERSION 2.0.28 ************
 
+KO/Pi:
+Fixed two problems in KO/Pi timetracking.
+Added context menu to month view for creating new events/todos.
+Fixed some other small bugs.
 
 
 ********** VERSION 2.0.27 ************
diff --git a/korganizer/koagenda.cpp b/korganizer/koagenda.cpp
index 114ed75..355f4bb 100644
--- a/korganizer/koagenda.cpp
+++ b/korganizer/koagenda.cpp
@@ -474,15 +474,8 @@ bool KOAgenda::eventFilter_mouse(QObject *object, QMouseEvent *me)
             } else if (me->button() == LeftButton) {
                 mActionItem = (KOAgendaItem *)object;
                 if (mActionItem) {
-                    if ( mSelectionHeight > 0 ) { 
-                        int selectionCellX = mSelectionCellX * mGridSpacingX;
-                        int selectionYTop = mSelectionYTop;
-                        int gridSpacingX = mGridSpacingX;
-                        int selectionHeight = mSelectionHeight;
-                        clearSelection();
-                        repaintContents( selectionCellX, selectionYTop,
-                                         gridSpacingX, selectionHeight,false );
-                    }
+                    emit signalClearSelection();
+                    slotClearSelection();
                     selectItem(mActionItem);
                     Incidence *incidence = mActionItem->incidence();
                     if ( incidence->isReadOnly() /*|| incidence->recurrence()->doesRecur() */) {
@@ -610,181 +603,7 @@ bool KOAgenda::eventFilter_mouse(QObject *object, QMouseEvent *me)
         break;
     }
     return true;
-#if 0
-    //qDebug("KOAgenda::eventFilter_mous ");
-    QPoint viewportPos;
-    if (object != viewport()) {
-        viewportPos = ((QWidget *)object)->mapToParent(me->pos());
-    } else {
-        viewportPos = me->pos();
-    }
-    static int startX = 0;
-    static int startY = 0;
-    static int blockmoveDist = ( QApplication::desktop()->width() < 480 ? 15 : 20 );
-    static bool blockMoving = true;
-    static bool leftMouseDown = false;
-    bool rightButtonPressed = false;
-    switch (me->type())  {
-    case QEvent::MouseButtonPress:
-        if (me->button() == LeftButton) {
-            leftMouseDown = true;
-        }
-        else if (me->button() == RightButton) {
-            leftMouseDown = false;
-        }
-        blockMoving = true;
-        startX = viewportPos.x();
-        startY = viewportPos.y(); 
-        if (object != viewport()) { // item clicked **************
-            if (me->button() == RightButton) {
-                leftMouseDown = false;
-                mClickedItem = (KOAgendaItem *)object;
-                if (mActionItem ) {
-                    endItemAction();
-                }
-                if (mClickedItem) {
-                    selectItem(mClickedItem);
-                    emit showIncidencePopupSignal(mClickedItem->incidence());
-                } 
-                return true;
-            } else if (me->button() == LeftButton) {
-                mActionItem = (KOAgendaItem *)object;
-                if (mActionItem) {
-                    if ( mSelectionHeight > 0 ) { 
-                        int selectionCellX = mSelectionCellX * mGridSpacingX;
-                        int selectionYTop = mSelectionYTop;
-                        int gridSpacingX = mGridSpacingX;
-                        int selectionHeight = mSelectionHeight;
-                        clearSelection();
-                        repaintContents( selectionCellX, selectionYTop,
-                                         gridSpacingX, selectionHeight,false );
-                    }
-                    selectItem(mActionItem);
-                    Incidence *incidence = mActionItem->incidence();
-                    if ( incidence->isReadOnly() /*|| incidence->recurrence()->doesRecur() */) {
-                        mActionItem = 0;
-                    } else {
-                        startItemAction(viewportPos);
-                    }
-                }
-            }
-        } else { // ---------- viewport()
-            selectItem(0);
-            mActionItem = 0;
-            if (me->button() == LeftButton ) {
-                setCursor(arrowCursor);
-                startSelectAction(viewportPos);
-            } else if (me->button() == RightButton ) {
-                setCursor(arrowCursor);
-                if ( leftMouseDown ) { // we have a simulated right click - clear left mouse action
-                    endSelectAction( false ); // do not emit new event signal
-                    leftMouseDown = false; // no more leftMouse computation
-                }
-                int x,y;
-                viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
-                int gx,gy;
-                contentsToGrid(x,y,gx,gy); 
-                mCurrentCellX = gx;
-                mCurrentCellY = gy;
-                mStartCellX = gx;
-                mStartCellY = gy;
-                mNewItemPopup->popup( viewport()->mapToGlobal( me->pos() ) );
-            }
-        }
-        break;
-
-    case QEvent::MouseButtonRelease:
-       
-        if (object != viewport()) { 
-            if (me->button() == LeftButton && leftMouseDown) {
-                if (mActionItem) {  
-                    QPoint clipperPos = clipper()->mapFromGlobal(viewport()->mapToGlobal(viewportPos));
-                    //qDebug(" %d %d %d  ",clipperPos.y(),visibleHeight() , 9  );
-                    if ( mActionType == MOVE && (clipperPos.y() > visibleHeight()-2 ||clipperPos.y() < 0 ) ) {
-                        mScrollUpTimer.stop();
-                        mScrollDownTimer.stop();
-                        mActionItem->resetMove();
-                        placeSubCells( mActionItem );
-                        // emit startDragSignal( mActionItem->incidence() );
-                        setCursor( arrowCursor );
-                        mActionItem = 0;
-                        mActionType = NOP;
-                        mItemMoved = 0;
-                        leftMouseDown = false;
-                        return true;
-                    }
-                    endItemAction();
-                }
-            }
-            
-        } else { // ---------- viewport()
-            if (me->button() == LeftButton && leftMouseDown ) { //left click
-                endSelectAction( true ); // emit new event signal
-            }
-        }
-        if (me->button() == LeftButton) 
-            leftMouseDown = false;
-       
-        break;
 
-    case QEvent::MouseMove:
-        if ( !leftMouseDown )
-            return true;
-        if ( blockMoving ) {
-            int dX, dY; 
-            dX = startX - viewportPos.x();
-            if ( dX < 0   )
-                dX = -dX;
-            dY = viewportPos.y() - startY;
-            if ( dY < 0 )
-                dY = -dY;
-            //qDebug("%d %d %d ", dX, dY , blockmoveDist  );
-            if (  dX > blockmoveDist  || dY > blockmoveDist  ) {
-                blockMoving = false;
-            }
-        }
-        if (object != viewport()) {
-            KOAgendaItem *moveItem = (KOAgendaItem *)object;
-            if (!moveItem->incidence()->isReadOnly()  ) {
-                if (!mActionItem)
-                    setNoActionCursor(moveItem,viewportPos);
-                else {
-                    if ( !blockMoving )
-                        performItemAction(viewportPos);
-                }
-            }
-        } else { // ---------- viewport()
-            if ( mActionType == SELECT ) {
-                performSelectAction( viewportPos );
-            }
-        }
-        break;
-
-    case QEvent::MouseButtonDblClick:
-        blockMoving = false;
-        leftMouseDown = false;
-        if (object == viewport()) {
-            selectItem(0);
-            int x,y;
-            viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
-            int gx,gy;
-            contentsToGrid(x,y,gx,gy);
-            emit newEventSignal(gx,gy);
-        } else {
-            KOAgendaItem *doubleClickedItem = (KOAgendaItem *)object;
-            selectItem(doubleClickedItem);
-            if ( KOPrefs::instance()->mEditOnDoubleClick )
-                emit editIncidenceSignal(doubleClickedItem->incidence());
-            else
-                emit showIncidenceSignal(doubleClickedItem->incidence());
-        }
-        break;
-
-    default:
-        break;
-    }
-    return true;
-#endif 
 }
 
 void KOAgenda::newItem( int item )
@@ -803,39 +622,44 @@ void KOAgenda::newItem( int item )
                 // 6Journal view
             }
 }
+void KOAgenda::slotClearSelection()
+{
+    if (mSelectionHeight) {
+        int selectionX = mSelectionCellX * mGridSpacingX; 
+        int top = mSelectionYTop - 2 *mGridSpacingY;
+        int hei = mSelectionHeight + 4 *mGridSpacingY;
+        clearSelection();
+        repaintContents( selectionX, top,
+                     mGridSpacingX, hei ,false );
+    }
+
+}
 void KOAgenda::startSelectAction(QPoint viewportPos)
 {
-    //emit newStartSelectSignal();
+   
+    emit signalClearSelection();
+    slotClearSelection();
 
-  mActionType = SELECT;
+    mActionType = SELECT;
 
-  int x,y;
-  viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
-  int gx,gy;
-  contentsToGrid(x,y,gx,gy);
+    int x,y;
+    viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
+    int gx,gy;
+    contentsToGrid(x,y,gx,gy);
 
-  mStartCellX = gx;
-  mStartCellY = gy;
-  mCurrentCellX = gx;
-  mCurrentCellY = gy;
-
-  // Store coordinates of old selection
-  int selectionX = mSelectionCellX * mGridSpacingX;
-  int selectionYTop = mSelectionYTop;
-  int selectionHeight = mSelectionHeight;
+    mStartCellX = gx;
+    mStartCellY = gy;
+    mCurrentCellX = gx;
+    mCurrentCellY = gy;
 
-  // Store new selection
-  mSelectionCellX = gx;
-  mSelectionYTop = gy * mGridSpacingY;
-  mSelectionHeight = mGridSpacingY;
+    // Store new selection
+    mSelectionCellX = gx;
+    mSelectionYTop = gy * mGridSpacingY;
+    mSelectionHeight = mGridSpacingY;
   
-  // Clear old selection
-  repaintContents( selectionX, selectionYTop,
-                   mGridSpacingX, selectionHeight,false );
-
-  // Paint new selection
-  //  repaintContents( mSelectionCellX * mGridSpacingX, mSelectionYTop,
-  //               mGridSpacingX, mSelectionHeight );
+    // Paint new selection
+    repaintContents( mSelectionCellX * mGridSpacingX+1, mSelectionYTop,
+                     mGridSpacingX-1, mSelectionHeight );
 }
 
 void KOAgenda::performSelectAction(QPoint viewportPos)
@@ -862,18 +686,11 @@ void KOAgenda::performSelectAction(QPoint viewportPos)
   if ( gy > mCurrentCellY ) {
     mSelectionHeight = ( gy + 1 ) * mGridSpacingY - mSelectionYTop;
 
-#if 0
-    // FIXME: Repaint only the newly selected region
-    repaintContents( mSelectionCellX * mGridSpacingX,
-                     mCurrentCellY + mGridSpacingY,
-                     mGridSpacingX,
-                     mSelectionHeight - ( gy - mCurrentCellY - 1 ) * mGridSpacingY );
-#else
+
     repaintContents( (KOGlobals::self()->reverseLayout() ?
                      mColumns - 1 - mSelectionCellX : mSelectionCellX) *
                      mGridSpacingX, mSelectionYTop,
                      mGridSpacingX, mSelectionHeight , false);
-#endif
 
     mCurrentCellY = gy;
   } else if ( gy < mCurrentCellY ) {
@@ -900,7 +717,6 @@ void KOAgenda::endSelectAction( bool emitNewEvent )
   
   emit newTimeSpanSignal(mStartCellX,mStartCellY,mCurrentCellX,mCurrentCellY);
   if ( emitNewEvent && mStartCellY < mCurrentCellY ) {
-      qDebug("ew event signal ");
       emit newEventSignal(mStartCellX,mStartCellY,mCurrentCellX,mCurrentCellY);
   }
 }
@@ -1100,17 +916,7 @@ void KOAgenda::endItemAction()
             emit itemModified( modifiedItem, mActionType ); 
         }  
         else {  
-#if 0
-            for ( item=oldconflictItems.first(); item != 0;
-                  item=oldconflictItems.next() ) {
-                placeSubCells(item);    
-            } 
-            while ( placeItem ) { 
-                //qDebug("placeItem  %s ", placeItem->incidence()->summary().latin1());  
-                placeSubCells( placeItem );
-                placeItem = placeItem->nextMultiItem();
-            }  
-#endif
+
 
             globalFlagBlockAgendaItemPaint = 1;
             for ( item=oldconflictItems.first(); item != 0;
diff --git a/korganizer/koagenda.h b/korganizer/koagenda.h
index 4f1fdb9..fb9983e 100644
--- a/korganizer/koagenda.h
+++ b/korganizer/koagenda.h
@@ -127,6 +127,7 @@ class KOAgenda : public QScrollView
     void setPopup( KOEventPopupMenu * p ) { mAllAgendaPopup = p; }
 
   public slots:
+    void slotClearSelection();
     void popupMenu();
     void newItem( int );
     void moveChild( QWidget *, int, int );
@@ -146,6 +147,7 @@ class KOAgenda : public QScrollView
     void finishResize();
 
   signals:
+    void signalClearSelection();
     void showDateView( int, int);
     void newEventSignal();
     void newEventSignal(int gx,int gy);
diff --git a/korganizer/koagendaview.cpp b/korganizer/koagendaview.cpp
index 17f791d..b43c40e 100644
--- a/korganizer/koagendaview.cpp
+++ b/korganizer/koagendaview.cpp
@@ -434,7 +434,6 @@ KOAgendaView::KOAgendaView(Calendar *cal,QWidget *parent,const char *name) :
     // create event indicator bars
     mEventIndicatorTop = new EventIndicator(EventIndicator::Top,agendaFrame);
 #ifndef DESKTOP_VERSION
-    // FIX
     mEventIndicatorTop->setPaintWidget( mSplitterAgenda );
 #endif
     mDayLabelsFrame = new QHBox(agendaFrame);
@@ -567,6 +566,8 @@ KOAgendaView::KOAgendaView(Calendar *cal,QWidget *parent,const char *name) :
     // connect( mAgenda, SIGNAL( cloneIncidenceSignal(Incidence *) ), SIGNAL( cloneIncidenceSignal(Incidence *) ) );
     //connect( mAllDayAgenda, SIGNAL( cloneIncidenceSignal(Incidence *) ), SIGNAL( cloneIncidenceSignal(Incidence *) ) );
 
+    connect( mAllDayAgenda, SIGNAL( signalClearSelection() ),mAgenda, SLOT( slotClearSelection()) ); 
+    connect( mAgenda, SIGNAL( signalClearSelection() ),mAllDayAgenda, SLOT( slotClearSelection()) );   
  
 }
 
@@ -725,12 +726,8 @@ void KOAgendaView::createDayLabels()
     dayLabel->setFont( dlf );
     dayLabel->setNum( -1 );
     //dayLabel->setAlignment(QLabel::AlignHCenter);
-#if 0
-    if ( QApplication::desktop()->width() <= 320 )
-        dayLabel->setText( KOGlobals::self()->calendarSystem()->monthName( mSelectedDates.first(), true ).left(2) );
-    else
-#endif
-        dayLabel->setText( KOGlobals::self()->calendarSystem()->monthName( mSelectedDates.first(), true ) );
+
+    dayLabel->setText( KOGlobals::self()->calendarSystem()->monthName( mSelectedDates.first(), true ) );
     dayLabel->show();
     DateList::ConstIterator dit;
     bool oneday = (mSelectedDates.first() == mSelectedDates.last() );
@@ -1179,16 +1176,7 @@ void KOAgendaView::fillAgenda()
                     } else {
                         mAllDayAgenda->insertAllDayItem(event,currentDate,curCol,curCol);
                     }
-#if 0
-                    if (beginX <= 0 && curCol == 0) {
-                        mAllDayAgenda->insertAllDayItem(event,currentDate,beginX,endX);
-                    } else if (beginX == curCol) {
-                        mAllDayAgenda->insertAllDayItem(event,currentDate,beginX,endX);
-                    } else {
-                        qDebug("skipped %d %d %d ",beginX , endX, curCol);
-                    }
-#endif
-                        //mAllDayAgenda->insertAllDayItem(event,currentDate,curCol,curCol);
+                    
                 } else {
                     if (beginX <= 0 && curCol == 0) {
                         mAllDayAgenda->insertAllDayItem(event,currentDate,beginX,endX);