[10:25] bingo! [10:25] I got something to work at last. [10:27] It works! Eureka! [10:28] jcastro: I'll test it a last time and after that I can finally continue to work on Banshee. [10:29] I can't believe it's actually working. :P [12:19] have you considered changing the spacing between the 'old' systray icons, to make it more consistent with the new indicators? [12:54] qense: hey will banshee get the app indicator in lucid now that UI freeze has been passed? [12:54] and FF freeze also [12:56] tgpraveen12: I'll try to get it in Banshee 1.5.5, and eventually Banshee 1.6.0 will be included in Lucid, so it should eventually reach Lucid, but it may need a FFe [12:58] qense: thx for the reply. have u talked to upstream? will it be included upstream/ [12:58] tgpraveen12: at the moment I'm working on getting the patch I provided into an acceptable state for upstream. [12:59] qense: awesome [12:59] I just stumbled upon another bug in the Mono bindings for AppInd, though. The Fallback and Unfallback methods aren't provided in C# and therefore I cannot override them. [13:29] moni [13:30] …I beg your pardon — pre-empted gnome-do there [14:07] ok, we've got a problem [14:07] the virtual-member support of the GAPI generator won't land until GTK# 3.0 [14:08] Which means no fallback and unfallback until September [14:36] jcastro: After several hours of working on getting fallback and unfallback to work in Mono I'm removing the use of that from the Banshee patch since GTK# is such a stupid project. [14:36] aargh [14:37] jcastro: I need virtual member support, and GAPI has already got that since a few months ago, before Karmic was released, but it won't be included until GTK#3.0. [14:37] Just when everything else worked... [14:38] sigh [14:38] Everything worked flawlessly! [14:38] All that was needed was GAPI to recognise a certain node type, which it already can in trunk. [16:23] qense: ok so you're saying it all works except for the fallback? [16:25] jcastro: We'll have to make do with the default fallback. I'm working on finishing the last bits of the patch, should be able to submit it today. [16:25] ok I don't understand [16:25] jcastro: So yes, the rest works. [16:26] can you explain how it will be different? [16:27] jcastro: The default fallback uses the AppInd menu and shows it on left mouse button. I wanted to use the GtkStatusIcon provided by Banshee since that provides a bit more functionality. [16:28] It means: no tooltip and no rating-from-the-tray-menu. [16:29] jcastro: Do you think I should add the artist and the album currently being played to the menu, just as Rhythmbox already does? [16:31] qense: I was just thinking about that [16:31] qense: yeah I think it makes sense to have them both behave the same in there [16:31] qense: I am asking upstream how they would feel about a monochrome icon too [16:32] the same as for Rhythmbox? [16:32] no [16:32] I was thinking the current logo but black and white [16:33] That probably fits better with the themes indeed. [21:28] jcastro: Giving up on showing the currently played song in the menu: due to a bug in GTK+ the label of a MenuItem isn't updated when its action's label is. Due to a bug in GTK# you can't change or read the label of a MenuItem after you've created it. [21:28] sigh... [21:28] Again just when I thought it worked. [21:30] qense: we change the Label of MenuItems all the time in Banshee by modifying their Gtk.Action [21:30] gabaug: Really? [21:31] maybe it is a AppInd bug, despite what the comments in the Rhythmbox patch say [21:31] qense: are you modifying the Gtk.Action and not the ActionEntry? [21:31] gabaug: yes I am [21:31] gabaug: and changing the Visible property does work [21:32] just not the Label property [21:32] yeah, is probably an appind bug, not sending the update over dbus [21:32] definitely not an issue with Gtk+ or Gtk# [21:32] I'll take a look at it then [22:10] qense, the problem is that the GtkMenuItem does not send the notify::label signal when the label of the action is changed [22:10] ah [22:10] can't we emit that ourselves? [22:11] qense, not really [22:12] qense, i will fix it on appindicator [22:12] jpetersen: How? [22:12] qense, listen also to the signal of the action related to the menu item [22:12] ah [22:12] qense, I will work on monday on it [22:13] jpetersen: great, thank you [22:13] your welcome [22:26] patch sent upstream [22:26] I'm off now, bye!