Merge lp:~jbicha/indicator-datetime/replace-deprecated-ui into lp:indicator-datetime/0.3

Proposed by Jeremy Bícha
Status: Work in progress
Proposed branch: lp:~jbicha/indicator-datetime/replace-deprecated-ui
Merge into: lp:indicator-datetime/0.3
Diff against target: 328 lines (+50/-51)
2 files modified
data/datetime-dialog.ui (+39/-40)
src/indicator-datetime.c (+11/-11)
To merge this branch: bzr merge lp:~jbicha/indicator-datetime/replace-deprecated-ui
Reviewer Review Type Date Requested Status
Michael Terry Needs Fixing
Review via email: mp+72997@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

To make the list button look right, you should also set the "inline-toolbar" style class for the toolbar. See gnome-control-center panels for example .ui xml for it. I don't know whether glade exposes it or not.

- <!-- column-name Visible Name -->
+ <!-- column-name Visible -->

Why this change?

- <object class="GtkVBox" id="vbox4">
+ <object class="GtkBox" id="box9">

This is missing its vertical orientation setting.

- GtkWidget * hbox = gtk_hbox_new(FALSE, 4);
+ GtkWidget * box = gtk_box_new(FALSE, 4);

FALSE here should be GTK_ORIENTATION_HORIZONTAL. Same for similar changes.

review: Needs Fixing

Unmerged revisions

125. By Jeremy Bícha

Use GNOME 3-style +/-
Replace GtkHBox & GtkVBox with GtkBox

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/datetime-dialog.ui'
--- data/datetime-dialog.ui 2011-08-17 19:30:53 +0000
+++ data/datetime-dialog.ui 2011-08-26 06:11:23 +0000
@@ -1,7 +1,6 @@
1<?xml version="1.0" encoding="UTF-8"?>1<?xml version="1.0" encoding="UTF-8"?>
2<interface>2<interface>
3 <requires lib="gtk+" version="2.24"/>3 <requires lib="gtk+" version="2.24"/>
4 <!-- interface-naming-policy project-wide -->
5 <object class="GtkAdjustment" id="dateAdjustment">4 <object class="GtkAdjustment" id="dateAdjustment">
6 <property name="upper">1.8446744073709552e+19</property>5 <property name="upper">1.8446744073709552e+19</property>
7 <property name="step_increment">86400</property>6 <property name="step_increment">86400</property>
@@ -15,15 +14,14 @@
15 <property name="destroy_with_parent">True</property>14 <property name="destroy_with_parent">True</property>
16 <property name="icon_name">time-admin</property>15 <property name="icon_name">time-admin</property>
17 <child>16 <child>
18 <object class="GtkVBox" id="vbox1">17 <object class="GtkBox" id="box10">
19 <property name="visible">True</property>18 <property name="visible">True</property>
20 <property name="can_focus">False</property>19 <property name="can_focus">False</property>
20 <property name="orientation">vertical</property>
21 <child>21 <child>
22 <object class="GtkScrolledWindow" id="scrolledwindow1">22 <object class="GtkScrolledWindow" id="scrolledwindow1">
23 <property name="visible">True</property>23 <property name="visible">True</property>
24 <property name="can_focus">True</property>24 <property name="can_focus">True</property>
25 <property name="hscrollbar_policy">automatic</property>
26 <property name="vscrollbar_policy">automatic</property>
27 <child>25 <child>
28 <object class="GtkTreeView" id="locationsView">26 <object class="GtkTreeView" id="locationsView">
29 <property name="visible">True</property>27 <property name="visible">True</property>
@@ -31,6 +29,9 @@
31 <property name="model">locationsStore</property>29 <property name="model">locationsStore</property>
32 <property name="reorderable">True</property>30 <property name="reorderable">True</property>
33 <property name="search_column">0</property>31 <property name="search_column">0</property>
32 <child internal-child="selection">
33 <object class="GtkTreeSelection" id="treeview-selection1"/>
34 </child>
34 </object>35 </object>
35 </child>36 </child>
36 </object>37 </object>
@@ -52,57 +53,48 @@
52 </packing>53 </packing>
53 </child>54 </child>
54 <child>55 <child>
55 <object class="GtkHBox" id="hbox10">56 <object class="GtkToolbar" id="toolbar1">
56 <property name="visible">True</property>57 <property name="visible">True</property>
57 <property name="can_focus">False</property>58 <property name="can_focus">False</property>
59 <property name="icon_size">1</property>
58 <child>60 <child>
59 <object class="GtkButton" id="addButton">61 <object class="GtkToolButton" id="addButton">
60 <property name="visible">True</property>62 <property name="visible">True</property>
61 <property name="can_focus">True</property>63 <property name="can_focus">True</property>
62 <property name="receives_default">True</property>64 <property name="receives_default">True</property>
63 <property name="use_action_appearance">False</property>65 <property name="use_action_appearance">False</property>
66 <property name="label" translatable="yes">Add</property>
67 <property name="use_underline">True</property>
68 <property name="icon_name">list-add-symbolic</property>
64 <child internal-child="accessible">69 <child internal-child="accessible">
65 <object class="AtkObject" id="addButton-atkobject">70 <object class="AtkObject" id="addButton-atkobject">
66 <property name="AtkObject::accessible-description" translatable="yes">Add a Location…</property>71 <property name="AtkObject::accessible-description" translatable="yes">Add a Location…</property>
67 </object>72 </object>
68 </child>73 </child>
69 <child>
70 <object class="GtkImage" id="addImage">
71 <property name="visible">True</property>
72 <property name="can_focus">False</property>
73 <property name="stock">gtk-add</property>
74 </object>
75 </child>
76 </object>74 </object>
77 <packing>75 <packing>
78 <property name="expand">False</property>76 <property name="expand">False</property>
79 <property name="fill">True</property>77 <property name="homogeneous">True</property>
80 <property name="position">0</property>
81 </packing>78 </packing>
82 </child>79 </child>
83 <child>80 <child>
84 <object class="GtkButton" id="removeButton">81 <object class="GtkToolButton" id="removeButton">
85 <property name="visible">True</property>82 <property name="visible">True</property>
86 <property name="can_focus">True</property>83 <property name="can_focus">True</property>
87 <property name="receives_default">True</property>84 <property name="receives_default">True</property>
88 <property name="use_action_appearance">False</property>85 <property name="use_action_appearance">False</property>
86 <property name="label" translatable="yes">Remove</property>
87 <property name="use_underline">True</property>
88 <property name="icon_name">list-remove-symbolic</property>
89 <child internal-child="accessible">89 <child internal-child="accessible">
90 <object class="AtkObject" id="removeButton-atkobject">90 <object class="AtkObject" id="removeButton-atkobject">
91 <property name="AtkObject::accessible-description" translatable="yes">Remove This Location</property>91 <property name="AtkObject::accessible-description" translatable="yes">Remove This Location</property>
92 </object>92 </object>
93 </child>93 </child>
94 <child>
95 <object class="GtkImage" id="removeImage">
96 <property name="visible">True</property>
97 <property name="can_focus">False</property>
98 <property name="stock">gtk-remove</property>
99 </object>
100 </child>
101 </object>94 </object>
102 <packing>95 <packing>
103 <property name="expand">False</property>96 <property name="expand">False</property>
104 <property name="fill">True</property>97 <property name="homogeneous">True</property>
105 <property name="position">1</property>
106 </packing>98 </packing>
107 </child>99 </child>
108 </object>100 </object>
@@ -123,7 +115,7 @@
123 <column type="gchararray"/>115 <column type="gchararray"/>
124 <!-- column-name Zone -->116 <!-- column-name Zone -->
125 <column type="gchararray"/>117 <column type="gchararray"/>
126 <!-- column-name Visible Name -->118 <!-- column-name Visible -->
127 <column type="gchararray"/>119 <column type="gchararray"/>
128 <!-- column-name Icon -->120 <!-- column-name Icon -->
129 <column type="gchararray"/>121 <column type="gchararray"/>
@@ -142,20 +134,23 @@
142 <property name="visible">True</property>134 <property name="visible">True</property>
143 <property name="can_focus">True</property>135 <property name="can_focus">True</property>
144 <child>136 <child>
145 <object class="GtkVBox" id="timeDateBox">137 <object class="GtkBox" id="timeDateBox">
146 <property name="visible">True</property>138 <property name="visible">True</property>
147 <property name="can_focus">False</property>139 <property name="can_focus">False</property>
148 <property name="border_width">12</property>140 <property name="border_width">12</property>
141 <property name="orientation">vertical</property>
149 <property name="spacing">6</property>142 <property name="spacing">6</property>
150 <child>143 <child>
151 <object class="GtkVBox" id="timeDateOptions">144 <object class="GtkBox" id="timeDateOptions">
152 <property name="visible">True</property>145 <property name="visible">True</property>
153 <property name="can_focus">False</property>146 <property name="can_focus">False</property>
147 <property name="orientation">vertical</property>
154 <property name="spacing">12</property>148 <property name="spacing">12</property>
155 <child>149 <child>
156 <object class="GtkVBox" id="vbox6">150 <object class="GtkBox" id="box1">
157 <property name="visible">True</property>151 <property name="visible">True</property>
158 <property name="can_focus">False</property>152 <property name="can_focus">False</property>
153 <property name="orientation">vertical</property>
159 <property name="spacing">6</property>154 <property name="spacing">6</property>
160 <child>155 <child>
161 <object class="GtkAspectFrame" id="mapBox">156 <object class="GtkAspectFrame" id="mapBox">
@@ -175,7 +170,7 @@
175 </packing>170 </packing>
176 </child>171 </child>
177 <child>172 <child>
178 <object class="GtkHBox" id="hbox1">173 <object class="GtkBox" id="box2">
179 <property name="visible">True</property>174 <property name="visible">True</property>
180 <property name="can_focus">False</property>175 <property name="can_focus">False</property>
181 <property name="spacing">6</property>176 <property name="spacing">6</property>
@@ -230,7 +225,7 @@
230 <property name="column_spacing">6</property>225 <property name="column_spacing">6</property>
231 <property name="row_spacing">6</property>226 <property name="row_spacing">6</property>
232 <child>227 <child>
233 <object class="GtkHBox" id="hbox2">228 <object class="GtkBox" id="box3">
234 <property name="visible">True</property>229 <property name="visible">True</property>
235 <property name="can_focus">False</property>230 <property name="can_focus">False</property>
236 <property name="spacing">6</property>231 <property name="spacing">6</property>
@@ -290,12 +285,12 @@
290 </packing>285 </packing>
291 </child>286 </child>
292 <child>287 <child>
293 <object class="GtkHBox" id="manualOptions">288 <object class="GtkBox" id="manualOptions">
294 <property name="visible">True</property>289 <property name="visible">True</property>
295 <property name="can_focus">False</property>290 <property name="can_focus">False</property>
296 <property name="spacing">12</property>291 <property name="spacing">12</property>
297 <child>292 <child>
298 <object class="GtkHBox" id="hbox8">293 <object class="GtkBox" id="box4">
299 <property name="visible">True</property>294 <property name="visible">True</property>
300 <property name="can_focus">False</property>295 <property name="can_focus">False</property>
301 <property name="spacing">6</property>296 <property name="spacing">6</property>
@@ -323,7 +318,7 @@
323 </packing>318 </packing>
324 </child>319 </child>
325 <child>320 <child>
326 <object class="GtkHBox" id="hbox9">321 <object class="GtkBox" id="box5">
327 <property name="visible">True</property>322 <property name="visible">True</property>
328 <property name="can_focus">False</property>323 <property name="can_focus">False</property>
329 <property name="spacing">6</property>324 <property name="spacing">6</property>
@@ -428,10 +423,11 @@
428 <property name="yalign">0</property>423 <property name="yalign">0</property>
429 <property name="yscale">0</property>424 <property name="yscale">0</property>
430 <child>425 <child>
431 <object class="GtkVBox" id="clockBox">426 <object class="GtkBox" id="clockBox">
432 <property name="visible">True</property>427 <property name="visible">True</property>
433 <property name="can_focus">False</property>428 <property name="can_focus">False</property>
434 <property name="border_width">12</property>429 <property name="border_width">12</property>
430 <property name="orientation">vertical</property>
435 <property name="spacing">12</property>431 <property name="spacing">12</property>
436 <child>432 <child>
437 <object class="GtkCheckButton" id="showClockCheck">433 <object class="GtkCheckButton" id="showClockCheck">
@@ -451,14 +447,15 @@
451 </packing>447 </packing>
452 </child>448 </child>
453 <child>449 <child>
454 <object class="GtkHBox" id="clockOptions">450 <object class="GtkBox" id="clockOptions">
455 <property name="visible">True</property>451 <property name="visible">True</property>
456 <property name="can_focus">False</property>452 <property name="can_focus">False</property>
457 <property name="homogeneous">True</property>453 <property name="homogeneous">True</property>
458 <child>454 <child>
459 <object class="GtkVBox" id="vbox2">455 <object class="GtkBox" id="box6">
460 <property name="visible">True</property>456 <property name="visible">True</property>
461 <property name="can_focus">False</property>457 <property name="can_focus">False</property>
458 <property name="orientation">vertical</property>
462 <property name="spacing">6</property>459 <property name="spacing">6</property>
463 <child>460 <child>
464 <object class="GtkLabel" id="label3">461 <object class="GtkLabel" id="label3">
@@ -508,9 +505,10 @@
508 </packing>505 </packing>
509 </child>506 </child>
510 <child>507 <child>
511 <object class="GtkVBox" id="vbox5">508 <object class="GtkBox" id="box7">
512 <property name="visible">True</property>509 <property name="visible">True</property>
513 <property name="can_focus">False</property>510 <property name="can_focus">False</property>
511 <property name="orientation">vertical</property>
514 <child>512 <child>
515 <object class="GtkRadioButton" id="show12HourRadio">513 <object class="GtkRadioButton" id="show12HourRadio">
516 <property name="label" translatable="yes">_12-hour time</property>514 <property name="label" translatable="yes">_12-hour time</property>
@@ -579,9 +577,10 @@
579 </packing>577 </packing>
580 </child>578 </child>
581 <child>579 <child>
582 <object class="GtkVBox" id="vbox3">580 <object class="GtkBox" id="box8">
583 <property name="visible">True</property>581 <property name="visible">True</property>
584 <property name="can_focus">False</property>582 <property name="can_focus">False</property>
583 <property name="orientation">vertical</property>
585 <property name="spacing">6</property>584 <property name="spacing">6</property>
586 <child>585 <child>
587 <object class="GtkLabel" id="label4">586 <object class="GtkLabel" id="label4">
@@ -622,7 +621,7 @@
622 <property name="yscale">0</property>621 <property name="yscale">0</property>
623 <property name="left_padding">24</property>622 <property name="left_padding">24</property>
624 <child>623 <child>
625 <object class="GtkVBox" id="vbox4">624 <object class="GtkBox" id="box9">
626 <property name="visible">True</property>625 <property name="visible">True</property>
627 <property name="can_focus">False</property>626 <property name="can_focus">False</property>
628 <property name="spacing">6</property>627 <property name="spacing">6</property>
629628
=== modified file 'src/indicator-datetime.c'
--- src/indicator-datetime.c 2011-08-17 15:50:43 +0000
+++ src/indicator-datetime.c 2011-08-26 06:11:23 +0000
@@ -1260,7 +1260,7 @@
12601260
1261 mi_data->gmi = gtk_menu_item_new();1261 mi_data->gmi = gtk_menu_item_new();
12621262
1263 GtkWidget * hbox = gtk_hbox_new(FALSE, 4);1263 GtkWidget * box = gtk_box_new(FALSE, 4);
12641264
1265 /* Icon, probably someone's face or avatar on an IM */1265 /* Icon, probably someone's face or avatar on an IM */
1266 mi_data->icon = gtk_image_new();1266 mi_data->icon = gtk_image_new();
@@ -1292,7 +1292,7 @@
1292 }1292 }
1293 }1293 }
1294 gtk_misc_set_alignment(GTK_MISC(mi_data->icon), 0.0, 0.5);1294 gtk_misc_set_alignment(GTK_MISC(mi_data->icon), 0.0, 0.5);
1295 gtk_box_pack_start(GTK_BOX(hbox), mi_data->icon, FALSE, FALSE, 0);1295 gtk_box_pack_start(GTK_BOX(box), mi_data->icon, FALSE, FALSE, 0);
1296 gtk_widget_show(mi_data->icon);1296 gtk_widget_show(mi_data->icon);
12971297
1298 /* Label, probably a username, chat room or mailbox name */1298 /* Label, probably a username, chat room or mailbox name */
@@ -1305,7 +1305,7 @@
1305 gtk_widget_set_size_request(GTK_WIDGET(mi_data->label), length, -1); // Set the min size in pixels1305 gtk_widget_set_size_request(GTK_WIDGET(mi_data->label), length, -1); // Set the min size in pixels
1306 1306
1307 gtk_label_set_ellipsize(GTK_LABEL(mi_data->label), PANGO_ELLIPSIZE_END);1307 gtk_label_set_ellipsize(GTK_LABEL(mi_data->label), PANGO_ELLIPSIZE_END);
1308 gtk_box_pack_start(GTK_BOX(hbox), mi_data->label, TRUE, TRUE, 0);1308 gtk_box_pack_start(GTK_BOX(box), mi_data->label, TRUE, TRUE, 0);
1309 gtk_widget_show(mi_data->label);1309 gtk_widget_show(mi_data->label);
13101310
1311 /* Usually either the time or the count on the individual1311 /* Usually either the time or the count on the individual
@@ -1313,11 +1313,11 @@
1313 mi_data->right = gtk_label_new(dbusmenu_menuitem_property_get(newitem, APPOINTMENT_MENUITEM_PROP_RIGHT));1313 mi_data->right = gtk_label_new(dbusmenu_menuitem_property_get(newitem, APPOINTMENT_MENUITEM_PROP_RIGHT));
1314 gtk_size_group_add_widget(self->priv->indicator_right_group, mi_data->right);1314 gtk_size_group_add_widget(self->priv->indicator_right_group, mi_data->right);
1315 gtk_misc_set_alignment(GTK_MISC(mi_data->right), 1.0, 0.5);1315 gtk_misc_set_alignment(GTK_MISC(mi_data->right), 1.0, 0.5);
1316 gtk_box_pack_start(GTK_BOX(hbox), mi_data->right, FALSE, FALSE, 0);1316 gtk_box_pack_start(GTK_BOX(box), mi_data->right, FALSE, FALSE, 0);
1317 gtk_widget_show(mi_data->right);1317 gtk_widget_show(mi_data->right);
13181318
1319 gtk_container_add(GTK_CONTAINER(mi_data->gmi), hbox);1319 gtk_container_add(GTK_CONTAINER(mi_data->gmi), box);
1320 gtk_widget_show(hbox);1320 gtk_widget_show(box);
13211321
1322 dbusmenu_gtkclient_newitem_base(DBUSMENU_GTKCLIENT(client), newitem, GTK_MENU_ITEM(mi_data->gmi), parent);1322 dbusmenu_gtkclient_newitem_base(DBUSMENU_GTKCLIENT(client), newitem, GTK_MENU_ITEM(mi_data->gmi), parent);
13231323
@@ -1470,12 +1470,12 @@
1470 gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(mi_data->gmi),1470 gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(mi_data->gmi),
1471 dbusmenu_menuitem_property_get_bool(newitem, TIMEZONE_MENUITEM_PROP_RADIO));1471 dbusmenu_menuitem_property_get_bool(newitem, TIMEZONE_MENUITEM_PROP_RADIO));
14721472
1473 GtkWidget * hbox = gtk_hbox_new(FALSE, 4);1473 GtkWidget * box = gtk_box_new(FALSE, 4);
14741474
1475 /* Label, probably a username, chat room or mailbox name */1475 /* Label, probably a username, chat room or mailbox name */
1476 mi_data->label = gtk_label_new("");1476 mi_data->label = gtk_label_new("");
1477 gtk_misc_set_alignment(GTK_MISC(mi_data->label), 0.0, 0.5);1477 gtk_misc_set_alignment(GTK_MISC(mi_data->label), 0.0, 0.5);
1478 gtk_box_pack_start(GTK_BOX(hbox), mi_data->label, TRUE, TRUE, 0);1478 gtk_box_pack_start(GTK_BOX(box), mi_data->label, TRUE, TRUE, 0);
1479 gtk_widget_show(mi_data->label);1479 gtk_widget_show(mi_data->label);
14801480
1481 /* Usually either the time or the count on the individual1481 /* Usually either the time or the count on the individual
@@ -1483,13 +1483,13 @@
1483 mi_data->right = gtk_label_new("");1483 mi_data->right = gtk_label_new("");
1484 gtk_size_group_add_widget(self->priv->indicator_right_group, mi_data->right);1484 gtk_size_group_add_widget(self->priv->indicator_right_group, mi_data->right);
1485 gtk_misc_set_alignment(GTK_MISC(mi_data->right), 1.0, 0.5);1485 gtk_misc_set_alignment(GTK_MISC(mi_data->right), 1.0, 0.5);
1486 gtk_box_pack_start(GTK_BOX(hbox), mi_data->right, FALSE, FALSE, 0);1486 gtk_box_pack_start(GTK_BOX(box), mi_data->right, FALSE, FALSE, 0);
1487 gtk_widget_show(mi_data->right);1487 gtk_widget_show(mi_data->right);
14881488
1489 timezone_update_labels(mi_data);1489 timezone_update_labels(mi_data);
14901490
1491 gtk_container_add(GTK_CONTAINER(mi_data->gmi), hbox);1491 gtk_container_add(GTK_CONTAINER(mi_data->gmi), box);
1492 gtk_widget_show(hbox);1492 gtk_widget_show(box);
14931493
1494 dbusmenu_gtkclient_newitem_base(DBUSMENU_GTKCLIENT(client), newitem, GTK_MENU_ITEM(mi_data->gmi), parent);1494 dbusmenu_gtkclient_newitem_base(DBUSMENU_GTKCLIENT(client), newitem, GTK_MENU_ITEM(mi_data->gmi), parent);
14951495

Subscribers

People subscribed via source and target branches