| jcastro | Nafai: around? | 00:46 |
|---|---|---|
| Nafai | Yes | 00:47 |
| jcastro | Nafai: are you on a laptop? | 00:47 |
| Nafai | Yes, Lenovo T-61p | 00:47 |
| jcastro | Nafai: does your power-manager dropdown work? | 00:47 |
| jcastro | I get a blank menu | 00:48 |
| Nafai | Let me see | 00:48 |
| jcastro | er, no menu at all I mean, just blank | 00:48 |
| Nafai | I get a menu just fine | 00:48 |
| Nafai | Though if it is a recent update, I haven't restarted things for a day or so | 00:49 |
| jcastro | when was the last time you logged out? | 00:49 |
| jcastro | ah ok | 00:49 |
| Nafai | Yeah, looks like a little over a day | 00:49 |
| Nafai | Am I running the old version? | 00:49 |
| jcastro | well, sometimes logging in and out makes a difference | 00:50 |
| jcastro | at your next natural break can you check? | 00:50 |
| Nafai | Sure thing | 00:50 |
| jcastro | we're frozen until like tomorrow night anyway | 00:50 |
| jcastro | so it's not a rush | 00:50 |
| Nafai | ok :) | 00:52 |
| Nafai | jcastro: Just logged out and back in -- same behavior you describe | 01:02 |
| jcastro | woo I will file a bug | 01:03 |
| Nafai | yay | 01:03 |
| jcastro | https://bugs.edge.launchpad.net/ubuntu/+source/gnome-power-manager/+bug/526747 | 01:06 |
| ubot4 | Launchpad bug 526747 in gnome-power-manager (Ubuntu) "battery applet popup is empty (affects: 3)" [Undecided,Triaged] | 01:06 |
| jcastro | Nafai: I even triaged it .... | 01:06 |
| * jcastro 's brain is broken | 01:06 | |
| Nafai | Pretty weird connecting to my desktop vino server from my phone :) | 01:09 |
| Nafai | Ok, the main GtkStatusIcon has been ported and fully tested for Vino | 01:19 |
| Nafai | But there is this other that I have code for but I have to figure out when it is shown :) | 01:19 |
| hyperair | who maintains notify-osd these days? there are several branches with merge proposals but have not been touched for months! | 01:45 |
| Nafai | w00t, finally | 04:13 |
| hyperair | can someone running lucid tell me if notify-osd still sucks with fractional font sizes? | 05:11 |
| nigelb | hyperair: you may have better luck in ubuntu+1 ;) | 05:15 |
| hyperair | meh. | 05:16 |
| hyperair | i've got a patch lying around to solve my issue | 05:16 |
| hyperair | but i'd be very concerned if it's still not fixed in lucid, since the patch was committed to some bzr branch somewhere | 05:16 |
| hyperair | hmm seems it's fixed | 05:21 |
| hyperair | oh well | 05:21 |
| hyperair | looks like it isn't as unmaintained as code.launchpad.net implies | 05:22 |
| === DrPepperKid is now known as MacSlow|afk | ||
| === 40FAAHTM9 is now known as djsiegel | ||
| === MacSlow|afk is now known as MacSlow | ||
| qense | GNOME Do merge request sent! | 11:32 |
| vish | qense: why is gnome-do using the app-indicators? | 11:56 |
| qense | vish: because it's got an icon | 11:56 |
| vish | ;p | 11:57 |
| qense | vish: the icon is going to be dumped in a short period of time, but that isn't in time for Lucid | 11:57 |
| qense | so I wrote a temporary patch for those that really want to use the icon | 11:57 |
| qense | plus: now the tray icon is using the monochrome icon, which is already provided by the humanity icon theme | 11:57 |
| vish | qense: the icon is already included in humanity gnome-do-panel | 11:57 |
| vish | heh ;) | 11:58 |
| qense | but it wasn't used for the tray icon yet | 11:58 |
| vish | qense: yeah , i just added some icons i had time to do :) but named them as -panel for the app to use it or not :) | 11:58 |
| vish | hmm , i dont think i included a 16px one | 11:59 |
| qense | vish: the icon name is "gnome-do-symbolic" | 11:59 |
| vish | yeah ,the old name ;) | 12:00 |
| vish | qense: gnome-do-panel is what i have in humanity ? are you sure it is -symbolic? that was the old name i had it renamed before release | 12:01 |
| qense | it is gnome-do-symbolic, I copied the line of code myself! | 12:01 |
| C10uD | hello, i added a diff to #527061 ..not sure of the result since i can't get the thing to install correctly here, anyone willing to try? | 12:02 |
| vish | qense: i meant in humanity ? ;) | 12:02 |
| qense | vish: I can't find any other icon that gnome-do-panel indeed | 12:04 |
| qense | but it did work | 12:04 |
| vish | yay | 12:04 |
| qense | strange | 12:04 |
| qense | is there some kind of redirect? | 12:04 |
| vish | probably , tedg might know | 12:05 |
| qense | vish: does appind add the suffix -panel to icon names? | 12:05 |
| qense | in that case I think I know how it works. | 12:05 |
| qense | gnome-do-symbolic doesn't exist so it passes gnome-do and then the applet checks for gnome-do-panel, which -- thanks to you -- does exist | 12:06 |
| qense | kenvandine: Could you review the patch attached to bug #527061 once you've got some spare minutes? | 12:13 |
| ubot4 | Launchpad bug 527061 in indicator-application "python bindings for libappindicator don't allow to change icon-theme-path (affects: 2)" [Undecided,Confirmed] https://launchpad.net/bugs/527061 | 12:13 |
| vish | qense: do you have a screenshot of the gnome-do icon in action? | 12:13 |
| qense | vish: I'll take one right now. | 12:14 |
| * vish wants to see if there is a need for a 16px icon ;p | 12:15 | |
| qense | vish: https://people.ubuntu.com/~qense/gnome-do-icon.png | 12:15 |
| qense | lunch time now, afk! | 12:16 |
| vish | anyone able to view qense 's ^ link? | 12:38 |
| qense | vish: try http://people.ubuntu.com/~qense/gnome-do-icon.png | 12:38 |
| vish | ah got it , had to use http | 12:38 |
| qense | my bad! | 12:38 |
| vish | qense: ;) | 12:38 |
| === MacSlow is now known as MacSlow|lunch | ||
| === MacSlow|lunch is now known as MacSlow | ||
| qense | grmbl | 13:54 |
| qense | GAPI won't work correctly for the signals/events of libappind | 13:54 |
| jcastro | jpetersen: you got the rhythmbox "next/prev" broken and power-manager empty menu right? | 14:35 |
| jcastro | jpetersen: bugs I mean, I assigned them yesterday, just making sure they're not landing in a spam bucket, heh | 14:35 |
| seb128 | jcastro, he closed the rhythmbox one already saying it's a libdbusmenu issue fixed | 14:36 |
| qense | me? | 14:36 |
| seb128 | tedg, ^ you confirm the checkbox, item update thing is fixed today? | 14:36 |
| jcastro | seb128: ah cool | 14:36 |
| seb128 | qense, what? | 14:36 |
| tedg | seb128: Yes. Confirmed. | 14:37 |
| seb128 | tedg, thanks | 14:37 |
| seb128 | tedg, new tarballs coming today right? | 14:37 |
| jpetersen | jcastro, Yes I got them | 14:37 |
| tedg | We need to release it, but it's fixed in trunk | 14:37 |
| tedg | seb128: Yeah, need to fix another issue, but yes. | 14:37 |
| qense | seb128: I've been triaging some of the related/duplicate bugs of that issue. | 14:37 |
| qense | tedg: If https://code.launchpad.net/~qense/indicator-application/fix-526620/+merge/20130 would be included in the release I'd be very happy. | 14:38 |
| qense | Although it's still useless for banshee since connection-changed is never emitted | 14:38 |
| tedg | qense: Cool, will do. | 14:39 |
| qense | thanks! | 14:39 |
| * tedg was wondering why he missed it, then realized it was 12 min ago :) | 14:39 | |
| qense | :) | 14:39 |
| jcastro | jpetersen: oh nice, I totally missed that you fixed g-p-m before poking you. | 14:43 |
| qense | tedg: I'm working on getting the library to emit the connection-changed signal, which according to the documentation is emitted when "we connect to a watcher or when it drops away". Should the signal we emitted on initialisation and dispose as well? | 14:48 |
| qense | I've chosen for emitting it near/at the end of register_service_cb() and watcher_proxy_destroyed() | 14:52 |
| tedg | qense: I don't think on init, as you can generally assume that then it wouldn't be connected. But I think so on dispose, if it was connected previously. | 14:52 |
| qense | tedg: but what if you use it for falling back? In that case you would launch the fallback mechanism when shutting down as well. | 14:53 |
| qense | Or is it advised to disconnect the signals before disposing? | 14:54 |
| tedg | qense: If you want a signal for fallback you should just override the fallback/unfallback functions with something that emits a signal. That'd probably be more reliable. | 14:54 |
| qense | ok | 14:54 |
| qense | tedg: thanks! I'll have a merge request ready soon if it doesn't turn out to be buggy. | 14:55 |
| jpetersen | I linked a branch with a fix to https://bugs.launchpad.net/ubuntu/+source/rhythmbox/+bug/526552 | 15:15 |
| ubot4 | Launchpad bug 526552 in rhythmbox (Ubuntu) "Music->Close menu item and Ctrl-W do nothing (affects: 1)" [Medium,Triaged] | 15:15 |
| seb128 | jpetersen, thanks | 15:20 |
| qense | tedg: If we don't want connection-changed to be emitted after the first connection we'd have to create a firstconnection bool to check that, otherwise it's undoable. Do you think that's worth it? | 15:34 |
| tedg | qense: ? I think as long as it defaults to "disconnected" it should emit everytime that changes. So on the first connection it'd emit, just not on init. | 15:35 |
| qense | I'm now sending it from app_indicator_dispose(), the end of check_connect() -- when we've just established a new connection -- and watcher_proxy_destroyed() | 15:36 |
| qense | tedg: Is that fine with you? | 15:37 |
| tedg | qense: Sounds like it on IRC. I'll double check the patch ;) | 15:37 |
| qense | tedg: good, I'll set you as the merge request reviewer. | 15:39 |
| qense | btw, it's impossible to override the Fallback and Unfallback methods in C# since you have to create a new class with AppIndicator as base. However, you cannot override the constructor, so you're doomed. | 15:58 |
| qense | That's why I'm using the connection-changed signals. | 15:58 |
| === jono_ is now known as jono | ||
| === MacSlow is now known as MacSlow|afk | ||
| kklimonda | is indicator-sound going to support using mouse wheel on the icon to change volume level? | 17:29 |
| tedg | kklimonda: Working on it. Tricky issues with levels of abstraction... | 17:39 |
| seb128 | kklimonda, no | 17:39 |
| seb128 | oh, ignore me then | 17:39 |
| tedg | kklimonda: I think the final answer is maybe, depending on who you like more :) | 17:40 |
| kklimonda | damn, a tough choice :) | 17:41 |
| seb128 | tedg, I though those were menus and would not do non standard things like tooltips or action in middle click | 17:47 |
| tedg | seb128: Yes. Apparently volume on scroll wheel was considered important enough to allow :) I think the reason we got mpt to agree to it is that there was no way anyone could actually physically see it, or know that exists, so it would be exclusively a power user feature. | 17:49 |
| tedg | seb128: Personally, I'd like to see scroll wheel on all app indicators as well. | 17:49 |
| tedg | seb128: For the same reasons. | 17:49 |
| tedg | seb128: But I didn't push for it in Lucid as I figured I had enough to do :) | 17:49 |
| seb128 | hehe | 17:50 |
| Nafai | jcastro: Is my plan for today the appropriate priorities? | 18:07 |
| jcastro | Nafai: yep! | 18:23 |
| Nafai | Cool, I should finish both of those things today! | 18:23 |
| jcastro | Nafai: I think once freeze is lifted and the stuff lands tomorrowish you'll get bugs filed. | 19:22 |
| Nafai | Yeah, I imagine so | 19:22 |
| Nafai | bbiab, grabbing lunch | 19:23 |
| Nafai | back | 20:13 |
| === MacSlow|afk is now known as MacSlow | ||
| gabaug | guys, please tell me the indentation here: http://www.lottanzb.org/wp-content/uploads/2010/02/indicator.png is a bug, and that the images and checkboxes are intended to be lined up/left-aligned | 21:45 |
| gabaug | https://wiki.ubuntu.com/CustomStatusMenuDesignGuidelines makes it sound like the only thing wrong w/ that last sshot is that "Show RB" and "Quit" are in the same group - but the problem (IMO) is that the indentation of GtkImageMenuItems is out of whack | 21:46 |
| qense | gabaug: I'm afraid it is something we're aware of but can't fix at the moment. | 21:50 |
| qense | gabaug: it's because some menu items have both checkboxes and images and we have to be able to show both. | 21:51 |
| gabaug | qense: ok - it's considered a bug, though? | 21:51 |
| qense | gabaug: I'm not sure, I would consider it a bug, but maybe it's not. | 21:51 |
| gabaug | qense: really? Gtk doesn't support that - why would you try? | 21:51 |
| qense | apparently some applications do really need it | 21:51 |
| qense | strangely | 21:51 |
| gabaug | qense: which? | 21:52 |
| qense | gabaug: If I'd know I'd let you know. :S | 21:52 |
| qense | tedg: Thank you for and congratulations with AppInd 0.0.14! A nice number of bugs crushed now. | 21:55 |
| tedg | qense: Thanks for you patches! Yeah, I'm excited about the release. No more bugs :) | 21:56 |
| tedg | gabaug: QT allows it, so QT apps use it. | 21:56 |
| qense | tedg: unfortunately I've found yet another one in the Mono bindings. :) | 21:56 |
| qense | I'm looking into it right now | 21:56 |
| gabaug | tedg: maybe optimize it so if the menu doesn't include any such items (eg all Gtk+ apps), then indentation is in the Gtk+ style? | 21:57 |
| tedg | gabaug: Then if someone added a new menu item everything would shift. | 21:57 |
| gabaug | tedg: if they added a new one w/ check+img? | 21:58 |
| tedg | gabaug: Yes | 21:58 |
| gabaug | tedg: OK - I think it's atrocious looking for Gtk+ apps, but I'll leave it there, obviously your decision | 21:59 |
| qense | kenvandine: Did you write the initial Mono bindings for AppInd? | 22:06 |
| jcastro | qense: I think it was DBO | 22:08 |
| qense | jcastro: ok, thanks | 22:08 |
| qense | he isn't here though | 22:08 |
| seb128 | jcastro, bug #497021 should be transparent no? | 22:09 |
| ubot4 | Launchpad bug 497021 in rhythmbox (Ubuntu) "provide a fallback for lack of application indicator support (affects: 3) (dups: 1)" [Low,Triaged] https://launchpad.net/bugs/497021 | 22:09 |
| seb128 | jcastro, the appindicator stack fallbacks to notification icon | 22:09 |
| jcastro | seb128: I don't know if it is or not, and I just tried it and it wasn't. | 22:09 |
| jcastro | seb128: does it fallback for you? | 22:09 |
| seb128 | weird | 22:09 |
| seb128 | jcastro, if indicator-application-service is not running yes | 22:10 |
| seb128 | you might have to stop the service if you just removed the applet | 22:10 |
| jcastro | oh | 22:10 |
| jcastro | I was just removing the applet | 22:10 |
| jcastro | seb128: aha! It does the right thing. I will resolve the bug | 22:11 |
| seb128 | jcastro, thanks | 22:12 |
| jcastro | I didn't think about the -service bit | 22:12 |
| seb128 | jcastro, I though so, easy mistake ;-) | 22:13 |
| jcastro | seb128: ok that was cool, when I added the applet back rhythmbox just moved over there. | 22:28 |
| seb128 | jcastro, kudos to ted | 22:28 |
| qense | we do have a fresh bug 528088 | 22:44 |
| ubot4 | Launchpad bug 528088 in indicator-application (Ubuntu) "Transmission applet does not show when there are no indicator applet (affects: 1)" [Undecided,New] https://launchpad.net/bugs/528088 | 22:44 |
| C10uD | ow, you didn't check bug 527061 before releasing .14 | 22:48 |
| C10uD | :| | 22:48 |
| ubot4 | Launchpad bug 527061 in indicator-application "python bindings for libappindicator don't allow to change icon-theme-path (affects: 2)" [Undecided,Confirmed] https://launchpad.net/bugs/527061 | 22:48 |
| qense | C10uD: I'd try a merge request, that's easier to review. | 22:51 |
| C10uD | qense, i don't know if that works, but for those that know what they're doing, it's an easy task i think | 23:33 |
| C10uD | anyway, gotta go, i hope someone will take a look | 23:33 |
| C10uD | see ya | 23:33 |
| qense | bye | 23:33 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
