← Back to team overview

elementaryart team mailing list archive

[Merge] lp:~evfool/marlin/addpadding into lp:marlin

 

Robert Roth has proposed merging lp:~evfool/marlin/addpadding into lp:marlin.

Requested reviews:
  The elementary Project (elementaryproject)
Related bugs:
  Bug #802051 in Marlin: "fix spacing around tabs in settings"
  https://bugs.launchpad.net/marlin/+bug/802051

For more details, see:
https://code.launchpad.net/~evfool/marlin/addpadding/+merge/66689

Add padding to settings dialog notebook, as suggested in bug #802051.
-- 
https://code.launchpad.net/~evfool/marlin/addpadding/+merge/66689
Your team The elementary Project is requested to review the proposed merge of lp:~evfool/marlin/addpadding into lp:marlin.
=== modified file 'src/View/SettingsDialog.vala'
--- src/View/SettingsDialog.vala	2011-06-01 15:52:03 +0000
+++ src/View/SettingsDialog.vala	2011-07-02 11:35:47 +0000
@@ -228,8 +228,10 @@
 
 
             mai_notebook.append_page(first_vbox, new Gtk.Label(_("Plugins")));
-
-            ((Gtk.Box)get_content_area()).pack_start(mai_notebook);
+            Gtk.Alignment alignment = new Gtk.Alignment(0.5f, 0.5f, 1.0f, 1.0f);
+            alignment.set_padding(6, 6, 6, 6);
+            alignment.add(mai_notebook);
+            ((Gtk.Box)get_content_area()).pack_start(alignment);
 
             this.show_all();
 


Follow ups