Bugfix: (bug #0000811) Events are now always atleast 12 pixels high in dayview.

1 files changed, 44 insertions, 44 deletions

diff --git a/core/pim/datebook/datebookday.cpp b/core/pim/datebook/datebookday.cpp
index af98e06..db4c2fd 100644
--- a/core/pim/datebook/datebookday.cpp
+++ b/core/pim/datebook/datebookday.cpp
@@ -548,151 +548,151 @@ void DateBookDay::slotWeekChanged( bool bStartOnMonday )
 void DateBookDay::keyPressEvent(QKeyEvent *e)
 {
     switch(e->key()) {
         case Key_Up:
             view->moveUp();
             break;
         case Key_Down:
             view->moveDown();
             break;
         case Key_Left:
             setDate(QDate(currDate).addDays(-1));
             break;
         case Key_Right:
             setDate(QDate(currDate).addDays(1));
             break;
         default:
             e->ignore();
     }
 }
 
 //===========================================================================
 
 DateBookDayWidget::DateBookDayWidget( const EffectiveEvent &e,
                                       DateBookDay *db )
     : QWidget( db->dayView()->viewport() ), ev( e ), dateBook( db )
 {
 
 
     // why would someone use "<"?  Oh well, fix it up...
     // I wonder what other things may be messed up...
     QString strDesc = ev.description();
     int where = strDesc.find( "<" );
     while ( where != -1 ) {
         strDesc.remove( where, 1 );
         strDesc.insert( where, "&#60;" );
         where = strDesc.find( "<", where );
     }
 
     QString strCat;
     // ### Fix later...
 //     QString strCat = ev.category();
 //     where = strCat.find( "<" );
 //     while ( where != -1 ) {
         // strCat.remove( where, 1 );
         // strCat.insert( where, "&#60;" );
         // where = strCat.find( "<", where );
 //     }
 
-    QString strNote = ev.notes();
-    where = strNote.find( "<" );
-    while ( where != -1 ) {
-        strNote.remove( where, 1 );
-        strNote.insert( where, "&#60;" );
-        where = strNote.find( "<", where );
-    }
+        QString strNote = ev.notes();
+        where = strNote.find( "<" );
+        while ( where != -1 ) {
+                strNote.remove( where, 1 );
+                strNote.insert( where, "&#60;" );
+                where = strNote.find( "<", where );
+        }
 
-    text = "<b>" + strDesc + "</b><br>" + "<i>";
-    if ( !strCat.isEmpty() )  {
-    text += strCat + "</i><br>";
-    }
-    if (ev.event().type() == Event::Normal )
-        setEventText( text );
-    else
-        setAllDayText( text );
+        text = "<b>" + strDesc + "</b><br>" + "<i>";
+        if ( !strCat.isEmpty() )  {
+                text += strCat + "</i><br>";
+        }
+        if (ev.event().type() == Event::Normal )
+                setEventText( text );
+        else
+                setAllDayText( text );
 
     text += "<br><br>" + strNote;
 
     setBackgroundMode( PaletteBase );
 
-    QTime start = ev.start();
-    QTime end = ev.end();
-    int y = start.hour()*60+start.minute();
-    int h = end.hour()*60+end.minute()-y;
-    int rh = dateBook->dayView()->rowHeight(0);
-    y = y*rh/60;
-    h = h*rh/60;
-    if ( h < 3 )  {
-        h = 3;
-    }
-    geom.setY( y );
-    geom.setHeight( h );
-    geom.setX( 0 );
-    geom.setWidth(dateBook->dayView()->columnWidth(0)-1);
+        QTime start = ev.start();
+        QTime end = ev.end();
+        int y = start.hour()*60+start.minute();
+        int h = end.hour()*60+end.minute()-y;
+        int rh = dateBook->dayView()->rowHeight(0);
+        y = y*rh/60;
+        h = h*rh/60;
+
+                if ( h < 12 ) h = 12;// Make sure the widget is no smaller than 12 pixels high, so that it's possible to read atleast the first line.
+                if ( y > ((24*rh)-12) ) y=(24*rh)-12;// Make sure the widget fits inside the dayview.
+        geom.setY( y );
+        geom.setHeight( h );
+        geom.setX( 0 );
+        geom.setWidth(dateBook->dayView()->columnWidth(0)-1);
 
 }
 void DateBookDayWidget::setAllDayText( QString &text ) {
-    text += "<b>" + tr("This is an all day event.") + "</b><br>";
+    text += "<b>" + tr("This is an all day event.") + "</b>";
 }
 void DateBookDayWidget::setEventText( QString& text ) {
-    bool whichClock = dateBook->dayView()->whichClock();
-    if ( ev.startDate() != ev.endDate() ) {
-        text += "<b>" + tr("Start") + "</b>: ";
-        text += TimeString::timeString( ev.event().start().time(), whichClock, FALSE );
-        text += " - " + TimeString::longDateString( ev.startDate() ) + "<br>";
-        text += "<b>" + tr("End") + "</b>: ";
-        text += TimeString::timeString( ev.event().end().time(), whichClock, FALSE );
-        text += " - " + TimeString::longDateString( ev.endDate() ) + "<br>";
-    } else {
-        text += "<b>" + tr("Time") + "</b>: ";
-        text += TimeString::timeString( ev.start(), whichClock, FALSE );
-        text += "<b>" + tr(" - ") + "</b>";
-        text += TimeString::timeString( ev.end(), whichClock, FALSE );
-    }
+        bool whichClock = dateBook->dayView()->whichClock();
+        if ( ev.startDate() != ev.endDate() ) {
+                text += "<b>" + tr("Start") + "</b>: ";
+                text += TimeString::timeString( ev.event().start().time(), whichClock, FALSE );
+                text += " - " + TimeString::longDateString( ev.startDate() ) + "<br>";
+                text += "<b>" + tr("End") + "</b>: ";
+                text += TimeString::timeString( ev.event().end().time(), whichClock, FALSE );
+                text += " - " + TimeString::longDateString( ev.endDate() );
+        } else {
+                text += "<b>" + tr("Time") + "</b>: ";
+                text += TimeString::timeString( ev.start(), whichClock, FALSE );
+                text += "<b>" + tr(" - ") + "</b>";
+                text += TimeString::timeString( ev.end(), whichClock, FALSE );
+        }
 }
 
 DateBookDayWidget::~DateBookDayWidget()
 {
 }
 
 void DateBookDayWidget::paintEvent( QPaintEvent *e )
 {
     QPainter p( this );
 
     if (dateBook->getSelectedWidget() == this)
     {
                    p.setBrush( QColor( 155, 240, 230 ) ); // selected item
     } else
     {
             if (dateBook->date() == QDate::currentDate())
         {
                 QTime curTime = QTime::currentTime();
 
             if (ev.end() < curTime)
             {
                       p.setBrush( QColor( 180, 180, 180 ) ); // grey, inactive
             } else
             {
                     //change color in dependence of the time till the event starts
                    int duration = curTime.secsTo(ev.start());
                   if (duration < 0) duration = 0;
                   int colChange = duration*160/86400; //86400: secs per day, 160: max color shift
 
                       p.setBrush( QColor( 200-colChange, 200-colChange, 255 ) ); //blue
             }
          } else
          {
                   p.setBrush( QColor( 220, 220, 220 ) ); //light grey, inactive (not current date)
                   //perhaps make a distinction between future/past dates
          }
     }
 
     p.setPen( QColor(100, 100, 100) );
     p.drawRect(rect());
 
  //   p.drawRect(0,0, 5, height());
 
     int y = 0;
     int d = 0;
 
     if ( ev.event().hasAlarm() ) {
         p.drawPixmap( width() - 16, 0, Resource::loadPixmap( "bell" ) );