[10:42] <SiDi> hi people
[10:43] <SiDi> MacSlow: just wanted to tell you, i got xfconf support and theming support working on a local branch. Didnt have the internet lately so i couldnt push it anywhere, and im quite busy today too, but i just wanted to let you know before you work on the gconf keys for text size / color ;)
[10:44] <MacSlow> SiDi, how do you define "theming" in that scope again?
[10:44] <SiDi> you can set colors for bubble shadow / bg / text title / text body / text shadow
[10:44] <SiDi> i might add gconf keys for sliders, they look bad with bright backgrounds
[10:45] <SiDi> i also added gconf keys for text title / body size, they work with gconf & xfconf afaik
[10:45] <MacSlow> SiDi, priority order for working on features/fixes for notify-osd has been shifted around during this week so I'll not be addressing gconf/xconf the coming week
[10:45] <SiDi> okies
[10:45] <MacSlow> SiDi, don't over commit there :)
[10:45] <MacSlow> SiDi, I only want to introduce gconf-keys for...
[10:45] <SiDi> i just read "gconf" in a notify-osd bug report so i wanted to make sure you dont do it since its done :P
[10:46] <MacSlow> bubble-bg-color, bubble-bg-opacity, bubble-shadow, title-font, body-font, text-shadow-color
[10:46] <SiDi> MacSlow: anyway it doesnt cost anything to have these keys :) i'll try to write a little app to turn notify-osd to black on white for people who have problems to read white on black
[10:46] <SiDi> oh, for opacity too ?
[10:47] <SiDi> (i used #rrggbbaa as strings for color, so actually it can handle bubble opacity)
[10:47] <MacSlow> SiDi, I looked at your first patch and decided I'll need/want to split out gconf and xconf related calls do separte files each
[10:47] <MacSlow> SiDi, hm... I don't think GdkColor takes rrggbbaa only rrggbb
[10:48] <SiDi> oh
[10:48] <MacSlow> SiDi, there's a gdk-function that can parse "#rrggbb" strings
[10:48] <SiDi> ah great
[10:48] <SiDi> i didnt have the internet so i wrote my own function :D
[10:48] <MacSlow> opacity I'll plan to cover using a float-value (0.0f - 1.0f)
[10:49] <SiDi> okey
[10:49] <MacSlow> SiDi, I'll replace that with the corresponding Gdk-call
[10:50] <SiDi> MacSlow: okey. Could you just write me a little guide with what i should do on the notify-osd + xfce bug report ? i'll write this properly this week-end
[10:50] <MacSlow> SiDi, FYI gboolean gdk_color_parse (const gchar* spec, GdkColor* color);
[10:51] <MacSlow> SiDi, you mean regarding splitting out the code or the color-parsing?
[10:51] <SiDi> well, splitting out the code
[10:52] <SiDi> its true its not easy to read at all right now, but it took me more time to make autotools work than to code it ;)
[10:52] <MacSlow> SiDi, oh... well I'm not 100% sure what/how I'll approach that
[10:52] <SiDi> me neither
[10:53] <SiDi> We could use defaults.c that would include defaults-gconf and defaults-xfconf ?
[10:53] <MacSlow> SiDi, just adapt your patch to use gdk_color_parse() for "#rrggbb" and use a float xconf-key for any opacity
[10:53] <SiDi> okey, i'll do that for theming
[10:53] <SiDi> i'll setup a branch as soon as i have the internet :p
[10:54] <MacSlow> SiDi, yeah something like that
[10:55] <MacSlow> SiDi, please avoid using the term "theming" it'll give the wrong impression on users
[10:56] <MacSlow> SiDi, it's just providing some "hook" to allow distro/desktop-environment integrators to tweak notify-osd at runtime a bit
[10:56] <SiDi> MacSlow: i wont pronounce it on bug reports :]
[10:56] <MacSlow> good :)
[10:56] <SiDi> MacSlow: i do want to have an utility to change the colors though, but i'll probably name it notify-osd-utils. Some users can really not read white on black
[10:57] <MacSlow> SiDi, don't be surprised if I'll shuffle around your patch (formatting & Co)
[10:57] <SiDi> no problem
[10:57] <SiDi> i had a hard time writing it, i have a different writing style, and im back from 3 weeks of python :/
[10:57] <MacSlow> SiDi, since I didn't yet find the time to write a hacking/coding-styleguide yet just do as you please (trying to follow notify-osd lead as good as you can :)
[10:58] <MacSlow> SiDi, well... cleaning up such stuff is certainly more my responsiblity than yours so don't stress too much
[10:58] <SiDi> okey, no problem
[10:58] <SiDi> i noticed there were spaces only in half of the code for indentation anyway ;D
[10:59] <MacSlow> SiDi, I'll also try to incorporate your battery/power info tweak suggestions into the g.p.m patch ... I'm trying to work on that today 
[10:59] <SiDi> okey
[11:00] <SiDi> i wont have time to give feedback on this before next week, though :P
[11:00] <MacSlow> SiDi, I think most of this is covered in your bug-report... there's quite a lot of comments
[11:00] <SiDi> Yup
[11:00] <MacSlow> I'll see if there are any gaps of info
[11:00] <mac_v> tedg: hi...the always present icon should be a seperate bug ?
[11:00] <SiDi> And ironically xubuntu will likely give g-p-m up so i have to patch xf-p-m :D
[11:00] <mac_v> indicator applet
[11:01] <SiDi> (hello mac_v howdy ?)
[11:01] <MacSlow> SiDi, well... that's you problem not mine ;)
[11:01]  * mac_v waves at SiDi 
[11:02] <SiDi> btw MacSlow i noticed the errors and strings you get from gconf are never freed, but i dont know gconf at all so i dont know if they should be or not
[11:02] <MacSlow> SiDi, no that should be fixed in trunk by now
[11:03] <SiDi> ok
[11:04] <MacSlow> SiDi, ah... no wait... that'll come in with the pending gconf-key work
[11:04] <MacSlow> SiDi, I was mixing it up with another class
[11:04] <SiDi> ok
[11:05] <SiDi> i hope my patches will still merge without conflict, MacSlow :D
[11:05] <MacSlow> SiDi, we'll see
[11:06] <tedg> mac_v: It's no a bug, it's a feature :)
[11:06] <tedg> mac_v: But the "No Indicators" was definitely a bug and needed to be fixed.  Upload issues.
[11:06] <mac_v> tedg: why! its kinda waste of space , when i dont use the apps
[11:07] <tedg> mac_v: We're going to support having the applications in there even when they're not being used.
[11:08] <tedg> mac_v: One of the things we found in usability testing is that folks would go there for their mail even when it wasn't running.  The idea that "this is the place that evolution lives" kind of thing.
[11:08] <tedg> mac_v: I don't know if it's going to suck or be great, but I think it's worth trying for a release :)
[11:09] <mac_v> tedg: "" One of the things we found in usability testing is that folks would go there for their mail even when it wasn't running."" this was with the icon displayed or without the icon?
[11:09] <tedg> mac_v: When the icon was displayed, but with another application running.  So it was there, but didn't have mail for instance.
[11:10] <tedg> mac_v: Also, I believe there has been a few bugs about the icon itself being transient.
[11:11] <mac_v> tedg: that is becuase the indicator applet is using a wrong icon! that is why they are checking for mail!
[11:11] <mac_v> mpt: ^
[11:11] <tedg> One of the biggest that people can't remove it without starting an application that they don't use :)
[11:11] <tedg> mac_v: Perhaps, but no one has proposed a better icon :)
[11:11] <mac_v> tedg: i saw that bug too , but it can be removed from startup applications
[11:12] <mac_v> tedg: why not use the "i"
[11:12] <mac_v> as icon?
[11:13] <tedg> mac_v: I don't know, that doesn't really say anything to me other than "generic".
[11:14] <mac_v> this icon problem was mentioned in the mailing list , let me get the link
[11:24] <mac_v> tedg: instead of using the general envelope icon for all > im icon can be used if pidgin is working > indicator applet shows the icon of pidgin  , and email icon is shown only when email client is open , and if both are present retain the mail icon
[11:25] <mac_v> tedg: i'll file a wishlist bug for this ,always present icon , maybe it can be considered for later Karmic+1
[11:33] <tedg> mac_v: Well, you might wait for the feature to be completed and decide if you like it first :)
[11:33] <mac_v> tedg: oops! i already did actually ! :( sorry
[11:35] <mac_v> tedg: the biggest problem for me is none of my apps[thunderbird/xchat] use indicator applet , and i rarely use evolution for calender ! so for me its a waste of space though i like to have it shown when evolution is used :(
[11:37] <tedg> mac_v: Heh, the problem there is that xchat and thunderbird don't use the messaging menu ;)
[11:38] <mac_v> tedg: yeah , thats why ;p ... anyways Bug #410220 , i filed it before you could reply...! 
[11:38] <ubot4> Launchpad bug 410220 in indicator-applet "Indicator applet Always shows icon" [Undecided,New] https://launchpad.net/bugs/410220
[11:43] <tedg> mac_v: No problem, I'll comment.
[11:43] <mac_v> :)
[16:00] <DBO> ping njpatel 
[16:33] <MacSlow> DBO, he's currently traveling (going back to London)
[16:36] <DBO> thanks MacSlow =)