=== Cimi_ is now known as Cimi === chrisccoulson_ is now known as chrisccoulson [11:34] davidbarth: I would move the deadline for the RGBA style property support in murrine after 27th august [11:35] before the 27th, I hope cody will send me patches for the widget ordering [11:35] and the resize gripper [11:46] Cimi: hi [11:50] Cimi: the argb style prop s not critical, but what's blocking to choose the name (as it seems the rest of the code is here already)? [11:50] Cimi: widget ordering, how critical is that for the theme? cody was not really keen on changing that at this point in the release === MacSlow is now known as MacSlow|lunch [11:58] davidbarth: it is not critical [11:58] it's just better :) [11:58] with that change, the theme will definitely look better [12:00] davidbarth: do you know of any possible dbusmenu bugs which might cause strange race conditions? [13:13] klattimer: hi [13:13] hey [13:13] what kind of side-effect are you getting? [13:14] well, if the ibus icon is passed through libindicator into the notification area it works, if it's in the indicator-applet it doesn't [13:14] throwing a dbus error which looks like a race condition [13:15] passed thru lib as opposed to the indicator? i don't follow you [13:15] well, if you don't have the indicator applet on the panel [13:15] it puts an icon in the status area instead [13:15] ah ok [13:16] and? (re-reading) [13:16] so the only difference being the existence in the indicator applet or the notification area, one works, one doesn't [13:16] so libindicator is not properly propagating the icon [13:17] the old systray works, not the new indicator one, right? [13:17] let me explain again [13:17] starting a patched ibus-daemon without an indicator applet in the panel, puts an icon in the systray (albeit incorrect, but that's a bug in the way icon picking works) [13:18] ok [13:18] normal fallback mode for libindicator [13:18] starting ibus-daemon with an indicator applet, will cause the indicator to throw a dbus error on selecting the input method engine [13:18] k [13:18] you can see the bug report here; https://bugs.launchpad.net/ubuntu/+source/ibus/+bug/564034 [13:18] Error: Could not parse data returned by Launchpad: list.index(x): x not in list (https://launchpad.net/bugs/564034) [13:18] comment #38 has the dbus message in it [13:18] checkng [13:19] when I tested this on friday things were working [13:19] so an update between then and now has caused the problem [13:21] klattimer: and i guess that breaks around line 201 in __set_im_icon [13:21] nope [13:22] around 520 in __im_menu_item_activated_cb [13:22] ah really? [13:22] the set_im_icon stuff works perfectly [13:22] yep... when calling focus_ic.set_engine(engine) [13:24] i don't see the connection between the engine and anything gtk that could trigger a signal and force libindicator to push a change across the bus [13:26] davidbarth: neither do I [13:26] so, I'm currently thinking that it's some kind of dbusmenu slowness or another race condition [13:26] hmm, there seems to be a reference to engine.icon and engine.longname somewhere in this panel.py module [13:27] right, set_im_icon(engine.icon) [13:27] so even if it's not pushed immediately, it's the link [13:27] the callback somehow triggers a signal that sends an update across dbus [13:28] but why does it break, no ideas so far [13:28] but why would it generate an error in dbus saying there's no engine [13:28] it makes no sense [13:28] and also, why would it work fine in the systray but not in the indicator-applet [13:29] the log of changes is here: https://code.edge.launchpad.net/~indicator-applet-developers/indicator-application/trunk [13:29] because of a change in API [13:29] the change broke the ABI [13:29] if you have elements that are not packaged you're relying on, you'd better make distclean and re-autogen.sh [13:30] right I'll give that a go [13:30] that's the most probable reason to me so far [13:31] klattimer: btw, do you have everything you need for the rest of the keyb patches? like icons? [13:31] davidbarth: the existing icons are being used atm [13:31] I've found that I need to produce a very long list of icons [13:31] if we're going to theme them [13:32] a good start would be the m17n icon list I already sent you [13:32] right [13:32] but design has it too, need to check when they can get back to you [13:33] k [13:33] I'm starting to think that ibus is going to miss the release of maverick [13:33] the code is unpredictable a lot of the time :/ [13:33] also, i wanted to mention a new change request for g-p-m, to update some strings and logic for displaying the time remaining for a battery [13:34] that's "Change the battery strings that are too long in the menu: TODO" in https://blueprints.edge.launchpad.net/ubuntu/+spec/dx-m-indicator-power [13:35] and the strings definitions are at: https://wiki.ubuntu.com/BatteryStatusMenu#Items [13:37] k [13:37] is there a bug report? === MacSlow|lunch is now known as MacSlow [13:40] klattimer: i'm looking for it, but the gpm bug list is, errr, impressive [13:42] mpt: ping? i'm almost sure you had a bug to change the menu strings to something more compact for the power indicator (referring to the "strech limo" of status strings or something) [13:42] oh great... I've somehow managed to lose my filesystem entirely :/ [13:42] thankfully it's a vm [13:42] klattimer: kernel issue? [13:43] no the disk strangely went into read only mode, so I rebooted the vm [13:43] and now apparently there is no /sbin/init [13:43] :/ [13:43] whole FS is gone [13:43] :/ [13:43] davidbarth, I don't remember there being a bug report about it. [13:44] mpt: ok, i'll open one for that then; at least getting that string properly fixed before it's too late [13:44] mpt taking https://wiki.ubuntu.com/BatteryStatusMenu#Items as the reference [13:51] Cimi: i've pinged ronoc, asking him to join this channel [13:52] Cimi: part of the sound menu widgets do come from the IDO lib (lp:ido), like the scrub bar [13:54] I would like to add small rounded corners for the cover art [13:54] and maybe [13:54] fixing the "player" [14:08] Cimi: hi [14:09] i hear you are interested in working onthe sound menu [14:09] yeah [14:09] I found some bugs [14:10] Cimi, yes it is a work in progress, todays release should fix some of those [14:10] which bugs in particular [14:10] for example [14:10] I guess below the playe [14:10] r [14:11] there's a squared area [14:11] filled with the menu bg color [14:11] klattimer: https://bugs.launchpad.net/ubuntu/+source/gnome-power-manager/+bug/619816 [14:11] Launchpad bug 619816 in gnome-power-manager (Ubuntu) "Battery status line too long (affected: 1, heat: 6)" [Medium,New] [14:11] davidbarth: yeah I saw that come in [14:11] cimi: fixed in this release 0.4.0 [14:11] ronoc: that does not work with the new menus we have in maverick [14:11] ok [14:11] maybe I should just need to wait [14:12] klattimer: watch out, there are string changes, but i guess there is also some of the algorithm to check to see if the time intervals (<12h, >, calculating) can match the different strings [14:12] klattimer: they make sense from a user point of view, but that may not be the code/HW reality; i haven't checked yet [14:12] k [14:13] Cimi: I think it would be best since I have not released in 3 weeks because i was waiting for mpris2 to stabilize, most UI quirks have been resolved. Nice to haves can come next once all functional stuff is resolved. The next week or two will be working on integration bugs with the clients who are at present moving over to mpris2 [14:15] great [14:15] will wait then [14:15] Cimi: so i think you should work from the indicator-sound source branch today if you want to adapt the theme [14:15] Cimi: or wait for tomorrow's release [14:16] wait tomorrow :) [14:16] Cimi: if you're into indicators, the datetime one would need some love to get the calendar styled better i think [14:17] Cimi: ie probably removing the surrounding frame to integrate it seamlessly into the rest of the menu [14:17] Cimi: i think only the month/year header should be decorated, whereas the rest should not [14:17] Cimi: but you may want to double check that with chaotic first [14:18] Cimi: or compare with his latest mockups [14:18] davidbarth: I'll upload the latest mockup for Cimi [14:19] yeah [14:19] chaotic: cool thanks; you're on all fronts ;) [14:19] davidbarth: ;) [14:19] Cimi: done [14:21] maybe just removing the bigger fram [14:21] e [14:21] the header is useful, because it is clickable [14:22] but maybe it can be fixed [14:24] is it possible to create a ppa directly from a launchpad bzr now? [14:26] chaotic: did you see the latest work I've done for the sound indicator? :) [14:26] look at the slider [14:26] ;) [14:26] also, I improved the rendering of disabled states for slider (mainly) and a touch at buttons too [14:27] Cimi: yes, it's looking much better - is that slider big enought o grab ;) [14:30] davidbarth: the code for the calendar widget is in ido? [14:30] chaotic: on my netbook I can grab it [14:31] chaotic: the lower slider is small [14:31] Cimi: it is [14:31] but you should speak with ronoc [14:32] Cimi: it's a scrubber - so I wonder if it should look a little different - I had intended that one to be round [14:57] Cimi: it is, yes; idocalendar or something [15:00] yeah improved btw [15:09] ok fixed the calendar for both themes [16:40] tedg: it seems my absolute filename patch to libindicator isn't released into the repos yet, can you tell me when I can expect it? [16:49] klattimer, Hmm, I believe that it is... [16:50] tedg: I'm upto date on the latest libindicator and testing the ibus patch once again [16:50] and all I'm getting is the missing-image [16:50] klattimer, Oh, no. So what happened is that cause another bug, which they then distro patched it to break things. [16:50] tedg: in ibus? [16:50] I saw there was a distro patch in ibus which reversed some of my changes and broke others [16:51] klattimer, No, in libindicator -- basically there was an error case that hit a g_error now and shutdown unity :) [16:51] oh [16:51] that's not great [16:52] Turns out people don't like it when their window manager shuts down :) [16:52] no, I can imagine it sticks in their craw a little [16:52] So I have a patch for that, we should release it. [16:52] cool, if you can do that, then I want to push my ibus packages to a ppa [16:52] so I can get them tested quickly [16:54] kenvandine, Can we update the distro patch to be my branch? [16:54] kenvandine, Do you have time for some libindicator packaging? [16:55] tedg, sure [16:55] kenvandine, Do you want a release, or just update the distro patch? [16:55] a patch is fine [16:55] or... a release if it means no release tomorrow :) [16:56] could kill two birds [16:56] if there is no more changes due [16:56] I can't imagine anymore... [16:56] So let's just make a release. [16:56] great [17:02] kenvandine: I need to sign a package but it's assuming someone elses email address [17:03] what do I need to change in order to make it work with my key? [17:04] set DEBEMAIL env variable [17:05] before doing the dch [17:05] to get it right in the changelog, which is what it looks for when signing [17:05] or [17:05] dch? [17:05] debsign -k [17:05] dch creates a new version, in debian/changelog [17:05] oh ok [17:06] debsign -k blah.sources [17:06] * klattimer does that stuff manually for some reason [17:06] to sign the package for upload, if it wasn't signed when created [17:06] use dch -i :) [17:06] k [17:06] I will in future [17:06] dch -i creates a new version [17:06] dch [17:06] edits [17:17] iainfarrell: I've found a bug in your blog :-P [17:17] hey cimi [17:17] thanks :) [17:17] ehehe [17:17] in design or hungfu? [17:17] look at the "team page" [17:17] :D [17:17] Thirteen heads are [17:17] better than one. [17:18] but there are fourteen heads there :P [17:18] heh! [17:18] unless your head does not count! ahah