[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    [Kde-pim] KORganizer "Go Forward / Backward" functions
From:       Kevin Gilbert <kev.gilbert () cdu ! edu ! au>
Date:       2005-10-26 22:53:21
Message-ID: 200510270823.07736.kev.gilbert () cdu ! edu ! au
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Hi,

I believe there is an inconsistency in the way KORganizer handles the "Go 
Forward / Backward" functions. All is fine for the Day, Work Week, Week and 
Month views. The problem is in the "Next 3 Days" view. Here the dates are 
advanced by one week.

I find this a bit of a pain and believe that the increment / decrement should 
match the view interval, ie, the view should scroll by +/- 3 days. The 
downside of this is that it is _really_ is three days, eg, Wed/Thu/Fri + 3 = 
Sat/Sun/Mon. However, I think this is in line with the idea of the "Next 3 
Days".

I recognise that this is a design question so there may be good reasons for 
the current behaviour. But, if not, the following patch will scroll the view 
by +/- 3 days.


====

--- datenavigator.cpp   2005-09-10 17:54:47.000000000 +0930
+++ /usr/local/src/kde/kde-3.5-beta-2/kdepim-3.4.92/korganizer/datenavigator.cpp        
2005-10-26 18:15:54.000000000 +0930
@@ -218,6 +218,11 @@
   if ( datesCount() == 1 ) {
     offset = -1;
   }
+  else
+  if ( datesCount() == 3 ) {
+    offset = -3;
+  }
+

   selectDates( mSelectedDates.first().addDays( offset ), datesCount() );
 }
@@ -228,6 +233,10 @@
   if ( datesCount() == 1 ) {
     offset = 1;
   }
+  else
+  if ( datesCount() == 3 ) {
+    offset = 3;
+  }

   selectDates( mSelectedDates.first().addDays( offset ), datesCount() );
 }

====

Cheers,

Kevin

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kde-pim mailing list
kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/

[prev in list] [next in list] [prev in thread] [next in thread]