apport-hackers team mailing list archive

[Thomas Bechtold <thomasbechtold@xxxxxxxxxxx>]

Thomas Bechtold has proposed merging lp:~toabctl/apport/fix-930562 into lp:apport.

Requested reviews:
  Apport upstream developers (apport-hackers)
Related bugs:
  Bug #930562 in apport (Ubuntu): "Problem report window can not be resized"
  https://bugs.launchpad.net/ubuntu/+source/apport/+bug/930562

For more details, see:
https://code.launchpad.net/~toabctl/apport/fix-930562/+merge/92670

in gtk/apport-gtk:on_expander_details_activate(self, widget), the expanded state was not correct set. this branch changes gtk/apport-gtk.ui to emit the signal after the expander is expanded.
-- 
https://code.launchpad.net/~toabctl/apport/fix-930562/+merge/92670
Your team Apport upstream developers is requested to review the proposed merge of lp:~toabctl/apport/fix-930562 into lp:apport.

=== modified file 'gtk/apport-gtk.ui'
--- gtk/apport-gtk.ui	2011-10-29 17:28:34 +0000
+++ gtk/apport-gtk.ui	2012-02-12 15:00:25 +0000
@@ -151,7 +151,7 @@
                 <property name="visible">True</property>
                 <property name="can_focus">True</property>
                 <property name="has_focus">True</property>
-                <signal name="activate" handler="on_expander_details_activate" swapped="no"/>
+                <signal name="activate" handler="on_expander_details_activate" after="yes" swapped="no"/>
                 <child>
                   <object class="GtkScrolledWindow" id="scrolledwindow1">
                     <property name="visible">True</property>