[Sugar-devel] [PATCH shell 2/2] Use proper constructor for Gtk.Alignment

Manuel Quiñones manuq at laptop.org
Wed Sep 19 23:10:46 EDT 2012


Is better to pass keyword arguments, like the toolkit does.  See
sugar-toolkit-gtk commit 20319cb3c4b2be4e8f584703d704ffda4b1cff29 .

Signed-off-by: Manuel Quiñones <manuq at laptop.org>
---
 extensions/cpsection/aboutme/view.py | 4 ++--
 extensions/cpsection/updater/view.py | 3 +--
 extensions/deviceicon/network.py     | 6 +++---
 src/jarabe/desktop/activitieslist.py | 3 ++-
 src/jarabe/intro/window.py           | 2 +-
 src/jarabe/journal/detailview.py     | 2 +-
 src/jarabe/journal/expandedentry.py  | 6 +++---
 src/jarabe/journal/listview.py       | 6 +++---
 src/jarabe/journal/modalalert.py     | 3 +--
 src/jarabe/view/launcher.py          | 2 +-
 10 files changed, 18 insertions(+), 19 deletions(-)

diff --git a/extensions/cpsection/aboutme/view.py b/extensions/cpsection/aboutme/view.py
index 293923d..a17d41a 100644
--- a/extensions/cpsection/aboutme/view.py
+++ b/extensions/cpsection/aboutme/view.py
@@ -204,7 +204,7 @@ class AboutMe(SectionView):
             self._nick_alert.props.msg = self.restart_msg
             self._nick_alert.show()
 
-        self._center_in_panel = Gtk.Alignment.new(0.5, 0, 0, 0)
+        self._center_in_panel = Gtk.Alignment(xalign=0.5)
         self._center_in_panel.add(self._nick_box)
         self.pack_start(self._center_in_panel, False, False, 0)
         self.pack_start(self._nick_alert_box, False, False, 0)
@@ -246,7 +246,7 @@ class AboutMe(SectionView):
             self._color_alert.props.msg = self.restart_msg
             self._color_alert.show()
 
-        self._center_in_panel = Gtk.Alignment.new(0.5, 0, 0, 0)
+        self._center_in_panel = Gtk.Alignment(xalign=0.5)
         self._center_in_panel.add(self._color_box)
         self.pack_start(self._color_label, False, False, 0)
         self.pack_start(self._center_in_panel, False, False, 0)
diff --git a/extensions/cpsection/updater/view.py b/extensions/cpsection/updater/view.py
index 2a1f643..64867d2 100644
--- a/extensions/cpsection/updater/view.py
+++ b/extensions/cpsection/updater/view.py
@@ -202,8 +202,7 @@ class ProgressPane(Gtk.VBox):
         self.pack_start(self._label, True, True, 0)
         self._label.show()
 
-        alignment_box = Gtk.Alignment.new(xalign=0.5, yalign=0.5,
-                                          xscale=0, yscale=0)
+        alignment_box = Gtk.Alignment(xalign=0.5, yalign=0.5)
         self.pack_start(alignment_box, True, True, 0)
         alignment_box.show()
 
diff --git a/extensions/deviceicon/network.py b/extensions/deviceicon/network.py
index b4a5393..855a514 100644
--- a/extensions/deviceicon/network.py
+++ b/extensions/deviceicon/network.py
@@ -89,7 +89,7 @@ class WirelessPalette(Palette):
         self._info.pack_start(separator, True, True, 0)
 
         def _padded(child, xalign=0, yalign=0.5):
-            padder = Gtk.Alignment.new(xalign=xalign, yalign=yalign,
+            padder = Gtk.Alignment(xalign=xalign, yalign=yalign,
                                    xscale=1, yscale=0.33)
             padder.set_padding(style.DEFAULT_SPACING,
                                style.DEFAULT_SPACING,
@@ -162,7 +162,7 @@ class WiredPalette(Palette):
         self._info = Gtk.VBox()
 
         def _padded(child, xalign=0, yalign=0.5):
-            padder = Gtk.Alignment.new(xalign=xalign, yalign=yalign,
+            padder = Gtk.Alignment(xalign=xalign, yalign=yalign,
                                    xscale=1, yscale=0.33)
             padder.set_padding(style.DEFAULT_SPACING,
                                style.DEFAULT_SPACING,
@@ -256,7 +256,7 @@ class GsmPalette(Palette):
         self.set_state(_GSM_STATE_NOT_READY)
 
     def _add_widget_with_padding(self, child, xalign=0, yalign=0.5):
-        alignment = Gtk.Alignment.new(xalign=xalign, yalign=yalign,
+        alignment = Gtk.Alignment(xalign=xalign, yalign=yalign,
                                   xscale=1, yscale=0.33)
         alignment.set_padding(style.DEFAULT_SPACING,
                               style.DEFAULT_SPACING,
diff --git a/src/jarabe/desktop/activitieslist.py b/src/jarabe/desktop/activitieslist.py
index b111164..2d3e9a8 100644
--- a/src/jarabe/desktop/activitieslist.py
+++ b/src/jarabe/desktop/activitieslist.py
@@ -311,7 +311,8 @@ class ClearMessageBox(Gtk.EventBox):
         self.modify_bg(Gtk.StateType.NORMAL,
                        style.COLOR_WHITE.get_gdk_color())
 
-        alignment = Gtk.Alignment.new(0.5, 0.5, 0.1, 0.1)
+        alignment = Gtk.Alignment(xalign=0.5, yalign=0.5, xscale=0.1,
+                                  yscale=0.1)
         self.add(alignment)
         alignment.show()
 
diff --git a/src/jarabe/intro/window.py b/src/jarabe/intro/window.py
index 3c979fb..b10d51e 100644
--- a/src/jarabe/intro/window.py
+++ b/src/jarabe/intro/window.py
@@ -98,7 +98,7 @@ class _NamePage(_Page):
         _Page.__init__(self)
         self._intro = intro
 
-        alignment = Gtk.Alignment.new(0.5, 0.5, 0, 0)
+        alignment = Gtk.Alignment(xalign=0.5, yalign=0.5)
         self.pack_start(alignment, expand=True, fill=True, padding=0)
 
         hbox = Gtk.HBox(spacing=style.DEFAULT_SPACING)
diff --git a/src/jarabe/journal/detailview.py b/src/jarabe/journal/detailview.py
index c7748ed..f1b5c18 100644
--- a/src/jarabe/journal/detailview.py
+++ b/src/jarabe/journal/detailview.py
@@ -92,7 +92,7 @@ class BackBar(Gtk.EventBox):
 
         label = Gtk.Label()
         label.set_text(_('Back'))
-        halign = Gtk.Alignment.new(0, 0.5, 0, 1)
+        halign = Gtk.Alignment(yalign=0.5, yscale=1)
         halign.add(label)
         hbox.pack_start(halign, True, True, 0)
         hbox.show()
diff --git a/src/jarabe/journal/expandedentry.py b/src/jarabe/journal/expandedentry.py
index 21a3c5f..1650df7 100644
--- a/src/jarabe/journal/expandedentry.py
+++ b/src/jarabe/journal/expandedentry.py
@@ -280,7 +280,7 @@ class ExpandedEntry(Gtk.EventBox):
         text = Gtk.Label()
         text.set_markup('<span foreground="%s">%s</span>' % (
                 style.COLOR_BUTTON_GREY.get_html(), label))
-        halign = Gtk.Alignment.new(0, 0, 0, 0)
+        halign = Gtk.Alignment()
         halign.add(text)
         vbox.pack_start(halign, False, False, 0)
 
@@ -306,7 +306,7 @@ class ExpandedEntry(Gtk.EventBox):
         text = Gtk.Label()
         text.set_markup('<span foreground="%s">%s</span>' % (
                 style.COLOR_BUTTON_GREY.get_html(), _('Participants:')))
-        halign = Gtk.Alignment.new(0, 0, 0, 0)
+        halign = Gtk.Alignment()
         halign.add(text)
         vbox.pack_start(halign, False, False, 0)
 
@@ -325,7 +325,7 @@ class ExpandedEntry(Gtk.EventBox):
         text.set_markup('<span foreground="%s">%s</span>' % (
                 style.COLOR_BUTTON_GREY.get_html(), label))
 
-        halign = Gtk.Alignment.new(0, 0, 0, 0)
+        halign = Gtk.Alignment()
         halign.add(text)
         vbox.pack_start(halign, False, False, 0)
 
diff --git a/src/jarabe/journal/listview.py b/src/jarabe/journal/listview.py
index 369f364..f91cff1 100644
--- a/src/jarabe/journal/listview.py
+++ b/src/jarabe/journal/listview.py
@@ -357,8 +357,7 @@ class BaseListView(Gtk.Bin):
             self._last_progress_bar_pulse = time.time()
 
     def _start_progress_bar(self):
-        alignment = Gtk.Alignment.new(xalign=0.5, yalign=0.5,
-                                      xscale=0.5, yscale=0)
+        alignment = Gtk.Alignment(xalign=0.5, yalign=0.5, xscale=0.5)
         self.remove(self.get_child())
         self.add(alignment)
         alignment.show()
@@ -384,7 +383,8 @@ class BaseListView(Gtk.Bin):
                                  style.COLOR_WHITE.get_gdk_color())
         self.add(background_box)
 
-        alignment = Gtk.Alignment.new(0.5, 0.5, 0.1, 0.1)
+        alignment = Gtk.Alignment(xalign=0.5, yalign=0.5, xscale=0.1,
+                                  yscale=0.1)
         background_box.add(alignment)
 
         box = Gtk.VBox()
diff --git a/src/jarabe/journal/modalalert.py b/src/jarabe/journal/modalalert.py
index db7a81f..bc48548 100644
--- a/src/jarabe/journal/modalalert.py
+++ b/src/jarabe/journal/modalalert.py
@@ -74,8 +74,7 @@ class ModalAlert(Gtk.Window):
                               fill=False, padding=0)
         self._message.show()
 
-        alignment = Gtk.Alignment.new(xalign=0.5, yalign=0.5,
-                                      xscale=0.0, yscale=0.0)
+        alignment = Gtk.Alignment(xalign=0.5, yalign=0.5)
         self._vbox.pack_start(alignment, expand=False, fill=True, padding=0)
         alignment.show()
 
diff --git a/src/jarabe/view/launcher.py b/src/jarabe/view/launcher.py
index aae07d2..f0fe523 100644
--- a/src/jarabe/view/launcher.py
+++ b/src/jarabe/view/launcher.py
@@ -68,7 +68,7 @@ class LaunchWindow(Gtk.Window):
         self.error_text.props.use_markup = True
         footer.pack_start(self.error_text, False, True, 0)
 
-        button_box = Gtk.Alignment.new(0.5, 0, 0, 0)
+        button_box = Gtk.Alignment(xalign=0.5)
         button_box.show()
         footer.pack_start(button_box, False, True, 0)
         self.cancel_button = Gtk.Button(stock=Gtk.STOCK_STOP)
-- 
1.7.11.4



More information about the Sugar-devel mailing list