Merge lp:~larsu/indicator-messages/fix-939953 into lp:indicator-messages/0.3

Proposed by Lars Karlitski
Status: Merged
Approved by: Ted Gould
Approved revision: 254
Merged at revision: 254
Proposed branch: lp:~larsu/indicator-messages/fix-939953
Merge into: lp:indicator-messages/0.3
Diff against target: 24 lines (+2/-1)
2 files modified
src/indicator-messages.c (+1/-1)
src/messages-service.c (+1/-0)
To merge this branch: bzr merge lp:~larsu/indicator-messages/fix-939953
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Review via email: mp+96611@code.launchpad.net

Description of the change

Inserts a blank icon for app shortcuts to make them align properly.

Also reverts r244, which is another alignment issue. For some reason, I needed to put more padding in back then than we need now, I assume it was the theme's fault.

To post a comment you must log in.
Revision history for this message
Ted Gould (ted) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/indicator-messages.c'
--- src/indicator-messages.c 2012-02-10 21:42:43 +0000
+++ src/indicator-messages.c 2012-03-08 16:38:59 +0000
@@ -724,7 +724,7 @@
724 gtk_widget_set_size_request(GTK_WIDGET (gmi), -1, height + 4);724 gtk_widget_set_size_request(GTK_WIDGET (gmi), -1, height + 4);
725725
726#if GTK_CHECK_VERSION(3, 0, 0)726#if GTK_CHECK_VERSION(3, 0, 0)
727 gtk_widget_set_margin_left (hbox, width + 2 * padding);727 gtk_widget_set_margin_left (hbox, width + padding);
728#endif728#endif
729729
730 GdkPixbuf * pixbuf = dbusmenu_menuitem_property_get_image(newitem, INDICATOR_MENUITEM_PROP_ICON);730 GdkPixbuf * pixbuf = dbusmenu_menuitem_property_get_image(newitem, INDICATOR_MENUITEM_PROP_ICON);
731731
=== modified file 'src/messages-service.c'
--- src/messages-service.c 2012-02-08 20:42:36 +0000
+++ src/messages-service.c 2012-03-08 16:38:59 +0000
@@ -621,6 +621,7 @@
621 g_debug("Application Shortcut added: %s", mi != NULL ? dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_LABEL) : "none");621 g_debug("Application Shortcut added: %s", mi != NULL ? dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_LABEL) : "none");
622 DbusmenuMenuitem * shell = DBUSMENU_MENUITEM(data);622 DbusmenuMenuitem * shell = DBUSMENU_MENUITEM(data);
623 if (mi != NULL) {623 if (mi != NULL) {
624 dbusmenu_menuitem_property_set (mi, DBUSMENU_MENUITEM_PROP_ICON_NAME, "");
624 dbusmenu_menuitem_child_append(shell, mi);625 dbusmenu_menuitem_child_append(shell, mi);
625 }626 }
626 resort_menu(shell);627 resort_menu(shell);

Subscribers

People subscribed via source and target branches