/srv/irclogs.ubuntu.com/2010/03/04/#ayatana.txt

Tiibiidiihi, i just saw the new theme, and instantly felt the window controls very very backwards.... then i read one comment on jono's blog that laments by mentioning fitt's law... then i thought: since the close windows action is often supported by a known key shortcut, but the other window actions aren't... may this be better?: to have the least useful (meaning that it's less often used and it's already backed up by a s01:01
Tiibiidiihortcut) action put in a less easily accessible position, and meanwhile promoting the other actions in the corner...01:01
Tiibiidiiright now i tweaked my gconf settings to replicate the future lucid theme01:02
Tiibiidiii'll try to keep it this way for some days to see if at the end it'll continue to seem so backwards to me, or if it's instead a good idea01:03
Tiibiidiiin the meanwhile, can someone point me to the reasoning that sits behind this change?01:04
Nafaijcastro: around?02:23
Nafai(I know it's late for you)02:23
jcastroNafai: yeah02:29
jcastrowhat's up?02:29
NafaiShould I assign bug #497853 back to pitti or to canonical-desktop-team?02:29
ubot4Launchpad bug 497853 in brasero (Ubuntu Lucid) (and 2 other projects) "Support application indicators (affects: 1)" [Wishlist,Incomplete] https://launchpad.net/bugs/49785302:30
jcastropitti02:30
jcastroi'll ping him tomorrow early and see if that's what he wants02:30
Nafaiok02:32
lamalexi posted a patch for https://bugs.edge.launchpad.net/hundredpapercuts/+bug/46028604:41
ubot4Launchpad bug 460286 in empathy (Ubuntu) (and 2 other projects) "enable notifications when chat is not focused (affects: 3)" [Wishlist,Triaged]04:41
vishkenvandine: ^^04:58
=== mpt_ is now known as mpt
=== MacSlow is now known as MacSlow|lunch
jcastroseb128: woo, brasero is in!13:46
seb128jcastro, only vino left?13:47
jcastroand hplip13:47
jcastrobut hplip has been through the pitti gauntlet already, needs an update13:47
jcastrosmithj: ping13:47
seb128ok13:48
jcastroseb128: all that's left distro-wise is adjusting the power menu like mpt wanted and after that it's all bugfixes as they come in and getting the patches upstream13:48
seb128jcastro, good13:49
NafaiGood morning13:52
NafaiI'm going to go ahead and pass brasero and gnome-bluetooth patches upstream this morning13:52
jcastroNafai: anything left to do on vino on your side?13:52
Nafainope, I've already passed the updated patches upstream and attached to the launchpad bug13:53
Nafaibrb13:56
Nafaiback14:03
seb128hey Nafai14:04
seb128I commented on your gnome-bluetooth change14:04
seb128I think we didn't understand each other yesterday14:04
NafaiOh14:04
* Nafai looks14:05
seb128I suggested changing the menu directly14:05
seb128ie the .ui14:05
seb128not doing weird code hacks14:05
Nafaiwell, for the items that us stock-id, if I change it to not use stock-id, we lose translations14:05
Nafaiplus, if I change the .ui, that makes it less acceptable as a patch to upstream, because they would still want the icons for the GtkStatusIcon as well14:07
seb128well that's why I asked you yesterday14:08
seb128is there any reason why they would want icons and we don't14:08
seb128we use the same menu is both cases14:08
seb128if icons don't look nice they should be cleaned in both cases14:08
seb128no?14:08
NafaiI'm trying to find a screenshot to see how they looked in the old one14:10
Nafaiactually, in Ubuntu at least, the status icon respects the no-icon menu setting14:12
Nafaiso they aren't shown anyway14:12
seb128why doesn't the indicator do that?14:12
seb128isn"t that an indicator bug?14:12
Nafaiwe've talked with tedg about it, and I think it is, but he also says that (and please correct me if I mis-remember jcastro) the applications should also handle that themselves14:13
NafaiFrom the screenshot I see on gnome.org (http://library.gnome.org/users/gnome-bluetooth/stable/gnome-bluetooth-applet.html.en), it renders fine normally14:14
jcastroI don't remember14:18
jcastroNafai: it's like UI freeze today so it's like our last chance to get it right14:18
Nafaiwhat's your take on this jcastro?14:20
NafaiFor the record, I don't disagree with seb128, I just want to do the right thing14:20
seb128jcastro, well making the applet respect the show icon flag would be a bug fix not an ui change14:21
seb128even if side effect is to fix an ui which was buggy14:21
jcastroseb128: yeah I know, I just want to make sure we do it right today and not have this come up tomorrow, monday, etc.14:22
seb128let's wait on ted14:22
jcastroNafai: when in doubt I always do what seb128 wants and everything works out. :D14:22
jcastroat least brasero is done, what a pain in the neck14:22
NafaiOur conversation from the other day: http://paste.ubuntu.com/388287/14:24
seb128speaking of who14:26
seb128tedg, hy14:26
seb128hey14:26
tedgGood morning.14:28
seb128tedg, so we were talking about the bluetooth indicator14:28
seb128and stock icons14:28
seb128is indicator applet not respecting the show icon option a known bug?14:28
seb128will it be fixed for lucid?14:28
tedgIt's known, but I forgot about it :)14:29
tedgIt shouldn't be too hard to fix.14:29
jcastrotedg: what's your schedule like today? It would be nice to get this fixed14:34
jcastrough, it's thursday14:34
tedgHeh, jcastro you're getting the rhythm ;)14:35
=== MacSlow|lunch is now known as MacSlow
seb128Nafai, ^14:49
seb128jpetersen, hi14:59
jpetersenseb128, hi15:00
seb128jpetersen, thanks for the rhythmbox update15:00
seb128jpetersen, it doesn't escape markers though15:00
seb128the <i></i> are displayed15:00
jpetersenseb128, hm ok15:00
seb128rather than having those words in italic15:00
seb128did it works for you?15:00
jpetersenseb128, it was not shown here15:05
jpetersenseb128, I will just test it again15:05
seb128jpetersen, thanks15:06
NafaiOk, brasero passed upstream15:06
Nafaibrb15:06
hyperairis application indicator supposed to use that much padding?15:24
* hyperair really dislikes having that much padding for anything.15:24
jpetersenseb128, ah ok I can reproduce that problem, I will fix it15:28
Nafaihyperair: You mean between icons?15:29
seb128jpetersen, thanks!15:29
seb128Nafai, so if ted fixes the show icon issue, is there anything left to change in bluetooth?15:37
Nafaiseb128: Nope.  So that should take care of #528527, nothing needs to be done with the package, right?15:39
hyperairNafai: yes, between the icons.15:40
seb128Nafai, ok, what I think too, good ;-)15:41
Nafaiso I'll go ahead and pass the existing patch (the one already packaged) upstream15:42
seb128Nafai, thanks15:43
Nafaiseb128: I actually was just reviewing your comments to the original bug for app indicator support and noticed I had forgot to change this to APP_INDICATOR_CATEGORY_HARDWARE15:54
NafaiI should probably do that and double check the other things were done15:54
seb128ok15:54
NafaiI've got a call here in a few minutes, but I can quickly make those changes after15:54
tedghyperair: There is supposed to be more than in the notification area, but I'm worried that I might have too much currently.  I haven't investigated though.16:01
tedghyperair: I think something might be getting padded twice.16:01
hyperairtedg: i see. what's the padding for?16:02
hyperairtedg: isn't the notification area cluttered enough without padding stretching it even wider?16:02
tedghyperair: Just to make it easier to hit the items.  And to make them look nice.  (probably more the second)16:02
hyperairheh16:02
hyperairalright16:02
* hyperair will need to undergo panel appletsectomy after upgrading to lucid16:03
seb128fedora did extra padding change previous cycle in their notification area too16:03
hyperairhmm, did they?16:04
seb128yes16:04
hyperairbut they don't have appind do they?16:04
hyperairso this padding is for the notification area icon alone?16:04
seb128no16:04
seb128yes16:04
hyperairi mean notification area16:04
seb128http://blogs.fedoraproject.org/wp/mclasen/2009/10/26/5-little-things/16:04
seb128the second item on the list there16:05
jpetersenseb128, I updated the rhythmbox patch to fix the markup issues16:05
seb128jpetersen, oh, nice, looking16:06
hyperairseb128: interesting.16:07
jpetersenI will be back soon16:13
qenseIs bug #528088 valid for all AppIndicators?17:37
ubot4Launchpad bug 528088 in transmission (Ubuntu) "Transmission applet does not show when there are no indicator applet (affects: 1)" [Undecided,Confirmed] https://launchpad.net/bugs/52808817:37
seb128qense, indicator fallback to a notification area icon17:39
seb128that bug is weird17:40
seb128I usually have the opposite issue17:40
seb128I've to make sure the service is not running to have the fallback working17:40
seb128ie removing the indicator applet with the service still running breaks17:40
qenseseb128: I've stumbled about your issue a few time as well.17:41
seb128but once you stop the service it works17:41
qensethat's true17:41
qensethe service keeps running17:41
qenseseb128: It would be great if there was some way for applications to subscribe to the availability of a certain application, something like libunique, so they reliably know when something is running and when not.17:43
qenseBoth the service and the applet could use it.17:43
qenseand probably many other applications as well.17:43
seb128qense, bug #532044 is a duplicate17:46
ubot4Launchpad bug 532044 in rhythmbox (Ubuntu) (and 1 other project) "When Rhythmbox is launched, all you see is the tray icon (affects: 1)" [Undecided,New] https://launchpad.net/bugs/53204417:46
seb128please close it17:46
qenseseb128: will do, thanks for mentioning17:46
Nafai  What currently happens if you set the status to attention and there isn't an attention icon set?  Does it just stay with the ACTIVE status?17:51
qenseI've dumped bug #527282 in gtk+2.0, but is that the right place for it?18:11
ubot4Launchpad bug 527282 in gtk+2.0 (Ubuntu) (and 1 other project) "GtkWindows cannot be raised, show() doesn't do the trick (affects: 1)" [Undecided,New] https://launchpad.net/bugs/52728218:11
qenseThe issue is with the fact that when you press Show Window in the tray menus the show() function is used, which doesn't raise the window.18:12
qenseBefore AppInd you could raise the hidden windows from the tray.18:12
NafaiOkay, I just finished #53210419:03
Nafaibug #53210419:03
ubot4Launchpad bug 532104 in gnome-bluetooth (Ubuntu) "Fix app indicator classification and other minor issues (affects: 1)" [Undecided,In progress] https://launchpad.net/bugs/53210419:03
NafaiAddresses feedback seb128 gave me before that I missed19:03
jcastroNafai: woo!19:05
NafaiIf this is good, I'll pass it upstream19:05
jcastroNafai: tomorrow I was thinking of sending the brasero and vino guys a mail19:06
jcastroif we could have a call I'd like to get some things straight19:06
NafaiK, vino has been sitting there for like a week or something19:06
Nafaisure thing19:06
jcastroexplain why it's so invasive, etc.19:06
jcastroyeah19:06
Nafaipitti and I successfully used Empathy/gtalk earlier today19:07
Nafaiworked well19:07
jcastroyeah it's getting much better19:09
jcastrolast cycle I couldn't use it at all but it's pretty good now19:09
jcastroall it really needs is voice cancellation19:09
Nafaiyeah19:09
NafaiI'm going to go grab lunch19:13
NafaiThen I'll need to figure out what's next :)19:13
Nafaiback20:07
Nafaijcastro: what would you suggest I work on next?20:07
jcastrowell, no outstanding bugs at the moment. 20:07
jcastroMaybe qense needs help with the banshee work?20:08
jcastroNafai: we're at UI freeze so it doesn't really make sense to port new apps20:08
Nafairight20:08
Nafaiqense: Around?20:09
jcastroseb128: anything you guys need help with?20:09
qensejcastro, Nafai: I'm around20:10
NafaiNeed any help with Banshee?20:11
seb128jcastro, landing the new artwork in lucid? ;-)20:12
qenseBanshee itself is not that much of a problem, I think it would be best if I would finish that myself since I've made myself acquainted with the code. However, libappindicator-cil is the real time drainer.20:12
qenseseb128: Plymouth should be there already, I'm anxiously waiting for the rest!20:12
seb128jcastro, out of that nothing I can think about today no20:12
jcastrook20:12
seb128qense, yeah I know20:12
seb128qense, wallpaper too20:12
qenseyes, forgot that one20:12
seb128qense, I just uploaded the gnome-panel icon theme20:12
jcastrotedg: how are your indicator-application bugs? perhaps Nafai can lend a hand when he's not fixing the apps themselves?20:12
NafaiYeah, I don't mind diving in20:13
qenseAlthough I have to say that I am closing in on the Mono binding problems with AppInd.20:13
NafaiI'm not very experienced with C# or Mono, so I'd probably be less helpful with binding work20:13
qenseNafai: how much do you know of C# and GAPI?20:13
Nafaiheh. :)20:14
tedgjcastro: I know of three.  The fallback thing in gpm.  The fallback on startup.  And then the python append one.20:14
qensetedg: is the transmission bug the only report about the fallback on startup?20:14
qensetedg: then I'll convert it to a more generic bug report20:14
tedgjcastro: So, I'd say that there isn't a huge problem there, but help is always appreciated :)20:14
tedgqense: That's the only one I've seen it on, but I can't imagine it's transmission only.20:15
qensetedg: I'll convert it then.20:15
tedgqense: That'd be just too weird :)20:15
jcastroNafai: ok so see if you can help tedg 20:15
Nafaicool, thanks.20:15
Nafaitedg: Which would you like me to look at?20:15
tedgNafai: The Python one would probably be best to start out, it's blocking apps.20:16
tedgNafai: The others are annoyances (that should get fixed)20:16
qensejcastro: I just had a busy week, but I hope to be able to finish it this weekend. Although I have said before that Banshee was almost finished. :)20:16
NafaiOkay.  Bug # and branch to work on?20:16
jcastroheh20:16
qenseatm I'm writing a blog to advertise AdoptUpstream20:17
jcastrohttps://bugs.edge.launchpad.net/ubuntu/+source/indicator-application/+bug/53013820:19
qenseI hope to have found a working solution for libappind-cil tomorrow so I can have a working library at least ready for personal use Saturday which means I can continue to work on the aptch.20:19
jcastrotedg: Nafai ^^^20:19
ubot4Launchpad bug 530138 in indicator-application (Ubuntu Lucid) (and 2 other projects) "Using .append() on a gtkmenu doesn't update the indicator's menu (affects: 1)" [High,Triaged]20:19
qenseWhat about the .set_menu() crash?20:19
jcastromarc needs that bug fixed so he can finish off app indicators for virt-manager20:19
Nafaitedg: Is this the branch I should base upon? https://code.launchpad.net/~indicator-applet-developers/indicator-application/lucid20:29
tedgNafai: No, please use lp:indicator-application20:29
Nafaiok, thanks20:29
Nafaiwee, now to figure out bzr :)20:30
tedgNafai: All you really need is:20:31
tedgbzr branch lp:indicator-application20:31
tedg<do work>20:31
tedgbzr commit20:31
tedg<do more work>20:31
tedgbzr commit20:31
tedgbzr push lp:~nafai/indicator-application/my-fix20:31
Nafaicool20:31
NafaiI've made the mistake in the past just doing bzr checkout and finding out bzr commit doesn't work so well :)20:31
qenseNafai: bzr commit --fixes lp:{bug_number} associates a commit with a certain bug20:32
* Nafai nods20:32
NafaiI think I figured out what may be going on.  GtkMenuShell.insert may not be emitting the add signal which is what the app indicator hooks up to21:37
Nafaibut calling GtkContainer.add *does* emit the signal (of course), which then causes app indicators to get it21:37
NafaiI'm doing some more digging to be sure21:38

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