diff options
Diffstat (limited to 'perl-install/ugtk2.pm')
| -rw-r--r-- | perl-install/ugtk2.pm | 37 | 
1 files changed, 20 insertions, 17 deletions
| diff --git a/perl-install/ugtk2.pm b/perl-install/ugtk2.pm index 05e361253..09f877c9f 100644 --- a/perl-install/ugtk2.pm +++ b/perl-install/ugtk2.pm @@ -212,7 +212,7 @@ sub gtkpowerpack {  	#- attr value directly in the arg list (avoiding confusion between value 0 and Gtk::Label("0"). That can simplify some writings but  	#- this arg(s) MUST then be present...  	my (%attr, $attrs); -	ref($_[0]) eq 'HASH' || ref($_[0]) eq 'ARRAY' and $attrs = shift; +	member(ref($_[0]), qw(HASH ARRAY)) and $attrs = shift;  	foreach (@attributes_list) {  	    if (($default_attrs->{$_} || '') eq 'arg') {  		ref($_[0]) and internal_error "error in packing definition\n"; @@ -523,11 +523,11 @@ sub create_okcancel {          $bprev = $w->{cancel} = gtknew('Button', text => $cancel, clicked => $w->{cancel_clicked} ||                                      sub { log::l("default cancel_clicked"); undef $w->{retval}; Gtk2->main_quit });      } -    $w->{wizcancel} = gtknew('Button', text =>  ($::Wizard_skip ? N("Skip") : N("Cancel")), clicked => sub { die 'wizcancel' }) if $::isWizard && !$::isInstall && !$::Wizard_no_cancel;; +    $w->{wizcancel} = gtknew('Button', text =>  ($::Wizard_skip ? N("Skip") : N("Cancel")), clicked => sub { die 'wizcancel' }) if $::isWizard && !$::isInstall && !$::Wizard_no_cancel;      if (!defined $wm_is_kde) {          require any;          my $wm = any::running_window_manager(); -        $wm_is_kde = !$::isInstall && ($wm eq "kwin" || $wm eq "compiz" && fuzzy_pidofs(qr/\bkde-window-decorator\b/)) || 0; +        $wm_is_kde = !$::isInstall && (member($wm, qw(kwin compiz)) && fuzzy_pidofs(qr/\bkde-window-decorator\b/)) || 0;      }      my $f = sub { $w->{buttons}{$_[0][0]} = ref($_[0][0]) =~ /Gtk2::Button/ ?                      $_[0][0] : @@ -748,7 +748,7 @@ sub new {  	if_(!$::isInstall, icon_no_error => wm_icon()),  	if_($o->{transient}, transient_for => $o->{transient}),       ); -    push @::main_windows, $::main_window = $window->{real_window} if !$opts{do_not_track_main_window}; +    mygtk2::register_main_window($window->{real_window}) if !$opts{do_not_track_main_window};      $window->set_border_width(10) if !$window->{pop_it} && !$::noborderWhenEmbedded;      $o->{rwindow} = $o->{window} = $window; @@ -760,7 +760,6 @@ sub new {  sub main {      my ($o, $o_completed, $o_canceled) = @_; -    gtkset_mousecursor_normal();      $o->show;      mygtk2::main($o->{rwindow}, @@ -788,7 +787,6 @@ sub shrink_topwindow {      $o->{real_window}->signal_emit('size_allocate', Gtk2::Gdk::Rectangle->new(-1, -1, -1, -1));  }  sub exit { -    gtkset_mousecursor_normal(); #- for restoring a normal in any case      flush();      if ($::isStandalone) {          require standalone; @@ -1145,7 +1143,7 @@ sub ask_browse_tree_info_given_widgets {      $w->{tree}->signal_connect(key_press_event => sub {  	my $c = chr($_[1]->keyval & 0xff); -	if ($_[1]->keyval >= 0x100 ? $c eq "\r" || $c eq "\x8d" : $c eq ' ') { +	if ($_[1]->keyval >= 0x100 ? member($c, "\r", "\x8d") : $c eq ' ') {  	    $toggle->(0);  	}  	0; @@ -1154,7 +1152,10 @@ sub ask_browse_tree_info_given_widgets {      $w->{tree}->get_selection->signal_connect(changed => sub {  	my ($model, $iter) = $_[0]->get_selected;  	$model && $iter or return; -	Glib::Source->remove($idle) if $idle; +	if ($idle) { +	    Glib::Source->remove($idle); +	    undef $idle; +	}  	if (!$model->iter_has_child($iter)) {  	    $curr = $model->get($iter, 0); @@ -1310,7 +1311,7 @@ sub toggle_expansion {  # compatible with Combo since its API is quite nice.  package Gtk2::OptionMenu; -use common; +use MDK::Common;  # try to get combox <==> option menu mapping  sub set_popdown_strings { @@ -1356,7 +1357,7 @@ sub set_text {  package Gtk2::ComboBox; -use common; +use MDK::Common;  # try to get combox <==> option menu mapping  sub set_popdown_strings { @@ -1448,6 +1449,7 @@ sub new {      $darea->set_size_request(-1, $d_height);      $darea->modify_font(Gtk2::Pango::FontDescription->from_string("13"));      eval { $darea->{icon} = ugtk2::gtkcreate_pixbuf($icon) }; +    $darea->{icon} ||= ugtk2::gtkcreate_pixbuf(ugtk2::wm_icon());      my $blue_part = eval { gtknew('Pixbuf', file => 'banner-blue-part', flip => mygtk2::text_direction_rtl()) };      my $blue_width = $blue_part->get_width;      $darea->{text} = $text; @@ -1466,8 +1468,8 @@ sub new {                                 my $d_width = $darea->allocation->width;                                 my $x_blue = $is_rtl ? $d_width - $blue_width : 0;                                 my $x_icon = $is_rtl ? $d_width - 12 - $width : 12; -                               # here: 48 is the amount of white background in the blue background we wish to ignore: -                               my $x_text = $is_rtl ? $d_width - $blue_width + 48 - $darea->{txt_width} : $blue_width - 48; +                               # here: 10 is the amount of padding (white background) wish to add betwen text & blue background: +                               my $x_text = $is_rtl ? $d_width - $blue_width - 10 - $darea->{txt_width} : $blue_width + 10;                                 $darea->{layout_height} ||= second($darea->{layout}->get_pixel_size);                                 $blue_part->render_to_drawable($darea->window, $style->bg_gc('normal'),                                                                    0, 0, $x_blue, 0, -1, -1, 'none', 0, 0); @@ -1516,7 +1518,7 @@ sub calc_size {  sub GET_SIZE {    my ($cell, $widget, $_cell_area) = @_; -  my $layout = $cell->get_layout($widget); +  my $layout = get_layout($cell, $widget);    $layout->set_text($cell->get('label'));    return calc_size($cell, $layout); @@ -1540,7 +1542,7 @@ sub RENDER { # not that efficient...        : 'normal';    } -  my $layout = $cell->get_layout($widget); +  my $layout = get_layout($cell, $widget);    $layout->set_text($cell->get('label'));    my $is_rtl = lang::text_direction_rtl(); @@ -1605,8 +1607,8 @@ sub add {  sub show {      my ($self) = @_; # perl_checker: $self = Gtk2::Notify->new      my $info = $self->{queue}[0]; -    my $notification = Gtk2::Notify->new_with_status_icon($info->{title}, $info->{message}, undef, $self->{statusicon}); -    $notification->set_icon_from_pixbuf($info->{pixbuf}); +    my $notification = Gtk2::Notify->new($info->{title}, $info->{message}, $self->{statusicon}); +    $notification->set_icon_from_pixbuf($info->{pixbuf}) if $info->{pixbuf};      $notification->set_urgency($info->{urgency}) if $info->{urgency};      foreach my $a (@{$info->{actions} || []}) {          $notification->add_action( @@ -1622,7 +1624,8 @@ sub show {                                          $info->{timeout}->() if $info->{timeout};                                          $self->process_next;                                      }); -    eval { $notification->show }; +    # this makes net_applet to segfault (mga#9102): +    #eval { $notification->show };  }  1; | 
