jumpbar fixes

1 files changed, 31 insertions, 16 deletions

diff --git a/kaddressbook/jumpbuttonbar.cpp b/kaddressbook/jumpbuttonbar.cpp
index e89ec52..32922a6 100644
--- a/kaddressbook/jumpbuttonbar.cpp
+++ b/kaddressbook/jumpbuttonbar.cpp
@@ -64,27 +64,27 @@ void JumpButton::setCharacter( const QString &character )
 }
 
 QString JumpButton::character() const
 {
   return mCharacter;
 }
 
 JumpButtonBar::JumpButtonBar( KABCore *core, QWidget *parent, const char *name )
   : QWidget( parent, name ), mCore( core )
 {
     if ( QApplication::desktop()->width() < 480 )
         
-        mButtonLayout = new QGridLayout( this,  2, 14 );
+        mButtonLayout = new QGridLayout( this,  1, 18 );
     else
-        mButtonLayout = new QGridLayout( this,  1, 28 );
+        mButtonLayout = new QGridLayout( this,  1, 20 );
   mButtonLayout->setAlignment( Qt::AlignTop );
 
   recreateButtons();
 }
 
 JumpButtonBar::~JumpButtonBar()
 {
 }
 
 QSizePolicy JumpButtonBar::sizePolicy() const
 {
 #ifndef KAB_EMBEDDED  
@@ -132,68 +132,83 @@ void JumpButtonBar::recreateButtons()
       if ( character != "!" )
           if ( !character.isEmpty() && !mCharacters.contains( character )  )
           mCharacters.append( character );
   }
   if ( mCharacters.count() == 0 ) {
       setEnabled( false );
       return;
   }
   
   int maxRows = mCharacters.count() / 2; // we use 2 columns
   if ( mCharacters.count() % 2 )
     maxRows++;
-  int fixwid = 20;
   sortListLocaleAware( mCharacters );
   bool skip2 = false;
   int skipcount = 0;
-  int maxHei = 25;
-  int roW = 15;//13 is ok
-  if ( QApplication::desktop()->width() < 480 && mCharacters.count() > roW ) {
-      skipcount = mCharacters.count()-roW ;
-      maxHei = (QApplication::desktop()->height()-65)/roW ;
-  }
-  else {
-      fixwid = 30;
-      if ( mCharacters.count() > 16 )
-          skipcount = mCharacters.count()- 16;
-      maxHei = (QApplication::desktop()->height()-120)/(16);
+  int maxHei = 0;
+#ifdef DESKTOP_VERSION
+  int maxChar = 26;
+  int heightDiff = 200;
+  int fixwid = 0;
+#else
+  //default for 240x320 diaplay:
+  int maxChar = 16;
+  int heightDiff = 64;
+  int fixwid = 20;
 
+  if ( QApplication::desktop()->height() == 480 ) {
+      maxChar = 16;
+      heightDiff = 135;
+      fixwid = 34;
+  } else if ( QApplication::desktop()->height() >= 640 ) {
+      maxChar = 20;
+      heightDiff = 135;
+      fixwid = 34;
   }
+#endif
+
+  if ( mCharacters.count() > maxChar )
+      skipcount = mCharacters.count()- maxChar;
+  maxHei = (QApplication::desktop()->height()-heightDiff)/(maxChar);
+
+
      maxRows = 28;
      bool skipcurrent = false;
   bool state = isUpdatesEnabled();
   setUpdatesEnabled( false );
   //qDebug("cc %d ",mCharacters.count() );
   JumpButton *button = 0;
   int row = 0, col = 0;
   JumpButton* cur = mButtons.first();
   for ( uint i = 0; i < mCharacters.count(); ++i ) {
       if ( skipcount > 0 && skipcurrent ) {
           --skipcount;
           if ( button ) {
               button->setCharacter( button->character() + mCharacters[ i ]);
           }
       } else {
           if ( cur ) {
               button = cur ;
               cur = mButtons.next();
               button->setCharacter(mCharacters[ i ]);
           } else {
               button = new JumpButton( mCharacters[ i ].upper(), this, mCharacters[ i ] );
-              button->setFixedWidth( fixwid );
+              if ( fixwid )
+                  button->setFixedWidth( fixwid );
               mButtons.append( button );
               connect( button, SIGNAL( clicked() ), this, SLOT( letterClicked() ) );
               mButtonLayout->addWidget( button, row, col );
           }
-          button->setMaximumHeight( maxHei );
+          if ( maxHei  )
+              button->setMaximumHeight( maxHei );
           button->show();
           
           if ( col == maxRows ) {
               row = 0;
               col++;
           } else
               row++;
       }
       if ( i > 0 )
           skipcurrent = !skipcurrent;
   }
   while ( cur ) {