/srv/irclogs.ubuntu.com/2010/08/10/#ayatana.txt

meebeyI have a question regards handling "seen" behavior of notifications which is not covered in the NotifyOSD wiki01:53
meebeysay for an IM app there is a private message received and the user is current not in the window window, a notification is shown01:54
meebeynow if the user goes back to the application (say because the notification is not supporting actions) should the app clear the notification as soon as the window is focused and the user is seeing the message?01:54
meebeykenvandine: you are around?03:35
meebeykenvandine: see twitter replies :)04:40
klattimermorning all08:18
didrockshey klattimer, how are you?08:22
klattimerdoing good08:22
klattimergonna have to split up a patch today and try and wangle it into a partly already patched bit of code08:23
klattimer*sigh* a programmers work is never done08:23
didrocksklattimer: yeah you can always refactoring and such ;)08:23
didrocksklattimer: I'm cleaning openoffice package this week, I can ensure you it's not a joy as well :)08:24
klattimerdidrocks: ouch, ooo is hell I'm sure08:24
didrocksklattimer: yeah, the makefile for the packaging (debian/rules) is more than 3 000 lines :/08:24
klattimerI can believe it08:24
klattimermorning seb12808:35
seb128hey klattimer08:36
=== DrPepperKid is now known as MacSlow
klattimerseb128: I can't seem to find the libappindicator with the set_label api in launchpad10:41
klattimerany ideas where it might be?10:41
seb128klattimer, https://code.edge.launchpad.net/indicator-application/10:42
seb128I guess lp:~ted/indicator-application/labels-in-the-indicator10:42
seb128or, and lp:~ted/indicator-application/label-in-the-lib10:43
klattimeroh I ended up on https://code.launchpad.net/ubuntu/+source/indicator-application10:43
seb128right that can be confusing on launchpad10:43
seb128if you browse from the packages you land on ubuntu specific things10:43
seb128not upstream projects which is what you want there10:43
klattimerk10:47
klattimeralthough launchpad is great10:47
klattimerit has some minor issues like this I'm slowly learning10:47
=== gord_ is now known as gord
djsiegelvish: oing11:42
djsiegelping*11:42
vishdjsiegel: hey.11:42
djsiegelvish: found a juicy paperctu11:47
djsiegelpapercut even11:47
vishdjsiegel: cool!11:48
thorwilinteresting metaphor-combo :)11:48
djsiegelmmm, taste the delicious papercut juice11:50
djsiegelvish: https://bugs.edge.launchpad.net/hundredpapercuts/+bug/61579311:53
ubot5Launchpad bug 615793 in One Hundred Paper Cuts "For each file received over bluetooth, a dialog is opened and must be dismissed (affected: 1, heat: 6)" [Undecided,New]11:53
vishhmm , i think there was a notify-osd spec for that already..11:54
vishmorphing windows?11:54
bdrungi am testing the ubuntufont. it looks ugly in the gnome-terminal, because the terminal tries to use it monospaced. what the plan for monospace?12:42
klattimerbdrung: you should never use a non-monospace font in a terminal because it'll bork badly12:44
klattimerswitch to another font, I'm not sure that the ubuntufont even has a monospace one planned12:45
bdrungklattimer: that's the question: is there a monospace version planned?12:47
klattimerdunno12:48
klattimermonospace fonts are hard as they require very very intrictate understanding of how monospaced fonts are applied12:48
klattimere.g. the { is better for programmers when it's very OBVIOUSLY a { same with 0 which should be struck through or dotted to make it clear12:48
=== MacSlow is now known as MacSlow|lunch
bratschebdrung: I've never heard that there is a monospace version planned.  I suspect there is not.13:42
bratschebdrung: Actually, I was just told by a couple guys on the design team that they think there is a plan for it, but I don't think there is any timeframe for it.13:43
bratschebrb13:44
bdrunghaving it on the todo list is better than nothing13:45
klattimerdavidbarth: I'm currently hacking on the gsd indicators and there's a bit of a problem13:49
klattimerit seems libgnomekbd might need patching in order to indicate the current layout13:50
klattimerseems it has a subclass in there for the status_icon13:50
klattimer:/13:50
davidbarthklattimer: hi13:50
klattimerso I doubt i'm going to be able to get this done by tomorrow13:51
davidbarthhmm, what about wrapping that class into an app. indicator version but at gsd's level?13:51
davidbarthhmm, or13:51
davidbarthklattimer: do you have an idea who else may be using that libgnomekbd class?13:51
klattimerdavidbarth: it looks like it isn't used by anyone else13:52
klattimerit uses cairo to draw the surface13:52
klattimerI'm trying to find a way to get the current layout text out of the thing to set as a label13:52
davidbarthwhat i'm wondering is if a g-s-d patch only gives that change a better chance of being accepted upstream as a build dependency / or something easier to distro-patch13:52
klattimerbut then I have the problem of how/when to set the label13:52
davidbarthas opposed to messing up with a lib api signature13:53
klattimeri think you're right13:53
klattimerhacking the lib would be problematic13:53
klattimerbut I doubt anything else is using the gkb_status_icon13:53
davidbarthwdym with how/when to set the label?13:54
davidbarthif done at the library level? or when youṕ re in g-s-d?13:54
klattimerwell, in the status icon, the label changes on click, or by the menu13:55
klattimerbut with our indicators it might be slightly more problematic13:55
klattimerthis has a lot of weird code crossover between libgnomekbd and gnome-settings-daemon13:55
klattimerit's not very well segregated at all13:55
davidbarthwell, i don't have the code handy right now13:56
klattimerI'm printing the function which I believe does the work13:56
klattimerI'll pull it apart and find the bits which matter13:56
davidbarthbut i'd say keep the lib manage the model side of the story, and duplicate what it'd take in g-s-d to cleanup the "view" side13:56
klattimersee how hard/easy it'll be to make gsd do this for app-indicators13:56
klattimerI really don't want to touch libgnomekbd13:56
davidbarthright13:57
seb128didn't the patch from jen in lucid did that?14:00
seb128ie was touching only g-s-d14:00
seb128libgnomekbd is also used in the gnome-screensaver unlock dialog to indicate the keyboard layout14:01
klattimerdavidbarth: seb128 I think I might be able to do it without libgnomekbd but it's going to be messy :/14:18
seb128we had a patch working last cycle, can't you just reuse that and use the new api to the set the label?14:19
seb128the only issue it had is that it used an icon and that didn't reflect the layout14:19
seb128because there was no set_label yet14:19
klattimerseb128: that's what I'm trying to do14:20
klattimerbut the problem is that libgnomekbd keeps all the labels private in a "globals" struct14:20
klattimerand has no method to get_short_name_for_group(int group) or some such14:20
klattimerwhich I could use directly14:20
klattimerinstead I'm going to have to recreate some of the code from libgnomekbd in order to get it to work14:20
seb128right14:21
seb128it's suboptimal but it seems there is no better way14:22
klattimerseb128: take a look14:22
klattimerhttp://paste.ubuntu.com/475924/14:22
klattimerthis is the code I've lovingly extracted from libgnomekbd14:22
klattimerin libgnomekbd this code is spread across so many different parts of the file, I'm not sure if there'll be some kind of dependency on the order of things or what14:23
klattimerbut that's basically what's needed in order to get the short names out for the label14:23
seb128ok14:23
seb128session restart brb14:23
davidbarthklattimer: things live in libklavier actually, right? so the libgnomekbd is just a wrapper around the actual model data; so i think that's fine to shortcircuit it14:24
klattimerdavidbarth: that's pretty much the plan14:33
klattimeralthough it's ugly14:33
klattimerand requires a bit of digging14:33
klattimernot sure I can get this done by feature freeze :/14:33
=== MacSlow|lunch is now known as MacSlow
davidbarthklattimer: you mean this week?14:37
klattimeryeah14:37
davidbarthwell, log off or /away; we'll let you work now ;)14:38
klattimer:)14:38
klattimerits OK, I've got multiple machines here14:38
klattimerit's like the setup in swordfish in my office14:39
klattimerexcept the multiple displays aren't arranged in a particularly cool way14:39
meebeyis appindicator a replacement of gtk statusicon if available?14:46
klattimermeebey: yeah14:52
klattimerin basic terms14:52
klattimerit's actually an improvement on the concept of the notification area in many many ways14:52
meebeyok, and whats the connection to messaging menu? is it ok if my application support messaging menu + app indicator?14:53
klattimermeebey: that depends on how you classify the usage of either function14:53
klattimerif it's ONLY messaging related, then messaging menu14:53
klattimerif it's got some bits which are outside of the messaging menu scope, then pop it in the indicator applet14:54
klattimermeebey: there are specifications for this on the wiki, I'm really busy right now, but hope you get on well with using the API's14:54
klattimerpersonally, I really really like app-indicators, and am currently working on porting apps to use it14:54
meebeyI am reading the wiki right now, thanks14:54
jcastrohttps://wiki.ubuntu.com/DesktopExperienceTeam/ApplicationIndicators14:55
jcastroand14:55
jcastrohttps://wiki.ubuntu.com/CustomStatusMenuDesignGuidelines14:55
meebeyI will treat gtk statusicon as deprecated then and add addindicator support to smuxi14:56
jcastrooh awesome, you're the smuxi person?14:56
meebeythe issue with statusicon is that I cant enable it by default as per GNOME HIG ;)14:56
meebeyjcastro: yeah I am the crazy hacker14:56
jcastroI think for irc programs it's supposed to be a messaging menu? at least that's how it is with xchat14:56
meebeyjcastro: look: http://twitter.com/smuxi/status/2076374689514:57
jcastrooh dude, nice14:58
meebeyjcastro: while reading all the madness patches, I think I have better ayatana support than all the other apps out there ;)14:58
jcastroI've been meaning to try smuxi for a while14:58
jcastromeebey: oh, check this out14:58
diekiHey, does anybody know if RGBA is still planned for Maverick?14:58
meebeyjcastro: I have find some design issues that should be discussion on the mailing list14:59
meebeys/find/found/14:59
jcastrodo you do anything funky with your menus? or just normal gtk menus?14:59
jcastrohttps://wiki.ubuntu.com/DesktopExperienceTeam/ApplicationMenu14:59
jcastrodieki: I don't think it is, I could be wrong though14:59
meebeyjcastro: no special menu, just plain gtk menu, I would like to know of ApplicationMenu works with smuxi or not15:00
meebeyjcastro: its not packaged in debian yet I think15:00
jcastromeebey: I see it in ubuntu, so it must be in debian15:00
diekijcastro: Okay, thanks. Wonder why it got delayed again... (Not that it being delayed from Lucid was really suprising)15:00
jcastroI can test it with the menu now15:00
meebeyjcastro: if you want to give the ayatana support in smuxi a go, grab it from git15:01
jcastrodieki: last I saw there were still bugs?15:01
jcastromeebey: let me rule out that the menu works first15:01
diekijcastro: Well, in that case it's probably right in being delayed. I'm suprised RGBA wasn't on during the alphas though, it would have gotten far more testing.15:03
jcastrodieki: I think it's more of a case of finding time to fix the bugs than getting bugs filed15:03
jcastromeebey: ok I need to update my netbook and whatnot, I'll test the menu and then try it from git15:04
diekijcastro: Ah, okay. Thanks.15:04
meebeyjcastro: the main purpose of the status icon currently is to hide/show main window and quit, or should I skip app indicator on this case? I am open to use th app indicator for new innovative things but this is new land to me :) and I dont have (actually know) a single application supporting this for some inspiration15:05
meebeyjcastro: thanks for testing :)15:06
jcastroI am unsure of the design aspects15:07
jcastroposting on the ayatana list asking for recommendations might be a good idea15:07
jcastroor maybe mpt is around to give a recommendation15:07
meebeyok, will do15:07
klattimerjcastro: hows it going?15:08
jcastroklattimer: good, you?15:09
klattimergood good15:09
klattimercooking up a patch for keyboard indicator right now15:09
jcastroklattimer: I don't know if I mailed you but I am off this week, just checking in on occasion.15:09
klattimerit's looking OK15:09
jcastrothey keeping you busy?15:09
klattimeryeah, there's been a lot to look over to get myself up to speed on some things15:09
klattimerbut I'm hanging in there and getting through it15:09
* jcastro nods15:09
klattimerhoping I can get this patch done today15:10
klattimerso I can maybe get the indicator for bluetooth tested and working by friday15:10
mptmeebey, I suggest using the messaging menu instead of having a separate menu.15:12
mptmeebey, how to handle IRC (where there's a hum of messages, and many of the messages aren't for you) is an interesting problem I haven't thought much about yet.15:13
meebeympt: I solved that already, it only triggers on highlights15:13
mptfair enough15:13
meebeympt: that is, nick said on channel or pms15:13
meebeyeven says who said your nick on channels :)15:14
meebeympt: the messaging menu can't have a context menu per application, I should live with that I guess? users should click on icon to get the window and then use the window menu, right?15:15
mptyes15:16
meebeympt: ok thanks15:17
meebeympt: another topic I would like to get guidelines for is the close behavior of a window15:17
meebeywhat is the recommended way for that?15:17
mptmeebey, I'm not sure what you mean15:19
gambsbdrung: ping?15:19
meebeympt: when you click the closee button the main window should it quit the app, hide the window, etc15:19
bdrunggambs: pong15:20
meebeympt: thats a question that pops up when messaging menu and/or application menu gets used15:20
mptmeebey, if it's a single-window application, closing the window should exit the application. We try to have a clear distinction between (a) applications, which are based around windows and sometimes have a status menu, and (b) services, which are most often accessed from a status menu but sometimes have a window.15:21
meebeyit's a tabbed single window for all the chats, so close should exit I guess15:22
meebeympt: thanks for the info15:29
* klattimer thinks he *might* have fixed the keyboard indicator 15:36
klattimerwooohooo15:36
klattimerbuilding now :)15:36
nicksterlo folks... ux bod here lurking for a bit...would like to contribute to the ayatana/ubuntu UX development, so will lurk until more clued15:43
mpthi nickster, welcome15:53
klattimerok, well that didn't fix it :(16:03
klattimernow I have no indicator at all16:03
klattimer:/16:03
klattimertedg: is your branch label-in-lib for libappindicator complete?16:06
tedgklattimer, Yes, I'm working on merging it now.16:06
klattimerok cool, I'm testing and hitting a snag, just wanted to check that it does what's expected right now, so this is definitely a problem with my code16:06
klattimerit's kinda hard to test gnome-settings-daemon as it starts/restarts on login so I can't get output16:07
klattimerdo you have any ideas wrt to that?16:07
klattimertedg: I'm having trouble with your branch16:15
klattimerI have app_indicator_new (...) and then immediately after app_indicator_set_label(app_indicator, "KBD_LABEL", NULL);16:16
klattimerand the label still isn't set16:16
klattimermaybe I should wait to test my patch until you've merged?16:16
tedgklattimer, As in the property isn't set or that the indicator isn't showing the label?16:16
klattimerthe indicator isn't showing the label16:16
tedgklattimer, Yeah, that branch only fixes the lib, there's another one that fixes the indicator.   I split it in to two to make the review easier :)16:17
klattimerok, well let me know when you've merged so I can test it16:17
klattimerI'll still be around tonight later as it's approaching "home time" in the UK16:17
klattimerso drop me a shout when it's done, I'm really eager to see if this patch is actually working16:18
klattimerwant to get it off my plate if you know what I mean :)16:18
tedgklattimer, There, all in.16:18
tedgklattimer, Yeah, I understand.16:19
klattimeroh whats the lp: branch?16:19
tedgklattimer, I want to see it work in more than the demo app as well too :)16:19
tedgklattimer, lp:indicator-application16:19
klattimerhehe16:19
klattimerwicked16:19
klattimer:)16:19
klattimerto clarify, does setting a label unset the icon?16:19
tedgklattimer, In the examples directory there is a small program that sets a label, just if you want to check things.16:19
tedgklattimer, No, it shows the icon as well.16:20
klattimertedg: is there a way to unset the icon, or can you just app_indicator_new with a null icon?16:20
tedgklattimer, No, I think the requirement from design was to always have an icon.  Though we should probably double check that.16:21
klattimermpt would you like to clarify?16:21
tedgmpt, I thought when we discussed putting labels in app indicators, we said that we were going to require an icon as well.  So there'd always be one.  Is that what you remember?16:21
mpttedg, yep.16:22
klattimermpt: are there no concerns about the space being taken up by an icon+label?16:22
mptklattimer, a label should be used only when it's extremely useful (e.g. for showing the time remaining on battery)16:24
mptMost of the time it should be just an icon.16:24
klattimermpt: well, we can't do icons on keyboard indicator16:24
klattimerwe just have GBr, GBr2, USA etc... as a label text16:24
mptklattimer, I've already specified in great detail what that icon should look like. I recognize you might not be able to implement it as an icon in time, but please don't persecute me about it16:25
klattimerok16:26
klattimercool16:26
mptHow you implement it is up to you and tedg16:26
klattimertedg: where's the test16:32
klattimerafter updating I still see no label16:32
klattimeroh hang on I've found it16:33
klattimerdumbass me16:33
klattimer:/16:33
tedgklattimer, You may need to restart the applet to load the new library.16:34
klattimertedg: I logged out and back in16:34
tedgklattimer, That should work too :)16:34
klattimerI dunno, I'm gonna look at your sample client16:34
klattimerto see what I've done wrong16:34
klattimertedg: the only difference I see is that I hadn't set the guide16:40
klattimerbut there's a chance that the plugin is crashing perhaps, so that would account for the non-existent label :/16:41
tedgklattimer, Hmm, that shouldn't make a difference.  Do you set a menu?  Sometimes people forget that, but it's required.16:41
klattimertedg: a menu is definitely set16:41
tedgklattimer, Is there an icon but no label?16:41
klattimeras it worked with just an icon16:41
klattimerno icon or label16:41
tedgklattimer, Hmm, look in ~/.cache at the log and see if it has anything?16:41
tedgklattimer, Also, if you grab libindicator-tools you can run $ /usr/lib/libindicator/indicator-loader /usr/lib/indicators/3/libapplication.so  and it will put the app indicators in their own window.  But then give you output when they get added and removed.16:42
klattimertedg: awesome tip, except gnome-settings-daemon is loaded on login/logout so it makes it a bit harder16:43
klattimercan't seem to load it any other way16:43
klattimernothing appears to be in the logs, I've changed my code, removing the major changes and just setting the label to see what's going on there16:43
tedgklattimer, Hmm.  I've not tried it before, but could you run the bustle logger at login and what the dbus traffic?16:45
tedgklattimer, I think there's some sort of race in libappindicator where sometimes things at startup fallback into the notification area, but that doesn't sound like what you're seeing either.16:46
tedgklattimer, But, perhaps the dbus traffic would give a clue.16:46
klattimermaybe16:47
klattimertedg: your code must be fine as without my additions to the patch it sets the label16:50
klattimerI'll have to work out what my bugs are16:50
klattimer;/16:50

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