Code review comment for lp:~javier.collado/checkbox/bug411526

Revision history for this message
Marc Tardif (cr3) wrote :

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

« Back to merge proposal