layout fix

1 files changed, 80 insertions, 92 deletions

diff --git a/korganizer/koagenda.cpp b/korganizer/koagenda.cpp
index 14f52b8..ed7a443 100644
--- a/korganizer/koagenda.cpp
+++ b/korganizer/koagenda.cpp
@@ -347,18 +347,9 @@ void KOAgenda::changeColumns(int columns)
 {
-  if (columns == 0) {
-    kdDebug() << "KOAgenda::changeColumns() called with argument 0" << endl;
-    return;
-  }
-
-  clear();
-  
-  mColumns = columns;
-//  setMinimumSize(mColumns * 10, mGridSpacingY + 1);
-//  init();
-//  update();
-  //qDebug("KOAgenda::changeColumns ");
+    if (columns == 0) {
+        qDebug("KOAgenda::changeColumns() called with argument 0 ");
+        return;
+    }
+    clear();
+    mColumns = columns;
     computeSizes();
-    // QResizeEvent event( size(), size() );
-
-    //QApplication::sendEvent( this, &event );
 }
@@ -371,4 +362,4 @@ bool KOAgenda::eventFilter ( QObject *object, QEvent *event )
 {
-//  kdDebug() << "KOAgenda::eventFilter" << endl;
-  switch(event->type()) {
+    //  kdDebug() << "KOAgenda::eventFilter" << endl;
+    switch(event->type()) {
     case QEvent::MouseButtonPress:
@@ -377,12 +368,12 @@ bool KOAgenda::eventFilter ( QObject *object, QEvent *event )
     case QEvent::MouseMove:
-      return eventFilter_mouse(object, static_cast<QMouseEvent *>(event));
+        return eventFilter_mouse(object, static_cast<QMouseEvent *>(event));
 
     case (QEvent::Leave):
-      if (!mActionItem)
-        setCursor(arrowCursor);
-      return true;
+        if (!mActionItem)
+            setCursor(arrowCursor);
+        return true;
 
     default:
-      return QScrollView::eventFilter(object,event);
-  }
+        return QScrollView::eventFilter(object,event);
+    }
 }
@@ -1750,6 +1741,4 @@ KOAgendaItem *KOAgenda::insertItem (Incidence *event,QDate qd,int X,int YTop,int
 {
-  //kdDebug() << "KOAgenda::insertItem:" << event->summary() << "-" << qd.toString() << " ;top, bottom:" << YTop << "," << YBottom << endl;
-
   if (mAllDayMode) {
-    kdDebug() << "KOAgenda: calling insertItem in all-day mode is illegal." << endl;
+      qDebug("KOAgenda: calling insertItem in all-day mode is illegal. ");
     return 0;
@@ -1762,3 +1751,2 @@ KOAgendaItem *KOAgenda::insertItem (Incidence *event,QDate qd,int X,int YTop,int
   if (YSize < 0) {
-    kdDebug() << "KOAgenda::insertItem(): Text: " << agendaItem->text() << " YSize<0" << endl;
     YSize = 1;
@@ -1892,58 +1880,58 @@ void KOAgenda::computeSizes()
         return;
+    int frameOffset = frameWidth() * 2 +1;
+    if (mAllDayMode) {
+        mGridSpacingX = (width()-frameOffset) / mColumns;
+        mGridSpacingY = height() - 2 * frameWidth() - 1;
+        resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY + 1);
+        //    mGridSpacingY = height();
+        //    resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
 
-  if (mAllDayMode) {
-    mGridSpacingX = (width()-3) / mColumns;
-    mGridSpacingY = height() - 2 * frameWidth() - 1;
-    resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY + 1);
-//    mGridSpacingY = height();
-//    resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
-
-    KOAgendaItem *item;
-    int subCellWidth;
-    for ( item=mItems.first(); item != 0; item=mItems.next() ) {
-      subCellWidth = mGridSpacingY / item->subCells();
-      item->resize(mGridSpacingX * item->cellWidth(),subCellWidth);
-      moveChild(item,KOGlobals::self()->reverseLayout() ?
-                     (mColumns - 1 - item->cellX()) * mGridSpacingX :
+        KOAgendaItem *item;
+        int subCellWidth;
+        for ( item=mItems.first(); item != 0; item=mItems.next() ) {
+            subCellWidth = mGridSpacingY / item->subCells();
+            item->resize(mGridSpacingX * item->cellWidth(),subCellWidth);
+            moveChild(item,KOGlobals::self()->reverseLayout() ?
+                      (mColumns - 1 - item->cellX()) * mGridSpacingX :
                       item->cellX() * mGridSpacingX,
                       item->subCell() * subCellWidth);
-    }
-    KOPrefs::instance()->mAllDaySize = mGridSpacingY;
-  } else {
-    mGridSpacingX = (width() - verticalScrollBar()->width()-3)/mColumns;
-    if (height() >   mGridSpacingY * mRows + 1 ) {
-        KOPrefs::instance()->mHourSize = ((height())/mRows)+1;
-        mGridSpacingY = KOPrefs::instance()->mHourSize ;
-        resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
-        emit resizedSignal();
-    } else
-        resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
-    KOAgendaItem *item;
-    int subCellWidth;
+        }
+        KOPrefs::instance()->mAllDaySize = mGridSpacingY;
+    } else {
+        mGridSpacingX = (width() - verticalScrollBar()->width()-frameOffset)/mColumns;
+        if (height() >   mGridSpacingY * mRows + 1 ) {
+            KOPrefs::instance()->mHourSize = ((height())/mRows)+1;
+            mGridSpacingY = KOPrefs::instance()->mHourSize ;
+            resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
+            emit resizedSignal();
+        } else
+            resizeContents( mGridSpacingX * mColumns + 1 , mGridSpacingY * mRows + 1 );
+        KOAgendaItem *item;
+        int subCellWidth;
     
-    for ( item=mItems.first(); item != 0; item=mItems.next() ) {
-      subCellWidth = mGridSpacingX / item->subCells();
-      item->resize(subCellWidth,item->height());
-      moveChild(item,(KOGlobals::self()->reverseLayout() ?
-                     (mColumns - 1 - item->cellX()) * mGridSpacingX :
-                     item->cellX() * mGridSpacingX) +
-                     item->subCell() * subCellWidth,childY(item));
+        for ( item=mItems.first(); item != 0; item=mItems.next() ) {
+            subCellWidth = mGridSpacingX / item->subCells();
+            item->resize(subCellWidth,item->height());
+            moveChild(item,(KOGlobals::self()->reverseLayout() ?
+                            (mColumns - 1 - item->cellX()) * mGridSpacingX :
+                            item->cellX() * mGridSpacingX) +
+                      item->subCell() * subCellWidth,childY(item));
+        }
+    }
+    int cw = contentsWidth();
+    int ch = contentsHeight();
+    if ( mAllDayMode ) { 
+        QPixmap* paintPixAll = KOAgendaItem::paintPixAllday();
+        if ( (paintPixAll->width() < cw || paintPixAll->height() <  ch) && cw > 0  && ch > 0  )
+            paintPixAll->resize( cw, ch ); 
+    } else {
+        QPixmap* paintPix = KOAgendaItem::paintPix();
+        if ( paintPix->width() < cw || paintPix->height() <  ch )
+            KOAgendaItem::resizePixmap( cw , ch );
     }
-  }
-  int cw = contentsWidth();
-  int ch = contentsHeight();
-  if ( mAllDayMode ) { 
-      QPixmap* paintPixAll = KOAgendaItem::paintPixAllday();
-      if ( (paintPixAll->width() < cw || paintPixAll->height() <  ch) && cw > 0  && ch > 0  )
-          paintPixAll->resize( cw, ch ); 
-  } else {
-      QPixmap* paintPix = KOAgendaItem::paintPix();
-      if ( paintPix->width() < cw || paintPix->height() <  ch )
-          KOAgendaItem::resizePixmap( cw , ch );
-  }
 
-  checkScrollBoundaries();
-  marcus_bains();
-  drawContentsToPainter();
-  viewport()->repaint(false);
+    checkScrollBoundaries();
+    marcus_bains();
+    drawContentsToPainter();
+    viewport()->repaint(false);
 }
@@ -1952,3 +1940,3 @@ void KOAgenda::scrollUp()
 {
-  scrollBy(0,-mScrollOffset);
+    scrollBy(0,-mScrollOffset);
 }
@@ -1958,3 +1946,3 @@ void KOAgenda::scrollDown()
 {
-  scrollBy(0,mScrollOffset);
+    scrollBy(0,mScrollOffset);
 }
@@ -1963,15 +1951,15 @@ void KOAgenda::popupAlarm()
 {
-  if (!mClickedItem) {
-      qDebug("KOAgenda::popupAlarm() called without having a clicked item ");
-    return;
-  }
-// TODO: deal correctly with multiple alarms
-  Alarm* alarm;
-  QPtrList<Alarm> list(mClickedItem->incidence()->alarms());
-  for(alarm=list.first();alarm;alarm=list.next()) {
-      alarm->toggleAlarm();
-  }
-  emit itemModified( mClickedItem , KOGlobals::EVENTEDITED );
-  mClickedItem->paintMe( true );
-  mClickedItem->repaint( false );
+    if (!mClickedItem) {
+        qDebug("KOAgenda::popupAlarm() called without having a clicked item ");
+        return;
+    }
+    // TODO: deal correctly with multiple alarms
+    Alarm* alarm;
+    QPtrList<Alarm> list(mClickedItem->incidence()->alarms());
+    for(alarm=list.first();alarm;alarm=list.next()) {
+        alarm->toggleAlarm();
+    }
+    emit itemModified( mClickedItem , KOGlobals::EVENTEDITED );
+    mClickedItem->paintMe( true );
+    mClickedItem->repaint( false );
 }