fixx

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
@@ -1,12 +1,16 @@
 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 ************
 
 More bugfixes.
 Added possibility to clone in KO/Pi a todo with all its subtodos.
 Added config option for alternate background in list views.
 
diff --git a/korganizer/koagenda.cpp b/korganizer/koagenda.cpp
index 114ed75..355f4bb 100644
--- a/korganizer/koagenda.cpp
+++ b/korganizer/koagenda.cpp
@@ -469,25 +469,18 @@ bool KOAgenda::eventFilter_mouse(QObject *object, QMouseEvent *me)
         if (object != viewport()) {
             mPopupItem = (KOAgendaItem *)object;
             mPopupKind = 1;
             if (me->button() == RightButton) {
                 popupMenu();
             } 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() */) {
                         mActionItem = 0;
                     } else {
                         startItemAction(viewportPos);
                     }
                 }
@@ -605,191 +598,17 @@ bool KOAgenda::eventFilter_mouse(QObject *object, QMouseEvent *me)
                 emit showIncidenceSignal(doubleClickedItem->incidence());
         }
         break;
 
     default:
         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 )
 {
     if ( item == 1 ) { //new event
         newEventSignal(mStartCellX ,mStartCellY  );
     } else
         if ( item == 2 ) { //new event
@@ -798,49 +617,54 @@ void KOAgenda::newItem( int item )
             {
                 emit showDateView( item, mStartCellX );
                 // 3Day view
                 // 4Week view
                 // 5Month view
                 // 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)
 {
   int x,y;
   viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
   int gx,gy;
   contentsToGrid(x,y,gx,gy);
@@ -857,28 +681,21 @@ void KOAgenda::performSelectAction(QPoint viewportPos)
   } else {
     mScrollUpTimer.stop();
     mScrollDownTimer.stop();
   }
 
   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 ) {
     if ( gy >= mStartCellY ) {
       int selectionHeight = mSelectionHeight;
       mSelectionHeight = ( gy + 1 ) * mGridSpacingY - mSelectionYTop;
 
       repaintContents( (KOGlobals::self()->reverseLayout() ?
@@ -895,17 +712,16 @@ void KOAgenda::performSelectAction(QPoint viewportPos)
 void KOAgenda::endSelectAction( bool emitNewEvent )
 {
   mActionType = NOP;
   mScrollUpTimer.stop();
   mScrollDownTimer.stop();
   
   emit newTimeSpanSignal(mStartCellX,mStartCellY,mCurrentCellX,mCurrentCellY);
   if ( emitNewEvent && mStartCellY < mCurrentCellY ) {
-      qDebug("ew event signal ");
       emit newEventSignal(mStartCellX,mStartCellY,mCurrentCellX,mCurrentCellY);
   }
 }
 
 void KOAgenda::startItemAction(QPoint viewportPos)
 {
   int x,y;
   viewportToContents(viewportPos.x(),viewportPos.y(),x,y);
@@ -1095,27 +911,17 @@ void KOAgenda::endItemAction()
   
         if ( placeItem->incidence()->type() == "Todo" ) {
             mSelectedItem = 0;
             //qDebug("todo %d %d %d ", mCurrentCellX, modifiedItem->cellX() ,modifiedItem->cellXWidth());
             modifiedItem->mLastMoveXPos = mCurrentCellX;
             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;
                   item=oldconflictItems.next() ) {
                 placeSubCells(item);    
             } 
             while ( placeItem ) { 
                 //qDebug("placeItem  %s ", placeItem->incidence()->summary().latin1());  
diff --git a/korganizer/koagenda.h b/korganizer/koagenda.h
index 4f1fdb9..fb9983e 100644
--- a/korganizer/koagenda.h
+++ b/korganizer/koagenda.h
@@ -122,16 +122,17 @@ class KOAgenda : public QScrollView
     void drawContentsToPainter( QPainter* paint = 0, bool backgroundOnly = false);
     void finishUpdate();
     void printSelection();
     void storePosition();
     void restorePosition();
     void setPopup( KOEventPopupMenu * p ) { mAllAgendaPopup = p; }
 
   public slots:
+    void slotClearSelection();
     void popupMenu();
     void newItem( int );
     void moveChild( QWidget *, int, int );
     void scrollUp();
     void scrollDown();
     void updateTodo( Todo * t, int , bool );
     void popupAlarm();
 
@@ -141,16 +142,17 @@ class KOAgenda : public QScrollView
     void deselectItem();
     /** Select item. If the argument is 0, the currently selected item gets
      deselected. This function emits the itemSelected(bool) signal to inform
      about selection/deseelction of events. */
     void selectItem(KOAgendaItem *);
     void finishResize();
 
   signals:
+    void signalClearSelection();
     void showDateView( int, int);
     void newEventSignal();
     void newEventSignal(int gx,int gy);
     void newTodoSignal(int gx,int gy);
     void newEventSignal(int gxStart, int gyStart, int gxEnd, int gyEnd);
     void newTimeSpanSignal(int gxStart, int gyStart, int gxEnd, int gyEnd);
     void newStartSelectSignal();
     void showIncidenceSignal(Incidence *);
diff --git a/korganizer/koagendaview.cpp b/korganizer/koagendaview.cpp
index 17f791d..b43c40e 100644
--- a/korganizer/koagendaview.cpp
+++ b/korganizer/koagendaview.cpp
@@ -429,17 +429,16 @@ KOAgendaView::KOAgendaView(Calendar *cal,QWidget *parent,const char *name) :
 
     // Create agenda frame
     QGridLayout *agendaLayout = new QGridLayout(agendaFrame,4,3);
     //  QHBox *agendaFrame = new QHBox(splitterAgenda);
 
     // create event indicator bars
     mEventIndicatorTop = new EventIndicator(EventIndicator::Top,agendaFrame);
 #ifndef DESKTOP_VERSION
-    // FIX
     mEventIndicatorTop->setPaintWidget( mSplitterAgenda );
 #endif
     mDayLabelsFrame = new QHBox(agendaFrame);
     //topLayout->addWidget(mDayLabelsFrame);
     mDayLabels = new QFrame (mDayLabelsFrame);
     mLayoutDayLabels = new QHBoxLayout(mDayLabels);
     agendaLayout->addMultiCellWidget(mDayLabelsFrame ,0,0,0,2);
     agendaLayout->addWidget(mEventIndicatorTop,1,1);
@@ -562,16 +561,18 @@ KOAgendaView::KOAgendaView(Calendar *cal,QWidget *parent,const char *name) :
              SLOT( updateConfig( ) ) );  
     connect( mAgenda, SIGNAL( addToCalSignal(Incidence *, Incidence *) ),
              SLOT( addToCalSlot(Incidence *, Incidence * ) ) );  
     connect( mAllDayAgenda, SIGNAL( addToCalSignal(Incidence * ,Incidence *) ),
              SLOT( addToCalSlot(Incidence * , Incidence *) ) );  
     // 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()) );   
  
 }
 
 void KOAgendaView::toggleAllDay()
 {
     if ( mSplitterAgenda->firstHandle() ) 
         mSplitterAgenda->firstHandle()->toggle();
 }
@@ -720,22 +721,18 @@ void KOAgendaView::createDayLabels()
     if ( !dayLabel  ) {
         appendLabels = true;
         dayLabel = getNewDaylabel();
     }
     dayLabel->setFixedWidth( mTimeLabels->width()+mAgenda->frameWidth() ); 
     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() );
     int counter = -1;
     for( dit = mSelectedDates.begin(); dit != mSelectedDates.end(); ++dit ) {
         ++counter;
         QDate date = *dit;
         // QBoxLayout *dayLayout = new QVBoxLayout(mLayoutDayLabels);
@@ -1174,26 +1171,17 @@ void KOAgendaView::fillAgenda()
                                 if ( curCol-endX < 0 ) {
                                     mAllDayAgenda->insertAllDayItem(event,currentDate,0,curCol);
                                 }
                             }
                         }
                     } 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);
                     } else if (beginX == curCol) {
                         mAllDayAgenda->insertAllDayItem(event,currentDate,beginX,endX);
                     }
                 }
             } else if (event->isMultiDay()) {