Changeset 10848

Timestamp:

02/20/10 01:13:11 (2 years ago)

Author:

azawawi

Message:

More UI fixes for the true/false radio buttons in the advanced preferences

File:

• trunk/Padre/lib/Padre/Wx/Dialog/Advanced.pm (modified) (2 diffs)

trunk/Padre/lib/Padre/Wx/Dialog/Advanced.pm

@@ -389 +389 @@
     my $is_boolean = ($pref->{type} == Padre::Constant::BOOLEAN) ? 1 : 0;
     if($is_boolean) {
-        $self->{true}->SetValue( $pref->{value} ? 1 : 0 );
-        $self->{false}->SetValue( $pref->{value} ? 0 : 1 );
+        $self->{true}->SetValue( $pref->{value} );
+        $self->{false}->SetValue( not $pref->{value} );
     } else {
         $self->{value}->SetValue( $self->_displayed_value( $type, $pref->{value} ) );
     }
-    $self->{value}->Show($is_boolean ? 0 : 1);
-    $self->{true}->Show($is_boolean ? 1 : 0);
-    $self->{false}->Show($is_boolean ? 1 : 0);
+    $self->{value}->Show(not $is_boolean);
+    $self->{true}->Show($is_boolean);
+    $self->{false}->Show($is_boolean);
     $self->Layout;
     
@@ -449 +449 @@
     my $is_default = $pref->{is_default};
 
-    $self->{value}->SetValue( $self->_displayed_value( $type, $value ) );
+    if($type == Padre::Constant::BOOLEAN) {
+        $self->{true}->SetValue( $value );
+        $self->{false}->SetValue( not $value );
+    } else {
+        $self->{value}->SetValue( $self->_displayed_value( $type, $value ) );
+    }
     $self->{default_value}->SetLabel( $self->_displayed_value( $type, $value ) );
     $self->{button_reset}->Enable( not $is_default );