NafaiBack again, sorry00:09
NafaiTrying now00:10
jcastroNafai: let's hope this is it!00:24
NafaiDoesn't appear to be any difference :(00:25
NafaiBut I will finish vino tonight!00:32
jcastroNafai: tomorrow I think while you, ted, jan and cody try to fix the submenus I'll ask smithj to take on gnome-1-polkit so we can get that out of the way00:33
NafaiSounds good, since I haven't yet touched it00:33
bratschejcastro: I fixed up the gtk+ issues today, so let me know if you need me for anything tomorrow.04:01
jcastrobratsche: awesome04:01
jcastrobratsche: if we can fix submenus tomorrow that would be great04:01
jcastrobratsche: you should go to sleep now though, heh04:02
bratscheI'm doing some Javascript hacking now before bed. :)04:02
=== MacSlow is now known as MacSlow|afk
=== chaotic_ is now known as chaotic
=== MacSlow|afk is now known as MacSlow
=== MacSlow is now known as MacSlow|lunch
=== MacSlow|lunch is now known as MacSlow
jcastrojpetersen: good morning!13:59
jpetersenjcastro, good morning13:59
jcastrojpetersen: bratsche was able to fix the gtk problems yesterday which mean he should be able to help today with those menus14:00
jpetersenjcastro, ah yes ok, i am just trying to figure out, what gnome-bluetooth does with that menu items14:02
jcastroif we can figure that out today that gives us brasero and gnome-bt done, he nearly finished vino yesterday, and that just leaves policykit-1-gnome14:03
jcastrosmithj: when do you think you can finish policykit-1-gnome?14:03
jcastroseb128: when does the archive open back up, friday?14:04
seb128jcastro, it's soft frozen14:04
seb128but otherwise yes14:04
seb128we can still upload though14:04
jcastroseb128: g-s-d and hplip are ready 14:05
jcastrohplip is odd, we need till to look at it14:05
seb128g-s-d is waiting on a zillion lines patches for libgnomekb which I don't want to review :p14:05
seb128it's adding new sources etc to the code14:06
jcastrook so want to "save" that one for friday?14:06
seb128I doubt I will want to review it later too ;-)14:06
seb128but I will try to have a look14:06
seb128would be nice if upstream could review this one14:06
seb128I will try pinging svu14:06
seb128I don't get why so much code need to be changed right now14:06
seb128but I didn't really look at the issue and the bug is not really descriptive about that14:07
jcastroseb128: can you CC me if you mail him?14:07
seb128jcastro, I was going for the IRC ping rather14:07
seb128I avoid writting formal emails when I can use IRC14:08
jpetersenseb128, basically we cannot just change gkbd-status.[ch] because it is a public API of libgnomekbd, so it is some nicely refactored API from gkbd-status.[ch] which can be used by an application-indicator14:10
seb128jpetersen, did you upstream that change yet?14:12
seb128I will try to have a look later today14:12
jpetersenseb128, i will send it to upstream today14:12
jcastrojpetersen: any other work you have outstanding that needs review from seb?14:15
jpetersenthere is just g-s-d and connman-gnome I think14:17
jpetersenI updated the patch for https://bugs.launchpad.net/bugs/524150 and tested it with gnome-bluetooth14:26
ubot4Launchpad bug 524150 in indicator-application "Menus updated at runtime aren't rendered correctly (affects: 1)" [High,In progress]14:26
jpetersenIt should work now correctly14:26
tedgjpetersen: Cool, I merged it into my branch where I think we have all the fixes to make gnome-bt work correctly: lp:~ted/indicator-application/menu-updates14:36
tedgjpetersen: Just an FYI, you can build your own branches of any project in LP without commit rights to trunk.  It's not like git in that way, in that the branches can have independent sets of permissions.14:39
jcastromorning tedg!14:40
jpetersentedg, ah ok I will try it out next :)14:40
tedgjcastro: I think we've got a full set of patches for gnome-bluetooth now.14:41
tedgjcastro: We probably should get Nafai to confirm, but I've posted the branch for review.14:41
jcastrotedg: awesome, he should be around in a bit.14:42
seb128tedg, let me know if you need those in lucid14:42
jcastrotedg: that just leaves brasero, vino (which are both nearly done), and then just gnome-1-polkit14:42
jcastrotedg: and then that's basically it.14:43
tedgseb128: Archive is froze for A3, right?  I was thinking we'd just get them on Thursday release day when it opens again.14:43
jcastroseb128: will we be allowed to continue porting past A3? We have some left like epiphany, packagekit-gnome, etc. that aren't really in the default install14:43
seb128tedg, soft frozen we can get bug fixes in14:44
seb128jcastro, yes14:44
jcastrojpetersen: ok so after all this menu stuff lands you can continue with packagekit-gnome14:44
jcastrojpetersen: thanks for being flexible on this menu thing, I thought we were doomed14:44
jpetersenjcastro, yes ok14:47
qensejcastro: Maybe we should generate a report from the indicator-application tag with all applications that are in the default install.14:57
qense(and give those bugs the status Low instead of Wishlist)14:57
jcastroooh, I can do that.14:58
qenseIt would give the developers an overview of applications that have priority.14:58
jcastrogood idea14:58
* jcastro goes14:58
jcastroqense: I noticed you claimed gnome-1-polkit, have you started it yet?15:00
qensedid I?15:00
qenseI claimed guake and gnome-do15:00
qense(and am still working on Banshee)15:00
qensebut no gnome-1-polkit15:00
jcastrooh, my bad, you just triaged it15:00
jcastroignore me15:00
qensewill do!15:00
jcastroqense: in hindsight I should have done the bugs papercut style with different milestones15:01
jcastroand layed out over a timeline15:01
qensejcastro: Maybe that would have worked better, yes. Maybe something for the next release?15:02
qenseI'd say: write it down somewhere15:03
jcastroqense: ok done.15:10
jcastroseb128: I see mdeslaur did virt-manager, would that fall under you guys?15:10
seb128jcastro, not really, why doesn't he just upload?15:11
jcastroseb128: he's subbed the release team already: https://bugs.edge.launchpad.net/ubuntu/+source/virt-manager/+bug/52546215:11
ubot4Launchpad bug 525462 in virt-manager (Ubuntu) (and 1 other project) "FFe: Support Application Indicators (affects: 1)" [Wishlist,Triaged]15:11
seb128jcastro, subscribe the standard sponsor team to the bug15:11
jcastroseb128: release team is subbed, should I assign it to the canonical-desk-team?15:12
qensedo you think bug 526499 is because Transmission needs a X-GNOME-Delaything key?15:12
ubot4Launchpad bug 526499 in transmission (Ubuntu) "Transmission doesnt use indicator-application if present in the startup applications (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52649915:12
* jcastro isn't sure where the package lives15:12
seb128jcastro, no, let it this way for now15:12
jcastroqense: that's a question for tedg 15:13
qensetedg! What do you think of bug 52649915:13
ubot4Launchpad bug 526499 in transmission (Ubuntu) "Transmission doesnt use indicator-application if present in the startup applications (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52649915:13
tedgIs transmission using a custom fallback?15:14
vishdamn that was quick ;p15:14
qensetedg: nope15:15
vishqense: how the world did you notice that bug?  i just reported it 7 mins ago and you have found it in 2 mins o.0  15:16
vishhow in *15:17
qensevish: charles pointed me at it15:17
vishah.. :)15:17
jcastrovish: the man is a legend15:17
vishtruly ;)15:18
jcastroNafai: ping me as soon as you're around please.15:42
jpetersenhow do I link a gnome bug to lp?15:49
jcastrojpetersen: in bugzilla there is a "Add Bug URLs" text box15:51
jcastroon the bottom right, above the version dropboxes15:51
jpetersenjcastro, ah yes and the other way around with "Also affects project" in launchpad?15:54
qensejpetersen: yes indeed15:56
jpetersenok thanks15:57
jpetersenI will be just offline for dinner, will be back in some hours15:59
jcastrotedg: seb128: have you noticed keyboard scrubbing in the app indicator area is messed up now? when I get to the sound indicator it grabs the keyboard and starts adjusting the volume16:02
jcastrowhich doesn't make sense unless I stop at the sound one, press down, and then left/right16:02
seb128jcastro, yes, ronoc knows about it16:02
jcastrooh ok, cool16:02
seb128it's due to the fix to make the slider works16:02
* jcastro nods16:02
seb128works with keyboard16:03
seb128jcastro, bug #497909 is NOTABUG16:05
ubot4Launchpad bug 497909 in libwnck (Ubuntu) "Support Application Indicators (affects: 1)" [Low,Triaged] https://launchpad.net/bugs/49790916:05
seb128jcastro, it uses the notification area for a test program in the source16:06
seb128that's nowhere in the distro16:06
seb128and I don't think we need to add a testcase there16:06
seb128should I just close the bug?16:06
jcastroseb128: awesome, please invalid it16:06
* jcastro nods16:06
tgpraveen12 jcastro: is someone working on adding indicator for vlc?16:23
jcastronot afaik16:23
jcastrohmm, why would a video player need that?16:23
qensetgpraveen12, jcastro: VLC is Qt, if it would get support it should use KNotify, or a Qt equivalent if such a thing exists.16:29
qensealthough I'm in favour of getting rid of the thing as well.16:29
jcastrofrom what seigo mentioned to me if KStatusNotifierIcon is successful it might be a good thing to move into Qt at some point16:30
jcastroI don't really see the point of a video player having an icon there.16:30
hyperairi think some people use vlc as a media player akin to banshee16:30
hyperairvlc has a playlist of sorts, doesn't it?16:31
qensetedg: Sorry to bother you, but why is there one more argument in the g_signal_new call for the connection_changed signal (pasted here: <http://pastebin.ubuntu.com/382357/>) than the GLib documentation says?16:31
qensehyperair: yes, it does16:31
qenseyou can also add streams to that playlist16:31
qensee.g., you could have used it like you say during the UDS16:31
hyperairjcastro: and there's the point =p16:31
hyperairqense: UDS?16:31
qenseUbuntu Developer Summit16:32
jcastroI didn't know it has a playlist thing16:32
tedgqense: No good reason.  Probably got changed from something else.16:32
hyperairqense: no i mean what use case is this?16:32
jcastroqense: so like streams get queued and listed in the dropdown?16:32
hyperairqense: for the shoutcast streams or something?16:32
qensejcastro: no, more like bookmarks16:32
tedgqense: It should get ignored.16:32
qensetedg: Does it do any harm?16:32
qensetedg: I get (test:9378): GLib-GObject-WARNING **: /build/buildd/glib2.0-2.23.4/gobject/gsignal.c:2199: signal `' is invalid for instance `0x2293850' when I try to use the ConnectionChanged event in C#, I thought that was related.16:33
tedgqense: It probably should be "connection-changed" ?16:33
qensenot in C#16:33
tedgqense: Though, I'm unsure if the C# bindings change that.16:33
qensethere it is ConnectionChanged16:33
qense<attr path="/api/namespace/object[@cname='AppIndicator']/signal[@field_name='connection_changed']" name="name">ConnectionChanged</attr>16:34
tedgAh, okay.16:34
NafaiI'm in now16:44
seb128tedg, hum16:45
NafaiSorry, had labs this morning16:45
seb128tedg, can you do a merge request for application-indicator based on 0.0.13-0ubuntu7 16:45
seb128and not 0.0.13-0ubuntu216:45
seb128there has been 5 revisions since16:45
seb128tedg, or tell me the rev numbers to backport16:45
tedgseb128: I merged it... 16:46
tedgseb128: It just put the changelog in the middle.16:46
seb128ok, let me try16:46
seb128could be the email diff being weird16:46
tedgseb128: You can kill my changelog entries.16:46
Nafaitedg, So there is a full set of patches in menu-updates-lucid?16:46
tedgNafai: Uhm, no, it doesn't have jpeterson's latest patch in there.16:47
tedgNafai: Everything is being merged by seb128 right now, so best to just wait :)16:47
NafaiOk, I can do that :)16:47
jcastroNafai: and after that it's all up to you for today!16:47
jcastroNafai: can you finish off vino while they upload the new app indicator bits?16:48
NafaiYes, indeed16:48
Nafaimainly verifying I've got everything and I've followed the coding standard right now16:48
jcastroNafai: gnome-bt, brasero, and vino done today would be epic win16:48
NafaiYes, indeed16:48
jcastroNafai: that gives you the rest of the week to figure out this display properties crackrock16:48
seb128tedg, the application-indicator change is uploaded16:53
seb128jcastro, Nafai^16:53
Nafaiyou are awesome seb12816:54
seb128kudos to ted who did all the work16:54
seb128I just uploaded16:54
jcastroNafai: finish gnome-bt first since that is on the default panel16:54
=== jono_ is now known as jono
tedgseb128: Sweet!  Thanks!16:54
tedgseb128: Heh, now jpetersen did a significant amount of it too!16:55
=== jono is now known as Guest66435
* tedg thinks we now know jono's secret chat ID16:55
seb128kudos to jpetersen too then!16:56
seb128tedg, you scared him away now ;-)16:56
Nafaijcastro: Ok16:56
qensetedg: Again a question: in the monobindings metadata you have "<attr path="/api/namespace/object[@cname='AppIndicator']/signal[@field_name='connection_changed']" name="name">ConnectionChanged</attr>": it uses connection_changed with an underscore. However, APP_INDICATOR_SIGNAL_CONNECTION_CHANGED -- the name passed to g_signal_new() as the signal name -- is defined as "connection-changed". Can you use an underscore for an hyphen in the metadata?17:16
tedgqense: Honestly, I'm not sure.  bratsche or DBO may know?17:17
qensetedg: lets hope that. Thank you for your answering my questions.17:17
DBOqense, not sure but generally _ and - are not interchangeable17:17
qenseI'll build the libraries with the hyphens in the mono bindings metadata and will see if that works.17:18
NafaiSo confused17:38
jcastroNafai: how's it going? (sorry I'm going to be annoying today)17:38
NafaiI *think* I have the lastest version seb128 uploaded17:38
NafaiI recompiled, even restarted because I think I had a kernel upgrade since I last rebooted, but the menu is still rendered the same17:38
jcastrodo you have ubuntu8?17:39
jono_is there a content limit on a notify-osd bubble?17:40
Nafainope, I guess I don't :)17:40
jcastrosnag it from there17:41
jcastroNafai: I usually subscribe to lucid-changes and if I need something that hasn't been published yet that gets me a link to the lp packages17:41
jcastroNafai: someone can probably explain it better but there's a lag between "I've pushed this to lucid" to when it's built (based on the builder load) and then when it's published. Add to that a mirror lag.17:42
NafaiYeah, I'm subscribed to lucid changes, but hadn't thought to check there17:43
NafaiIt works!!!!!17:51
NafaiNow a final code review and make sure all of the code paths work17:51
NafaiThanks a ton bratsche, tedg, jpetersen, seb128 and everyone else :)17:51
jcastroNafai: be quick but be thorough, it would suck to have to ping pong with the desktop team over a mistake.17:52
jcastroie. err on the side of caution17:52
jpetersenjcastro, whom shall I assign LP #497904 to?18:17
ubot4Launchpad bug 497904 in connman-gnome (Ubuntu) "Support Application Indicators (affects: 1)" [Low,In progress] https://launchpad.net/bugs/49790418:17
jcastrojpetersen: canonical-desktop-team from now on 18:21
jcastrosmithj: around?18:22
jpetersenjcastro, ok i will do18:23
=== funkyHat is now known as crazyHat
qenseCould someone merge this already accepted merge request of mine <https://code.edge.launchpad.net/~qense/notify-osd/fix-465801/+merge/14265>? It has been lingering around for quite a while already and it would be nice if this issue would be fixed in time for Lucid.18:49
seb128qense, try pinging Macslow when he's around18:50
qenseseb128: ok, thx18:50
NafaiRight now I'm just looking to see what I have to do to get a checkbox updating correctly19:03
qenseNafai: there is a bug report for that19:06
NafaiOh, okay.  bug #?  I'd like to make sure it applies to mine.19:06
qensebug 52430819:06
ubot4Launchpad bug 524308 in indicator-application (Ubuntu) (and 1 other project) "Application Indicator doesn't process menu changes (e.g. checkboxes) (affects: 5) (dups: 2)" [Medium,Triaged] https://launchpad.net/bugs/52430819:06
Nafaiok, I can verify that choosing the option is indeed working, it's just not rendering correct19:08
Nafaiwhoops, wrong window :)19:12
Nafaiquick design question, perhaps jcastro or someone could answer since mpt isn't around19:12
Nafaignome-bt already has a insensitive menu item that indicates if bluetooth is on or off19:13
Nafaithe original applet additionally sets a tooltip that says (as far as I can see, these are the only options) Bluetooth: Enabled or Bluetooth: Disabled19:13
NafaiThat seems to be redundant, should I just forego replicating what the tooltip says?19:14
jcastroNafai: yeah we don't do tooltips.19:14
jcastroNafai: the icon should be sufficient 19:14
NafaiWell, I usually would just set the info in an insensitive menu item19:14
NafaiBut that same info is already there, just expressed slightly differently19:14
* jcastro checks his laptop19:15
jcastroNafai: I think just replicating the menu is fine19:16
NafaiOkay, I'll leave out the tooltip stuff19:17
jcastroI don't think it supports tooltips anyway19:17
NafaiYay, now time for cleanups19:17
jcastroseb128: get ready!19:17
jcastrotedg: chrisccoulson and I had some questions about gnome-settings-daemon19:19
tedgjcastro: K19:19
jcastroand how when you launch it it does that overlay on your screens19:19
chrisccoulsonhey tedg / jcastro19:19
tedgOkay.  For keybindings?19:21
chrisccoulsontedg - this is for the xrandr settings19:21
chrisccoulsoncurrently, when you click on the status icon to show the menu, g-s-d draws some identifiers on each screen19:21
chrisccoulsoni'm just wondering how we handle this with libappindicator19:22
* tedg looks19:22
tedgHmm, doesn't work with -nvidia19:23
jcastroyeah but it shows the overlay thing19:24
qensetedg: Try executing System->Preferences->Screen19:24
qenseit does give a warning, but it does draw the identifiers19:24
tedgThat's pretty crazy19:28
tedgDoes it list all the things possible in the applet there, or just ones you've prechosen?19:29
NafaiHow do I link to another bug in a comment in launchpad?19:30
Nafai(any special syntax or do I just put the full URL?)19:30
tedgOh, so it's just the rotations in the applet.19:31
qenseNafai: just type bug {bug_number}, or bug #{bug_number}19:31
NafaiOkay, thanks19:32
tedgI'm a little concerned that since we can't do the colored backgrounds, showing the rectangles at all would be confusing :-/19:32
tedgFor instance, with two "Acer 19"s how would the menu be useful?19:33
jpetersentedg, i thought about having the rectangles as an icon for the menu item?19:33
jpetersenso that would show the colour19:33
jpetersenbut we would need a signal when the menu is shown/hidden19:33
ubot4Launchpad bug 497856 in gnome-bluetooth (Ubuntu) (and 1 other project) "Support application indicator (affects: 2)" [Wishlist,Fix committed]19:34
Nafaijcastro: I'm probably closest on vino, should I go back to that and then brasero?19:35
tedgjpetersen: Yeah.  Hmmm...19:35
NafaiAs soon as I get initial feedback on that patch, I'll attach it upstream19:35
tedgjcastro: Have you been told whether we'd get any of mpt's time this week?  I know he's at the hackfest, but I'd hate to do something like that and have him come up with a really clever solution :)19:35
jcastrotedg: no idea, but looking at the calendar ...19:36
tedgThe colors seem bad overall, as it's an a11y issue.  And, I'm skeptical that the most important thing to do quickly is rotate your monitor?  Seems like adjusting size is *much* more useful.19:36
tedgThat's what OSX has in there.19:37
jcastrothe whole applet needs a UI redo IMO. but given then time and scope I think 1-for-1 for lucid should be the goal19:37
tedg(I think)19:37
jcastroand then for +1 fix it for real19:37
jpetersenyes I agree that the applet is not really good UI wise19:37
tedgFor 1:1, we need to get AboutToShow working in dbusmenu, which isn't simple though.19:39
qensejcastro: I think I've found the solution to the problem of all those people complaining about the new Application Indicators lacking functionality. AppInds are meant for allowing quick access to a small subset of the actions an application offers. If you want something else, something more complicated, then you should write a panel applet.19:39
qenseWe should tell that too all those people.19:39
tedgqense: The problem there is that panel applets are deprecated.19:40
qenseah, that's a problem indeed19:40
tedgUhg, GSD has turned out to be a problem all over the place.19:40
jcastrotedg: ok so we need to determine (probably today) if it's worth the risk to look into this (given the schedule and desktop team workload) or if we do like we did for Tomboy19:41
tedgjcastro: Yeah.19:42
tedgjcastro: Personally, I'm leaning toward don't touch it for Lucid, and fix it for Lucid + 1.  It seems kinda silly to port it to IA and the blow it up.19:42
jcastroI agree19:42
jcastrojono: any objections?19:42
* jono reads up19:43
jcastrotedg: the icon is off by default anyway19:43
jcastrochrisccoulson: https://bugs.edge.launchpad.net/ubuntu/+source/gnome-bluetooth/+bug/497856 is ready to go!19:43
ubot4Launchpad bug 497856 in gnome-bluetooth (Ubuntu) (and 1 other project) "Support application indicator (affects: 2)" [Wishlist,Fix committed]19:43
jcastrochrisccoulson: vino will be next19:43
jonojcastro, what specifically are you asking?19:44
jcastrojono: in display properties you can check a box and it shows you a label on each screen when you're like connected to a projector19:45
jcastroit uses custom UI (this is that xrandr thing we discussed last week)19:45
jcastrohttp://people.gnome.org/~federico/news-2010-02.html#12 <-- this thing19:45
jcastrojono: basically it's not a standard GtkStatusIcon thing and supporting the labels would be non-trivial19:46
jcastrotedg: ^^ right?19:46
tedgjcastro: jono: The labels, and we also don't today, have a signal for menu opened.  Plus, having an applet to rotate your screen (not set size) seems a little silly :)19:47
jcastrotedg: in other news, can you look at LP #52430819:50
ubot4Launchpad bug 524308 in indicator-application (Ubuntu) (and 1 other project) "Application Indicator doesn't process menu changes (e.g. checkboxes) (affects: 5) (dups: 2)" [Medium,Triaged] https://launchpad.net/bugs/52430819:50
jcastrojono: fwiw the rest of g-s-d is done (they keyboard bits), it's just this display thing that's the problem19:53
jonosorry have been on a call and network issues19:54
tedgjcastro: Hmm, we most of broken something.  That used to work.19:54
tedgs/most of/must have/19:55
jcastrotedg: ok I have a call with jono in an hour anyway I'll explain it to him19:56
qense:P My AppInd implementation for GNOME Do apparently is one big memory leak.19:57
qenseI couldn't do anything else but resetting my system.19:57
jonojcastro, can we do our call now?20:03
jonojcastro, I might drop off the wireless in a sec, so if you can, just call my phone20:05
jcastrojono: ok on it20:06
jonoready when you are20:06
=== crazyHat is now known as funkyHat
jcastrotedg: ok so jono doesn't feel strongly one way or the other and basically said "what does seb128 think?"20:40
jcastroseb128: gnome-bt is ready fyi!20:41
seb128jcastro, context?20:41
seb128think about what?20:41
jcastroseb128: the icon in the display properties thing20:41
tedgseb128: Wrong answer, try again "I agree with what ever tedg says"20:41
seb128jcastro, I would say do the opposite of what tedg suggested ;-)20:42
jcastroseb128: wondering wether it's worth the effort to port that "click me to rotate the screen" thing20:42
seb128is that complicated to do?20:42
jcastroseb128: actually he makes sense, I agree that it's  not worth porting and look at it for +1 since the whole applet UI is kind of crap20:42
tedgseb128: The problem is the colored backgrounds on the menu items, plus the "AboutToShow" for the menu coming up to do the floating windows.20:42
tedgseb128: I'm contending that most people don't need to rotate their screens quickly. :)20:43
seb128I don't know of anybody using tha ticon20:44
seb128that icon20:44
seb128I don't really care if it's not ported 20:44
* tedg is kinda curious if we should kill the checkbox to put it on the panel as well.20:44
jcastrook, let's leave it then.20:44
seb128does that reply to your question? ;-)20:44
jcastrotedg: I was thinking the same thing20:44
seb128what checkbox?20:45
tedgjcastro: It's probably the humane thing to do ;)20:45
tedgseb128: In the display settings, at the bottom.  It turns on the notification area icon.20:45
jcastroseb128: the "Show monitors in panel"20:45
seb128oh right20:45
seb128no strong opinion20:46
qensedo we support the Ctrl+An_arrow_key shortcut for rotating the screen?20:46
seb128I like our layout change better than federico's one btw ;-)20:46
qenseIf that is supported then the applet is less useful than it already was.20:46
jcastrotedg: I don't have a strong opinion either, I think we should just leave it. It's off by default and for +1 we need to look at that entire applet anyway20:48
tedgjcastro: Sounds good to me.  You gonna schedule the UDS session? :)20:48
seb128no need of a session for such small changes20:49
jcastrotedg: I'd rather revisit it for +1 than carry a patch for 3 years because the app is full of crack. :D20:49
tedgseb128: Heh, I was told that app indicators are life changing (not in a good way) :)20:50
jcastrotedg: fixing 524308 is a much better investment. \o/20:51
seb128tedg, yeah, we don't do UDS session for life changes things20:51
seb128tedg, we do just those for fun topics usually20:51
tedgSadly, the paintball session has been thrown out every year...20:52
qenseWe should held the next DX meeting in Amsterdam... :) I'm curious what the results will be of such a session.20:53
jcastrooh hey qense 20:55
jcastroI made something to show people who don't like the idea20:55
qensejcastro: wonderful!20:55
qensenow spam it all over the place!20:55
Nafaijcastro, Nice! :)20:56
* Nafai tries out transmission20:57
NafaiYay for app indicators!20:57
qensejcastro, it's the most eloquent explanation of the reason we've got AppInd I've seen so far!20:58
qensechrisccoulson, do you think bug 526499 can be solved with a "X-GNOME-Autostart-Delay"?21:00
seb128bug #52649921:00
ubot4Launchpad bug 526499 in transmission (Ubuntu) "Transmission doesnt use indicator-application if present in the startup applications (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52649921:00
qenseis the dash really needed?21:01
seb128qense, what dash?21:01
jcastroseb128: vino will be done next21:01
qensethe hash?21:01
jcastrowhat you didn't finished gnome-bt yet? :)21:01
qenseI used the wrong English word21:01
chrisccoulsonqense - i'd prefer not to do that21:06
chrisccoulsonthat seems like a hacky solution21:06
chrisccoulsoni'd rather fix the real issue, which is that it falls apart if transmission starts before indicator-application-service21:07
qenseshouldn't the library launch the service over DBus?21:07
jpetersen_qense, that bug also happens with other autostarted applications like gnome-settings-daemon and gnome-power-manager21:08
qensejpetersen_: do you know if there are bugs reported for those?21:08
jpetersen_qense, I think not yet21:09
qenseok, then I'll assign the Transmission bug report to Indicator Applicaiton21:09
seb128that's a design thing21:09
seb128did you ask to ted?21:09
qenseI haven't got a response from him on the issue, but I did ask. I think it got lost in the many other questions people ask him.21:11
tedgqense: Sorry.21:12
qenseno point21:12
tedgqense: Should the application start indicator-application-service?21:12
qensetedg: here it is causing problems21:12
qensebut it might be desired behaviour when people remove the applet21:12
qensefrom their pannel21:12
tedgqense: I was thinking no, as if they haven't got the applet, they're effectively voting no.21:13
tedgqense: Plus there is no way to start applets externally.21:13
tedgqense: We could start the service, but not do much with it.21:13
tedgqense: This is another reason gnome-panel needs to die :)21:13
qensethat's true, but it would require the autostart applications to use the X-GNOME-Autostart-Delay key21:14
qensewhich is a bit hacky21:14
qenseisn't there a way to give the service priority over other applications?21:14
tedgqense: No, we need to fix it so that they'll wait a bit.21:14
tedgqense: They already don't fallback immediately.21:14
tedgqense: There's a bit of hysteresis so that if the service dies, the user shouldn't notice as it'll get restarted. 21:15
tedgqense: The applet will restart it immediately.21:15
qenseI've noticed that already. :) it's unkillable!21:15
qenseso a delay key it is, you say?21:16
qenseoh, chrisccoulson!21:16
chrisccoulsoni'll have a look at the scrollback in a minute or so ;)21:17
tedgqense: I think we should probably do it in the library really.21:32
qensetedg: In that case, shall I assign the bug report to indicator-application and paste this log in the description?21:34
tedgqense: Hmph.  I'm allergic to bugs :)21:34
qenseI'll only subscribe you to it!21:35
qenseA lot of the bug reports have titles of the type "Indicator Application doesn't support ..."21:37
qenseso this is a change!21:37
qensetedg: While I'm busy burdening your shoulders with one more bug I would like to ask your opinion about the validity of two bugs: bug #522146 and bug #522153. The titles explain enough.21:38
ubot4Launchpad bug 522146 in indicator-application (Ubuntu) "indicator-application does not support icon by filename (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52214621:38
ubot4Launchpad bug 522153 in indicator-application (Ubuntu) "indicator-application does not support text/GdkPixbuf as an icon (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52215321:38
qenseDo you think these are valid, or should the be Won't Fix, considering the fact that this is behaviour we would like to prevent.21:39
qenseBtw, I should reintroduce myself now:21:39
tedgqense: I think that those could be wishlist.  We'd really like to avoid them as in both those cases we can't theme the icons.  And since we don't provide enough information about the theme to the application, they can't theme them, and to fix that -- well, that'd be a lot of information.21:39
qenseHi! I'm qense, I'm the new Application Indicator adoptee in Ubuntu! I'll be your downstream contact person regarding bugs!21:40
tedgqense: I think that in both cases they enable things that we can't do now though.21:40
tedgqense: Cool!21:40
tedgqense: I feel like we can't reject them because we don't have  good answer for those use cases.21:40
qenseI'll mark them as Wishlist then.21:41
tedgqense: Thanks!21:41
qensetedg: I'm subscribed to the bug reports for AppInd in Ubuntu and will try to keep them manageable. If there is anything you'd like me to do with them, or if you need something form them, please tell.21:41
qenseI could also write an Apport hook if that's necessary21:41
jcastrotedg: you said you wanted bug help. *cough*21:46
tedgqense: Great, thank you for helping out.  An apport hook would be cool, though, I'm not sure what we could collect.21:47
tedgqense: Perhaps the ~/.cache/indicator-applet.log21:47
tedgHmm, we really should put logging in the service.21:47
qensetedg: I'll have a look at a hook then21:49
qenseI call the day a night and am off22:00
jcastrosmithj: around?22:05
seb128jcastro, if you want to build a list of issues, bug #526552 is one due to the appindicator change22:20
ubot4Launchpad bug 526552 in rhythmbox (Ubuntu) "Music->Close menu item and Ctrl-W do nothing (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52655222:20
jcastroseb128: ok I'm just going to compile them under the "indicator-application" tag22:23
jcastroseb128: and then assign them to the person who did the patch22:25
jcastrotedg: check this out: wrt. missing track details: https://bugzilla.gnome.org/show_bug.cgi?id=606972#c822:25
ubot4Gnome bug 606972 in User Interface "Support for application-indicators/StatusNotifierIcon" [Enhancement,Unconfirmed]22:25
seb128jcastro, thanks22:30
tedgjcastro: Yeah, I think that makes sense.  Just an insensitive item.22:36
RAOFAh!  I think I may have found qense's gnome-do stack overflow.  It's a pity qense seems to be in the exact opposite time zone to me.22:50
RAOFI wonder if there's a bzr branch available...22:56
jcastroNafai: how's vino coming along?22:57
jcastrosmithj: around?22:57
Nafaijcastro, Pretty well, found some issues (no bugs in outside stuff) in the way I did some things as I tested things.  So I'm going through and cleaning things up22:58
seb128Nafai, don't use fix commited if the fix is not commited yet23:02
seb128nobody is going to review it though23:03
seb128ie bug #49785623:03
ubot4Launchpad bug 497856 in gnome-bluetooth (Ubuntu) (and 1 other project) "Support application indicator (affects: 2)" [Wishlist,Fix committed] https://launchpad.net/bugs/49785623:03
seb128though -> otherwise23:03
NafaiSorry, just following what I was told23:03
NafaiSo, don't change status, just assign to canonical-desktop-team after the patch is attached?23:03
seb128jcastro, ^ is that you who adviced that?23:04
seb128Nafai, we use triaged or in progress usually23:04
seb128those for work being done23:04
jcastroI thought we just assigned it to canonical-desktop-team?23:04
Nafaihttps://wiki.ubuntu.com/DesktopExperienceTeam/ApplicationIndicators/ContractorWorkflow <- See #323:05
seb128the fix commited status is used when fix is commit upstream or to the ubuntu vcs23:05
* Nafai nods23:06
seb128Nafai, ok, I'm not blaming you in any case don't worry23:06
Nafaithanks ;)23:06
seb128but it will lead people to think the change has been reviewed and commited23:06
seb128and nobody will look at those again23:06
jcastroseb128: we can change it to what is right23:06
Nafaimakes sense23:06
seb128jcastro, you can use in progress rather23:07
jcastrook fixing23:07
seb128jcastro, thanks23:07
seb128bug #52430823:08
ubot4Launchpad bug 524308 in indicator-application (Ubuntu) (and 1 other project) "Application Indicator doesn't process menu changes (e.g. checkboxes) (affects: 5) (dups: 2)" [Medium,Triaged] https://launchpad.net/bugs/52430823:08
seb128is that being worked?23:08
jcastrothat is the one you uploaded today, that should be fixed committed23:08
seb128dx could do a better work at triaging ;-)23:09
jcastroseb128: qense just adopted indicator-application so he's going to be helping them with that23:09
Nafaiqense rocks23:09
smithjjcastro: am now23:46
smithjjust read your message23:47
smithji haven't started on gnome-1-polkit yet, so if jan isn't busy you might want to reassign23:47
jcastrosmithj: ok, when can g-d-u be ready?23:48
seb128jcastro, bug #52674723:50
ubot4Launchpad bug 526747 in indicator-applet (Ubuntu) "battery applet popup is empty (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52674723:50
seb128jcastro, you might want to assign that one too23:50
smithjjcastro: i'll work on it today. probably today or tomorrow23:51
jcastroseb128: done23:52
seb128jcastro, thanks23:52
smithji didn't get a chance to work on it last night. i'm on call at my day job and we had an... incident :-/23:52
jcastrosmithj: ok please assign "canonical-desktop-team" to it as soon as it's done23:52
smithjok. not assigning to ken?23:52
smithjkenvandine: you slacker23:52
jcastronope, just the team23:52

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!