Merge lp:~evfool/software-center/specs-update into lp:software-center/5.2

Proposed by Robert Roth
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp:~evfool/software-center/specs-update
Merge into: lp:software-center/5.2
Diff against target: 58 lines (+14/-14)
1 file modified
softwarecenter/ui/gtk3/views/appdetailsview.py (+14/-14)
To merge this branch: bzr merge lp:~evfool/software-center/specs-update
Reviewer Review Type Date Requested Status
Michael Vogt Approve
Review via email: mp+106242@code.launchpad.net

Description of the change

This branch contains the following changes (based on the specification https://wiki.ubuntu.com/SoftwareCenter#Software_item_screen) to the application details view
* Moved the recommended applications section after the package details
* Moved the addon status bar after the addons list
* Aligned the star ratings widget vertically with the titles
* Added some vertical padding before the star ratings widget to look better

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks, I merge that into trunk.

review: Approve
Revision history for this message
Michael Vogt (mvo) wrote :

This is merged into trunk now.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'softwarecenter/ui/gtk3/views/appdetailsview.py'
--- softwarecenter/ui/gtk3/views/appdetailsview.py 2012-05-14 03:16:07 +0000
+++ softwarecenter/ui/gtk3/views/appdetailsview.py 2012-05-17 19:05:21 +0000
@@ -1168,9 +1168,9 @@
1168 self.review_stats_widget = StarRatingsWidget()1168 self.review_stats_widget = StarRatingsWidget()
1169 self.review_stats = Gtk.HBox()1169 self.review_stats = Gtk.HBox()
1170 vb_inner.pack_start(1170 vb_inner.pack_start(
1171 self.review_stats, False, False, 0)1171 self.review_stats, False, False, StockEms.SMALL)
1172 self.review_stats.pack_start(1172 self.review_stats.pack_start(
1173 self.review_stats_widget, False, False, StockEms.SMALL)1173 self.review_stats_widget, False, False, 0)
11741174
1175 #~ vb_inner.set_property("can-focus", True)1175 #~ vb_inner.set_property("can-focus", True)
1176 self.title.a11y = vb_inner.get_accessible()1176 self.title.a11y = vb_inner.get_accessible()
@@ -1264,23 +1264,12 @@
1264 info_vb.pack_start(self.addon_view, False, False, 0)1264 info_vb.pack_start(self.addon_view, False, False, 0)
12651265
1266 self.addons_statusbar = self.addons_manager.status_bar1266 self.addons_statusbar = self.addons_manager.status_bar
1267 self.addon_view.pack_start(self.addons_statusbar, False, False, 0)1267 self.addon_view.pack_end(self.addons_statusbar, False, False, 0)
1268 self.addon_view.connect('table-built', self._on_addon_table_built)1268 self.addon_view.connect('table-built', self._on_addon_table_built)
12691269
1270 self.addons_hbar = self._hbars[1]1270 self.addons_hbar = self._hbars[1]
1271 info_vb.pack_start(self.addons_hbar, False, False, StockEms.SMALL)1271 info_vb.pack_start(self.addons_hbar, False, False, StockEms.SMALL)
12721272
1273 # recommendations
1274 catview = CategoriesViewGtk(
1275 self.datadir, None, self.cache, self.db, self.icons, None)
1276 self.recommended_for_app_panel = RecommendationsPanelDetails(catview)
1277 self.recommended_for_app_panel.connect(
1278 "application-activated",
1279 self._on_recommended_application_activated)
1280 self.recommended_for_app_panel.show_all()
1281 self.info_vb.pack_start(self.recommended_for_app_panel, False,
1282 False, 0)
1283
1284 # package info1273 # package info
1285 self.info_keys = []1274 self.info_keys = []
12861275
@@ -1309,6 +1298,17 @@
13091298
1310 vb.pack_start(self._hbars[2], False, False, 0)1299 vb.pack_start(self._hbars[2], False, False, 0)
13111300
1301 # recommendations
1302 catview = CategoriesViewGtk(
1303 self.datadir, None, self.cache, self.db, self.icons, None)
1304 self.recommended_for_app_panel = RecommendationsPanelDetails(catview)
1305 self.recommended_for_app_panel.connect(
1306 "application-activated",
1307 self._on_recommended_application_activated)
1308 self.recommended_for_app_panel.show_all()
1309 self.info_vb.pack_start(self.recommended_for_app_panel, False,
1310 False, 0)
1311
1312 # reviews cascade1312 # reviews cascade
1313 self.reviews.connect("new-review", self._on_review_new)1313 self.reviews.connect("new-review", self._on_review_new)
1314 self.reviews.connect("report-abuse", self._on_review_report_abuse)1314 self.reviews.connect("report-abuse", self._on_review_report_abuse)

Subscribers

People subscribed via source and target branches