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

List:       kde-commits
Subject:    branches/KDE/4.1/kdeedu/kstars/kstars
From:       Akarsh Simha <akarshsimha () gmail ! com>
Date:       2009-01-05 17:52:12
Message-ID: 1231177932.592100.8491.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 906170 by asimha:

Backporting further fix to bug 179034. Quoting the original commit
log:

----

Fixing (further) problems with the FOV editor. The localized versions
would accept FOVs that were formatted in the localized format (60,00
in German, for instance) but wouldn't work. This was because the same
bug had to be fixed in two other places.

The FOV editor now seems to work correctly.

----

CCMAIL: kstars-devel@kde.org



 M  +2 -2      fovdialog.cpp  


--- branches/KDE/4.1/kdeedu/kstars/kstars/fovdialog.cpp #906169:906170
@@ -131,7 +131,7 @@
     NewFOV newfdlg( this );
 
     if ( newfdlg.exec() == QDialog::Accepted ) {
-        FOV *newfov = new FOV( newfdlg.ui->FOVName->text(), \
newfdlg.ui->FOVEdit->text().toDouble(), +        FOV *newfov = new FOV( \
newfdlg.ui->FOVName->text(), newfdlg.ui->FOVEdit->text().replace( \
                KGlobal::locale()->decimalSymbol(), "." ).toDouble(),
                                newfdlg.ui->ShapeBox->currentIndex(), \
newfdlg.ui->ColorButton->color().name() );  
         FOVList.append( newfov );
@@ -156,7 +156,7 @@
     newfdlg.slotUpdateFOV();
 
     if ( newfdlg.exec() == QDialog::Accepted ) {
-        FOV *newfov = new FOV( newfdlg.ui->FOVName->text(), \
newfdlg.ui->FOVEdit->text().toDouble(), +        FOV *newfov = new FOV( \
newfdlg.ui->FOVName->text(), newfdlg.ui->FOVEdit->text().replace( \
                KGlobal::locale()->decimalSymbol(), "." ).toDouble(),
                                newfdlg.ui->ShapeBox->currentIndex(), \
newfdlg.ui->ColorButton->color().name() );  
         fov->FOVListBox->currentItem()->setText( newfdlg.ui->FOVName->text() );


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

Configure | About | News | Add a list | Sponsored by KoreLogic