Changeset 10848

Show
Ignore:
Timestamp:
02/20/10 01:13:11 (6 months ago)
Author:
azawawi
Message:

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

Files:
1 modified

Legend:

Unmodified
Added
Removed

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

    r10847 r10848  
    389389        my $is_boolean = ($pref->{type} == Padre::Constant::BOOLEAN) ? 1 : 0; 
    390390        if($is_boolean) { 
    391                 $self->{true}->SetValue( $pref->{value} ? 1 : 0 ); 
    392                 $self->{false}->SetValue( $pref->{value} ? 0 : 1 ); 
     391                $self->{true}->SetValue( $pref->{value} ); 
     392                $self->{false}->SetValue( not $pref->{value} ); 
    393393        } else { 
    394394                $self->{value}->SetValue( $self->_displayed_value( $type, $pref->{value} ) ); 
    395395        } 
    396         $self->{value}->Show($is_boolean ? 0 : 1); 
    397         $self->{true}->Show($is_boolean ? 1 : 0); 
    398         $self->{false}->Show($is_boolean ? 1 : 0); 
     396        $self->{value}->Show(not $is_boolean); 
     397        $self->{true}->Show($is_boolean); 
     398        $self->{false}->Show($is_boolean); 
    399399        $self->Layout; 
    400400         
    … …  
    449449        my $is_default = $pref->{is_default}; 
    450450 
    451         $self->{value}->SetValue( $self->_displayed_value( $type, $value ) ); 
     451        if($type == Padre::Constant::BOOLEAN) { 
     452                $self->{true}->SetValue( $value ); 
     453                $self->{false}->SetValue( not $value ); 
     454        } else { 
     455                $self->{value}->SetValue( $self->_displayed_value( $type, $value ) ); 
     456        } 
    452457        $self->{default_value}->SetLabel( $self->_displayed_value( $type, $value ) ); 
    453458        $self->{button_reset}->Enable( not $is_default );