Merge lp:~zeitgeist/activity-log-manager/smaller-exclude-icons into lp:activity-log-manager

[Manish Sinha (मनीष सिन्हा)]

Fixes LP: #1198364 where exclude list has smaller icons

Looks like: http://i.imgur.com/zPwGMPp.png

[Matthew Paul Thomas (mpt)]

This looks promising, but all icons should be the same size. Folder and document icons should be just as small as app icons.

review: Needs Fixing (design)

[Rico Tzschichholz (ricotz)]

I don't see the need in reducing the icon-sizes. The problem which you want to fix is not caused by them. The underlying widget structure is statically sized and therefore doesn't adapt to different font-sizes.

And yes, all icons should have the same sizes.

So this approach is not the right thing to do imo. (which makes the bug invalid as well)

review: Disapprove

[Matthew Paul Thomas (mpt)]

Ah, I didn't read the bug report, sorry.

I think that for consistency, icons in this list should be laid out exactly the same way as icons in menu items are. That does mean they should be smaller. However, Rico is right that that wouldn't actually fix the vertical alignment of the text. In the screenshot, the text for the previous-sized folder icons is too high, but the text for the newly-sized application icons is too low. It should be vertically centered, just like text in menu items.

Unmerged revisions

76. By Manish Sinha (मनीष सिन्हा) on 2013-07-09

Fixes LP: #1198364 where exclude list has smaller icons

=== modified file 'src/unified-privacy.vala'
--- src/unified-privacy.vala	2013-07-09 01:34:49 +0000
+++ src/unified-privacy.vala	2013-07-09 09:33:25 +0000
@@ -481,7 +481,7 @@
 				else if (dir.to_string() == "G_USER_DIRECTORY_PUBLIC_SHARE")
 					nautilus_icon = new ThemedIcon("folder-publicshare");
 				if(nautilus_icon != null) {
-					var pixbuf = ApplicationsTreeView.get_pixbuf_from_gio_icon(nautilus_icon, 24);
+					var pixbuf = ApplicationsTreeView.get_pixbuf_from_gio_icon(nautilus_icon, 16);
 					if (pixbuf != null)
 						icon = pixbuf;
 				}
@@ -519,7 +519,7 @@
 		private void add_app_to_view (string app) {
 			DesktopAppInfo app_info = new DesktopAppInfo(app);
 			if (app_info != null ) {
-				var pix = ApplicationsTreeView.get_pixbuf_from_gio_icon(app_info.get_icon(), 24);
+				var pix = ApplicationsTreeView.get_pixbuf_from_gio_icon(app_info.get_icon(), 16);
 				var markup = ApplicationsTreeView.markup_for_app(app_info);
 				TreeIter iter;
 				this.exception_list_store.append(out iter);
@@ -564,11 +564,11 @@
 //Based on gnome-contacts contacts-cell-renderer-shape.vala
 public class ExceptionCellRenderer : CellRenderer {
 
-	private const int PIXBUF_SIZE = 24;
+	private const int PIXBUF_SIZE = 16;
 	private const int xspacing = 3; 
 	
 	private const int default_width = 60;
-	private const int renderer_height = 30;
+	private const int renderer_height = 22;
 	
 	private Widget current_widget;