← Back to team overview

zeitgeist team mailing list archive

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

 

Manish Sinha (मनीष सिन्हा) has proposed merging lp:~zeitgeist/activity-log-manager/smaller-exclude-icons into lp:activity-log-manager.

Requested reviews:
  Rico Tzschichholz (ricotz)
  Jeremy Bicha (jbicha)
Related bugs:
  Bug #1198364 in Activity Log Manager: "Exclude list should have smaller icons"
  https://bugs.launchpad.net/activity-log-manager/+bug/1198364

For more details, see:
https://code.launchpad.net/~zeitgeist/activity-log-manager/smaller-exclude-icons/+merge/173678

Fixes LP: #1198364 where exclude list has smaller icons
-- 
https://code.launchpad.net/~zeitgeist/activity-log-manager/smaller-exclude-icons/+merge/173678
Your team Zeitgeist Framework Team is subscribed to branch lp:~zeitgeist/activity-log-manager/smaller-exclude-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;
 	


Follow ups