Merge lp:~javier.collado/checkbox/bug411526 into lp:checkbox

[Javier Collado]

[Javier Collado (javier.collado)]

This branch solves bug 411526 (Suites prompt should support more than 13 check boxes)

The solution is to create a new vbox that contains all the checkbox buttons and wrap it with a viewport and a scrolled window; so that when the list of test suites doesn't fit in the window, it can be scrolled.

All the changes to the widget hierarchy are performed using code instead of changing the glade file since I think is better not to modify it at least until bug#403534 is solved.

[David Murphy (schwuk)]

review: Approve

[Marc Tardif (cr3)]

It would be more consistent if the layout were defined in the glade xml file, even though this will eventually be migrated to gtkbuilder. Also, the user experience might become painful if they have to scroll both vertically and horizontally. So, I will disapprove this branch and I have pushed branch lp:~cr3/checkbox/411526 which I will submit for your review in a moment.

Thanks for the patch, it has been instrumental in creating the branch I just pushed!

review: Disapprove

[Marc Tardif (cr3)]

See comment made above.

review: Disapprove

Unmerged revisions

691. By Javier Collado on 2009-11-03

ScrolledWindow is created for checkboxes so that they can be scrolled when their size is bigger than the top level window

=== modified file 'checkbox_gtk/gtk_interface.py'
--- checkbox_gtk/gtk_interface.py	2009-10-12 15:46:11 +0000
+++ checkbox_gtk/gtk_interface.py	2009-11-03 11:45:25 +0000
@@ -263,7 +263,11 @@
 
         # Set options
         option_table = {}
-        vbox = self._get_widget("vbox_options")
+
+        vbox_options = self._get_widget("vbox_options")
+
+        vbox = gtk.VBox()
+
         for option in options:
             label = "_%s" % option.capitalize()
             radio_button = gtk.CheckButton(label)
@@ -274,6 +278,15 @@
                 radio_button.set_active(True)
 
         self._set_hyper_text_view("hyper_text_view_options", text)
+        scrolled_text = self._get_widget("scrolledwindow_options")
+        vbox_options.set_child_packing(scrolled_text, False, False, 0, gtk.PACK_START)
+
+        scrolled = gtk.ScrolledWindow()
+        scrolled.set_policy(gtk.POLICY_AUTOMATIC, gtk.POLICY_AUTOMATIC)
+        scrolled.add_with_viewport(vbox)
+
+        vbox_options.pack_start(scrolled)
+        scrolled.show_all()
 
         self._run_dialog()
 
@@ -282,7 +295,7 @@
         for option, check_button in option_table.items():
             if check_button.get_active():
                 results.append(option)
-            vbox.remove(check_button)
+        scrolled.destroy()
 
         return results