| ia | hello. I have a question about integration app (PyGTK + appindicator module) into AppIndicator applet, so I will be very appreciate for any clues: could anyone tell me, please, how to "append"/"attach" (is it possible itself?) some text right next to some appindicator's icon? | 10:45 |
|---|---|---|
| qense | ia: that's not possible, it would be against the design document | 10:54 |
| ia | qense: ok, thanks for the answer. | 10:55 |
| qense | you're welcome | 10:55 |
| dashua | vish, Is the Banshee monochrome icon working for you? | 13:47 |
| qense | dashua: iirc Banshee has to use AppIndicator in order to use the monochrome icon. | 13:50 |
| qense | dashua: I've written a patch for that, but it's not accepted yet. | 13:51 |
| dashua | I'm using it now from git | 13:53 |
| dashua | It's awesome. | 13:53 |
| qense | dashua: What do you mean? Is there an AppIndicator in GIT already? | 13:54 |
| dashua | http://www.ubuntu-pics.de/bild/46973/screenshot_1_dg7n5E.png | 13:55 |
| dashua | Just have to set the flag | 13:55 |
| dashua | --enable-appindicator | 13:55 |
| qense | dashua: Since when is that? | 13:55 |
| dashua | The icon is not monochrome | 13:56 |
| qense | dashua: I know, I wrote the darn patch. :P But I wasn't told that the patch was included. | 13:56 |
| dashua | HA | 13:56 |
| dashua | I guess it is ;) | 13:56 |
| dashua | Works great. | 13:56 |
| qense | jcastro! | 13:56 |
| dashua | The only reason I was still using RB waiting fro the AppIndicator. | 13:57 |
| qense | dashua: Are you sure you're using the plain git master? I can't find a recent commit of that patch. | 13:58 |
| dashua | Yeah | 14:00 |
| qense | I can't find it anywhere in the code as well. | 14:00 |
| dashua | I can't find it either. | 14:00 |
| qense | dashua: Where did you get the code from? | 14:00 |
| qense | git://git.gnome.org/banshee ? | 14:00 |
| dashua | yeah | 14:00 |
| dashua | 1.6 RC1 | 14:01 |
| qense | and you didn't apply any patch? | 14:01 |
| qense | I don't see any RC tag as well. | 14:01 |
| dashua | http://www.ubuntu-pics.de/bild/46974/screenshot_031_iNIKC6.png | 14:02 |
| dashua | I don't think I applied any patches | 14:02 |
| qense | If you've got an AppIndicator you must have. I wouldn't know how you could have got the Application Indicator otherwise. | 14:03 |
| dashua | Not sure, but I love it ::) | 14:03 |
| qense | :) | 14:03 |
| dashua | qense, I think I may have tried the 1.5.3 patch, but it did not pass. | 14:04 |
| dashua | I guess it did. | 14:05 |
| qense | it must have | 14:05 |
| qense | dashua: I'll take a quick look at the code to see how the icon name is selected. | 14:05 |
| dashua | I thought it was in master | 14:05 |
| qense | It's most certainly not. | 14:05 |
| dashua | Yeah, I see that. | 14:06 |
| dashua | Humanity has it as banshee-panel | 14:07 |
| vish | dashua: the icon is there , banshee needs to use it | 14:15 |
| qense | vish: The icon name of Banshee is "media-player-banshee". | 14:17 |
| qense | vish: That means that AppInd doesn't try 'banshee-panel', but 'media-player-banshee-panel', which doesn't exist. | 14:17 |
| qense | vish: Would you mind renaming the icon? | 14:17 |
| dashua | http://www.ubuntu-pics.de/bild/46977/screenshot_1_Lu44XX.png | 14:22 |
| dashua | There we go | 14:22 |
| dashua | media-player-banshee-panel | 14:23 |
| dashua | That worked | 14:23 |
| qense | good | 14:23 |
| dashua | Thx | 14:23 |
| qense | dashua: I assume the large amount of spacing between your menu items is your theme? | 14:23 |
| dashua | They all seems to have the same spacing. | 14:26 |
| dashua | seem* | 14:26 |
| qense | ok | 14:27 |
| vish | qense: ok , np.. i'll do it with the next update , since there was no mention on the bug report from the banshee devs ,i just used the same name as other apps | 14:28 |
| qense | vish: It's a bit strange indeed, something they could have told you about. | 14:28 |
| qense | maybe they assumed you'd look in the code for the name they use ;) | 14:28 |
| vish | yeah , i forgot to check the bug before pushing the change :( | 14:29 |
| dashua | vish, Do you need a bug? | 14:29 |
| vish | dashua: that would be nice | 14:29 |
| dashua | vish, Ok np. | 14:29 |
| tgpraveen12 | qense: u are doing the app indicator for banshee right | 14:38 |
| tgpraveen12 | http://www.ubuntu-pics.de/bild/46977/screenshot_1_Lu44XX.png according | 14:38 |
| tgpraveen12 | to that is show notification option really needed | 14:38 |
| tgpraveen12 | one would hardly ever change that option. | 14:38 |
| qense | tgpraveen12: That option was added by the Banshee developers. | 14:42 |
| qense | tgpraveen12: I'm reusing the same context menu that's used for the regular status icon. | 14:42 |
| kklimonda | heh, this menu is a perfect just why we should write indicators from scratch :/ | 14:45 |
| tgpraveen12 | qense: i would still suggest not having that option. maybe someone else should also comment on this. | 14:46 |
| tgpraveen12 | i dont think there is any reason to resuse the same options as that of the regular status icon | 14:46 |
| tgpraveen12 | app indicators were supposed to be about fixing things. just my 2 cents | 14:46 |
| kklimonda | tgpraveen12: I agree, there is no reason to enable/disable notifications from indicator | 14:47 |
| qense | tgpraveen12, kklimonda: In that case I would file a bug against Banshee on GNOME Bugzilla with a request for removing that option from the status icon menu. | 14:47 |
| qense | Or do you think it is suitable in the regular context menu? | 14:47 |
| kklimonda | tgpraveen12: heh, that's my point - if we are just reusing old interface and then make it even worse (by removing right click, tooltips etc.) then there is no reason to do that at all - we'll just piss users :/ | 14:48 |
| kklimonda | qense: my opinion is that the only place this setting should be at is preferences menu - and only if we really have to make it a setting. I'm pretty sure that the only reason why is it configurable is that there is no do-not-disturb mode for notifications. | 14:49 |
| qense | ok | 14:49 |
| qense | I agree with that, but I didn't want to edit the context menu without prior consent. | 14:50 |
| qense | kklimonda, tgpraveen12: However, the whole project is frozen now they've already released the first release candidate for 1.6.0. | 14:51 |
| qense | I'm even sure whether the AppInd patch will be accepted before the release. | 14:52 |
| tgpraveen12 | qense: so its not going to be in lucid? | 14:55 |
| qense | tgpraveen12: I've attached a patch to the Launchpad bug report that's suitable for applying to the Debian -- we sync Banshee -- package with quilt. | 14:55 |
| qense | It's up to others whether an exception will be granted for this patch. | 14:56 |
| kklimonda | qense: do you know what is the future of appindicator? is it going to migrate to GtkAction and then be accepted by GTK+ guys or is it going to live as a Canonical-only "feature"? | 14:58 |
| qense | kklimonda: Things similar to the whole Notify OSD and Indicators family are demonstrated in the GNOME Shell mock-ups, so I assume at the very least we'll see an implementation of something similar to the AppInd in GNOME. | 14:59 |
| kitallis | anyone mind looking at this : https://lists.ubuntu.com/archives/ubuntu-soc/2010-March/000047.html :) | 16:20 |
| qense | kitallis: I'm afraid you're in the wrong IRC channel for that. | 16:26 |
| qense | kitallis: I'd try #ubuntu-gsoc | 16:27 |
| kitallis | qense, try a /whois | 16:27 |
| qense | ? | 16:27 |
| kitallis | i've already discussed it over there. | 16:28 |
| qense | They told you to discuss it here? | 16:28 |
| kitallis | i was hoping to find someone interested, here. | 16:28 |
| qense | Ah, in that case I'd advise you to come back on a working day when there are more people. | 16:28 |
| kitallis | oh. alright. | 16:28 |
| qense | kitallis: Mirco Müller, MacSlow on IRC, is leading the development of NotifyOSD a bit. You might want to talk to him. | 16:33 |
| kklimonda | what if upstream developers decide against "fixing" their applications to follow our new Units Policy? Should we patch them and take the beating? | 19:05 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
