Added runtime detection of Qt version, because all qt/e's <= 2.3.3 return

font sizes *10

2 files changed, 118 insertions, 95 deletions

diff --git a/libopie/ofontselector.cpp b/libopie/ofontselector.cpp
index 39440af..e1090b5 100644
--- a/libopie/ofontselector.cpp
+++ b/libopie/ofontselector.cpp
@@ -32,8 +32,21 @@
 #include <qlabel.h>
 #include <qfont.h>
 
+#include <qpe/fontdatabase.h>
+
 #include "ofontselector.h"
 
+class OFontSelectorPrivate {
+public:
+        QListBox *   m_font_family_list;
+        QComboBox *  m_font_style_list;
+        QComboBox *  m_font_size_list;
+
+        bool         m_pointbug;
+
+        FontDatabase m_fdb;
+};
+
 
 class FontListItem : public QListBoxText {
 public:
@@ -80,74 +93,84 @@ static int findItemCB ( QComboBox *box, const QString &str )
 }
 
 
+static int qt_version ( )
+{
+        const char *qver = qVersion ( );
+
+        return ( qver [0] - '0' ) * 100 + ( qver [2] - '0' ) * 10 + ( qver [4] - '0' );
+}
+
 
 OFontSelector::OFontSelector ( QWidget *parent, const char *name, WFlags fl ) : QWidget ( parent, name, fl )
 {
-    QGridLayout *gridLayout = new QGridLayout ( this, 0, 0, 4, 4 );
-    gridLayout->setRowStretch ( 4, 10 );
+        d = new OFontSelectorPrivate ( );
+
+        QGridLayout *gridLayout = new QGridLayout ( this, 0, 0, 4, 4 );
+        gridLayout->setRowStretch ( 4, 10 );
 
-    m_font_family_list = new QListBox( this, "FontListBox" );
-    gridLayout->addMultiCellWidget( m_font_family_list, 0, 4, 0, 0 );
-    connect( m_font_family_list, SIGNAL( highlighted( int ) ), this, SLOT( fontFamilyClicked( int ) ) );
+        d-> m_font_family_list = new QListBox( this, "FontListBox" );
+        gridLayout->addMultiCellWidget( d-> m_font_family_list, 0, 4, 0, 0 );
+        connect( d-> m_font_family_list, SIGNAL( highlighted( int ) ), this, SLOT( fontFamilyClicked( int ) ) );
 
-    QLabel *label = new QLabel( tr( "Style" ), this );
-    gridLayout->addWidget( label, 0, 1 );
+        QLabel *label = new QLabel( tr( "Style" ), this );
+        gridLayout->addWidget( label, 0, 1 );
 
-    m_font_style_list = new QComboBox( this, "StyleListBox" );
-    connect( m_font_style_list, SIGNAL(  activated( int ) ), this, SLOT( fontStyleClicked( int ) ) );
-    gridLayout->addWidget( m_font_style_list, 1, 1 );
+        d-> m_font_style_list = new QComboBox( this, "StyleListBox" );
+        connect( d-> m_font_style_list, SIGNAL(  activated( int ) ), this, SLOT( fontStyleClicked( int ) ) );
+        gridLayout->addWidget( d-> m_font_style_list, 1, 1 );
 
-    label = new QLabel( tr( "Size" ), this );
-    gridLayout->addWidget( label, 2, 1 );
+        label = new QLabel( tr( "Size" ), this );
+        gridLayout->addWidget( label, 2, 1 );
 
-    m_font_size_list = new QComboBox( this, "SizeListBox" );
-    connect( m_font_size_list, SIGNAL(  activated( int ) ),
-             this, SLOT( fontSizeClicked( int ) ) );
-    gridLayout->addWidget( m_font_size_list, 3, 1 );
+        d-> m_font_size_list = new QComboBox( this, "SizeListBox" );
+        connect( d-> m_font_size_list, SIGNAL(  activated( int ) ),
+                         this, SLOT( fontSizeClicked( int ) ) );
+        gridLayout->addWidget( d-> m_font_size_list, 3, 1 );
 
-        loadFonts ( m_font_family_list );
+        d-> m_pointbug = ( qt_version ( ) <= 233 );
+
+        loadFonts ( d-> m_font_family_list );
 }
 
 OFontSelector::~OFontSelector ( )
 {
+        delete d;
 }
 
 bool OFontSelector::setSelectedFont ( const QFont &f )
 {
-        return setSelectedFont ( f. family ( ), m_fdb. styleString ( f ), f. pointSize ( ), QFont::encodingName ( f. charSet ( )));
+        return setSelectedFont ( f. family ( ), d-> m_fdb. styleString ( f ), f. pointSize ( ), QFont::encodingName ( f. charSet ( )));
 }
 
 bool OFontSelector::setSelectedFont ( const QString &familyStr, const QString &styleStr, int sizeVal, const QString & /*charset*/ )
 {
         QString sizeStr = QString::number ( sizeVal );
 
-        QListBoxItem *family = m_font_family_list-> findItem ( familyStr );
+        QListBoxItem *family = d-> m_font_family_list-> findItem ( familyStr );
         if ( !family )
-                family = m_font_family_list-> findItem ( "Helvetica" );
+                family = d-> m_font_family_list-> findItem ( "Helvetica" );
         if ( !family )
-                family = m_font_family_list-> firstItem ( );
-            m_font_family_list-> setCurrentItem ( family );    
-    fontFamilyClicked ( m_font_family_list-> index ( family ));
+                family = d-> m_font_family_list-> firstItem ( );
+                 d-> m_font_family_list-> setCurrentItem ( family );
+        fontFamilyClicked ( d-> m_font_family_list-> index ( family ));
 
-        int style = findItemCB ( m_font_style_list, styleStr );
+        int style = findItemCB ( d-> m_font_style_list, styleStr );
         if ( style < 0 )
-                style = findItemCB ( m_font_style_list, "Regular" );
-        if ( style < 0 && m_font_style_list-> count ( ) > 0 )
+                style = findItemCB ( d-> m_font_style_list, "Regular" );
+        if ( style < 0 && d-> m_font_style_list-> count ( ) > 0 )
                 style = 0;
-        m_font_style_list-> setCurrentItem ( style );
+        d-> m_font_style_list-> setCurrentItem ( style );
         fontStyleClicked ( style );
 
-        int size = findItemCB ( m_font_size_list, sizeStr );
+        int size = findItemCB ( d-> m_font_size_list, sizeStr );
         if ( size < 0 )
-                size = findItemCB ( m_font_size_list, "10" );
-        if ( size < 0 && m_font_size_list-> count ( ) > 0 )
+                size = findItemCB ( d-> m_font_size_list, "10" );
+        if ( size < 0 && d-> m_font_size_list-> count ( ) > 0 )
                 size = 0;
-        m_font_size_list-> setCurrentItem ( size );
+        d-> m_font_size_list-> setCurrentItem ( size );
         fontSizeClicked ( size );
         
-        return (( family ) &&
-                ( style >= 0 ) &&
-                ( size >= 0 ));
+        return (( family ) && ( style >= 0 ) && ( size >= 0 ));
 }
 
 bool OFontSelector::selectedFont ( QString &family, QString &style, int &size )
@@ -159,47 +182,47 @@ bool OFontSelector::selectedFont ( QString &family, QString &style, int &size )
 
 QString OFontSelector::fontFamily ( ) const
 {
-        FontListItem *fli = (FontListItem *) m_font_family_list-> item ( m_font_family_list-> currentItem ( ));
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( d-> m_font_family_list-> currentItem ( ));
         
         return fli ? fli-> family ( ) : QString::null;
 }
 
 QString OFontSelector::fontStyle ( ) const
 {
-        FontListItem *fli = (FontListItem *) m_font_family_list-> item ( m_font_family_list-> currentItem ( ));
-    int fst = m_font_style_list-> currentItem ( ); 
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( d-> m_font_family_list-> currentItem ( ));
+        int fst = d-> m_font_style_list-> currentItem ( ); 
 
         return ( fli && fst >= 0 ) ? fli-> styles ( ) [fst] : QString::null;
 }
 
 int OFontSelector::fontSize ( ) const
 {
-        FontListItem *fli = (FontListItem *) m_font_family_list-> item ( m_font_family_list-> currentItem ( ));
-    int fsi = m_font_size_list-> currentItem ( ); 
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( d-> m_font_family_list-> currentItem ( ));
+        int fsi = d-> m_font_size_list-> currentItem ( ); 
 
         return ( fli && fsi >= 0 ) ? fli-> sizes ( ) [fsi] : 10;
 }
 
 QString OFontSelector::fontCharSet ( ) const
 {
-        FontListItem *fli = (FontListItem *) m_font_family_list-> item ( m_font_family_list-> currentItem ( ));
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( d-> m_font_family_list-> currentItem ( ));
 
-        return fli ? m_fdb. charSets ( fli-> family ( )) [0] : QString::null;
+        return fli ? d-> m_fdb. charSets ( fli-> family ( )) [0] : QString::null;
 }
 
 bool OFontSelector::selectedFont ( QString &family, QString &style, int &size, QString &charset )
 {
-    int ffa = m_font_family_list-> currentItem ( );
-    int fst = m_font_style_list-> currentItem ( ); 
-    int fsi = m_font_size_list-> currentItem ( );
-    
-    FontListItem *fli = (FontListItem *) m_font_family_list-> item ( ffa );
-    
-    if ( fli ) {
-            family = fli-> family ( );
-            style = fst >= 0 ? fli-> styles ( ) [fst] : QString::null;
-            size = fsi >= 0 ? fli-> sizes ( ) [fsi] : 10;
-            charset = m_fdb. charSets ( fli-> family ( )) [0];
+        int ffa = d-> m_font_family_list-> currentItem ( );
+        int fst = d-> m_font_style_list-> currentItem ( ); 
+        int fsi = d-> m_font_size_list-> currentItem ( );
+        
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( ffa );
+        
+        if ( fli ) {
+                family = fli-> family ( );
+                style = fst >= 0 ? fli-> styles ( ) [fst] : QString::null;
+                size = fsi >= 0 ? fli-> sizes ( ) [fsi] : 10;
+                charset = d-> m_fdb. charSets ( fli-> family ( )) [0];
 
                 return true;
         }
@@ -208,54 +231,62 @@ bool OFontSelector::selectedFont ( QString &family, QString &style, int &size, Q
 }
 
 
-            
+                        
 
 void OFontSelector::loadFonts ( QListBox *list )
 {
-        QStringList f = m_fdb. families ( );
+        QStringList f = d-> m_fdb. families ( );
         
-        for ( QStringList::ConstIterator it = f. begin ( ); it != f. end ( ); ++it )
-                list-> insertItem ( new FontListItem ( *it, m_fdb. styles ( *it ), m_fdb. pointSizes ( *it )));
+        for ( QStringList::ConstIterator it = f. begin ( ); it != f. end ( ); ++it ) {
+                QValueList <int> ps = d-> m_fdb. pointSizes ( *it );
+                
+                if ( d-> m_pointbug ) {
+                        for ( QValueList <int>::Iterator it = ps. begin ( ); it != ps. end ( ); it++ )
+                                *it /= 10;
+                }
+        
+                list-> insertItem ( new FontListItem ( *it, d-> m_fdb. styles ( *it ), ps ));
+        }
 }
 
 void OFontSelector::fontFamilyClicked ( int index )
 {
-        QString oldstyle = m_font_style_list-> currentText ( );
-        QString oldsize  = m_font_size_list-> currentText ( );
+        QString oldstyle = d-> m_font_style_list-> currentText ( );
+        QString oldsize  = d-> m_font_size_list-> currentText ( );
         
-    FontListItem *fli = (FontListItem *) m_font_family_list-> item ( index );
-    
-        m_font_style_list-> clear ( );    
-        m_font_style_list-> insertStringList ( fli-> styles ( ));
-        m_font_style_list-> setEnabled ( !fli-> styles ( ). isEmpty ( ));
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( index );
+        
+                d-> m_font_style_list-> clear ( );
+        d-> m_font_style_list-> insertStringList ( fli-> styles ( ));
+        d-> m_font_style_list-> setEnabled ( !fli-> styles ( ). isEmpty ( ));
 
         int i;
                 
-        i = findItemCB ( m_font_style_list, oldstyle );
+        i = findItemCB ( d-> m_font_style_list, oldstyle );
         if ( i < 0 )
-                i = findItemCB ( m_font_style_list, "Regular" );
-        if (( i < 0 ) && ( m_font_style_list-> count ( ) > 0 ))
+                i = findItemCB ( d-> m_font_style_list, "Regular" );
+        if (( i < 0 ) && ( d-> m_font_style_list-> count ( ) > 0 ))
                 i = 0;
                 
         if ( i >= 0 ) {
-                        m_font_style_list-> setCurrentItem ( i );       
+                        d-> m_font_style_list-> setCurrentItem ( i );   
                 fontStyleClicked ( i );
         }
         
-        m_font_size_list-> clear ( );
+        d-> m_font_size_list-> clear ( );
         QValueList<int> sl = fli-> sizes ( );
         
         for ( QValueList<int>::Iterator it = sl. begin ( ); it != sl. end ( ); ++it ) 
-                m_font_size_list-> insertItem ( QString::number ( *it ));
+                d-> m_font_size_list-> insertItem ( QString::number ( *it ));
 
-        i = findItemCB ( m_font_size_list, oldsize );
+        i = findItemCB ( d-> m_font_size_list, oldsize );
         if ( i < 0 )
-                i = findItemCB ( m_font_size_list, "10" );
-        if (( i < 0 ) && ( m_font_size_list-> count ( ) > 0 ))
+                i = findItemCB ( d-> m_font_size_list, "10" );
+        if (( i < 0 ) && ( d-> m_font_size_list-> count ( ) > 0 ))
                 i = 0;
                 
         if ( i >= 0 ) {
-                        m_font_size_list-> setCurrentItem ( i );                                
+                        d-> m_font_size_list-> setCurrentItem ( i );                                                            
                 fontSizeClicked ( i );
         }
                 changeFont ( );
@@ -279,21 +310,18 @@ void OFontSelector::changeFont ( )
 
 QFont OFontSelector::selectedFont ( )
 {
-    int ffa = m_font_family_list-> currentItem ( );
-    int fst = m_font_style_list-> currentItem ( ); 
-    int fsi = m_font_size_list-> currentItem ( );
-    
-    FontListItem *fli = (FontListItem *) m_font_family_list-> item ( ffa );
-    
-    if ( fli ) {
-                return m_fdb. font ( fli-> family ( ), \
-                                     fst >= 0 ? fli-> styles ( ) [fst] : QString::null, \
-                                     fsi >= 0 ? fli-> sizes ( ) [fsi] : 10, \
-                                     m_fdb. charSets ( fli-> family ( )) [0] );
+        int ffa = d-> m_font_family_list-> currentItem ( );
+        int fst = d-> m_font_style_list-> currentItem ( ); 
+        int fsi = d-> m_font_size_list-> currentItem ( );
+        
+        FontListItem *fli = (FontListItem *) d-> m_font_family_list-> item ( ffa );
+        
+        if ( fli ) {
+                return d-> m_fdb. font ( fli-> family ( ), \
+                                         fst >= 0 ? fli-> styles ( ) [fst] : QString::null, \
+                                         fsi >= 0 ? fli-> sizes ( ) [fsi] : 10, \
+                                         d-> m_fdb. charSets ( fli-> family ( )) [0] );
         }
         else
                 return QFont ( );
 }
-
-
-                
\ No newline at end of file
diff --git a/libopie/ofontselector.h b/libopie/ofontselector.h
index b3aa862..a9c8a72 100644
--- a/libopie/ofontselector.h
+++ b/libopie/ofontselector.h
@@ -30,11 +30,10 @@
 #define __OPIE_FONTSELECTOR_H__
 
 #include <qwidget.h>
-#include <qpe/fontdatabase.h>
 
-class QComboBox;
 class QListBox;
-class QFont;
+class OFontSelectorPrivate;
+
 
 class OFontSelector : public QWidget
 {
@@ -71,11 +70,7 @@ private:
     void changeFont ( );
         
 private:
-    QListBox *    m_font_family_list;
-    QComboBox *   m_font_style_list;
-    QComboBox *   m_font_size_list;
-    
-    FontDatabase  m_fdb;
+        OFontSelectorPrivate *d;
 };
 
 #endif