=== kenvandine_ is now known as kenvandine
=== yofel_ is now known as yofel
[06:35] <andyrock> good morning
[06:54] <RAOF> smspillaz: I've just had gtk-window-decorator go totally insane and eat all my memory via mesa - bug #827818.  Anything you want from me?
[06:54] <ubot5> Launchpad bug 827818 in unity (Ubuntu) "Runaway memory usage in gtk-window-decorator" [Undecided,New] https://launchpad.net/bugs/827818
[07:02] <oSoMoN> good morning
=== Amaranthus is now known as Amaranth
[08:02] <mardy> Kaleo: hi, practical question: https://bugs.launchpad.net/unity-2d/+bug/757818 has been fixed in Bamf, should I mark it as fixed in unity-2d too? What is the correct procedure here?
[08:02] <ubot5> Ubuntu bug 757818 in unity-2d "bamfdaemon crashed with SIGABRT in dbus_g_connection_register_g_object()" [Critical,New]
[08:15] <Kaleo> mardy: you can mark it as committed in u2d as well, and then released when it is in bamf
[08:15] <mardy> Kaleo: ok, thanks
[08:20] <mardy> nerochiaro: hi, about bug 827673: in r672 the panel doesn't set a size policy on its widgets -- maybe that code just need to be removed from r673?
[08:20] <ubot5> Launchpad bug 827673 in unity-2d "[panel] indicators are shifted offscreen to the right" [Undecided,In progress] https://launchpad.net/bugs/827673
[08:22] <nerochiaro> mardy: i think that the problem is that the appname widget was setting its own policy before, and now i'm setting it globally, but i noticed i use a different policy. i'm gonna try and use the old one
[08:25] <mardy> nerochiaro: you are right, it was set to "Ignored". But then, unless we introduce a complex set of flags for the dconf schema, or we let the applets set their size policy (I vote for the latter, for the time being)
=== API is now known as Guest35515
[08:26] <mardy> s/unless/either/
[08:27] <nerochiaro> mardy: what if if the expanding flag is set i just set the policy to Ignored instead of Expanding ? I mean, it would be exactly what was happening before, just moved to the panel manager
[08:28] <nerochiaro> mardy: the logic is not wrong, the only mistake was to use Expanding instead of Ignored
[08:28] <nerochiaro> mardy: i vote for just fixing that
[08:28] <mardy> nerochiaro: wouldn't make much sense...
[08:28] <nerochiaro> mardy: why not ? it was making sense before
[08:28] <mardy> nerochiaro: a better option IMHO is if the "!" flag means: "the size policy is set by the applet"
[08:28] <smspillaz> RAOF: gtk-window-decorator doesn't use mesa
[08:28] <smspillaz> RAOF: there is a memory leak however, I've got a fix for that
[08:29] <mardy> nerochiaro: actually no...
[08:29] <nerochiaro> mardy: the ! flag right now is used by something completely different
[08:29] <nerochiaro> mardy: i.e. leftmost only
[08:29] <mardy> nerochiaro: sorry, got confused with the "expanding" key
[08:30] <nerochiaro> mardy: but we can add a new flag that say "don't force a size policy on this applet" if you think it's better
[08:30] <mardy> nerochiaro: I just vote for removing that dconf key, it's useless and confusing now
[08:30] <mardy> nerochiaro: yes, "don't force a size policy on this applet" is better, but then why can you specify it for just one applet? It doesn't make much sense...
[08:32] <nerochiaro> mardy: right. so what about a new flag like ! that you can put on as many applets as you like and for all those applets the policy won't be forced ?
[08:33] <mardy> nerochiaro: that might be, but OTOH I don't see why we need it now
[08:33] <mardy> nerochiaro: now we can simply let all applets specify their size
[08:33] <mardy> nerochiaro: if we want to change it later, we can add the size information in the dconf file, and the panel will override the applet's size policy
[08:36] <nerochiaro> mardy: ok, i'll do that then. the only reason why i wanted the size policy to be in dconf is for flexibility and not having to hack the unity-2d code when we need something like, for example, a different size policy on the appname applet (for whatever reason).
[08:37] <nerochiaro> without having to hack the appname applet
[08:37] <nerochiaro> but i'm ok not having that flexibility for now, it's not something OEM needs right now
[08:38] <mardy> nerochiaro: yep. But then we'll probably need something more complex than just a choice between Ignore and Fixed (we might want Expanding, MinimumExpanding, etc)
[08:38] <mardy> nerochiaro: so it's better to wait till we have some requirement, and then design a schema that could accomodate all
[08:39] <nerochiaro> mardy: agreed. let's just remove it for now and let each applet set its own policy. in this case are you ok for having the base applet class set the default policy to MinimumExpanding and then other applet override it (for example appname will have Ignored)
[08:39] <nerochiaro> ?
[08:40] <nerochiaro> mardy: and i mean Fixed when i said MinimumExpanding :)
[08:41] <mardy> nerochiaro: sure, seems reasonable
[08:41] <mardy> nerochiaro: see, that that class is useful ;-)
[08:41] <nerochiaro> mardy: it was like that before anyway ;) ok. i'm going to do that and then push an MR for you, if you're ok reviewing it
[08:42] <mardy> nerochiaro: sure, great :-)
[09:22] <mardy> Kaleo: you can now use my address @canonical.com, it will help me to keep the e-mails in the right folder :-)
[09:23] <nerochiaro> mardy: https://code.launchpad.net/~uriboni/unity-2d/panel-size-policy-up-to-applets/+merge/71829
[09:29] <mardy> nerochiaro: cool! :-) I approved and added one unrelated note; if you are still modifiying that file, you might want to add that check in, too
[09:30] <nerochiaro> mardy: i'll have to submit another MR for that, but it can be done if you want to
[09:30] <nerochiaro> mardy: i still have the file open ;)
[09:31] <mardy> nerochiaro: if you don't mind, I think it could save us some trouble in the future, when 3rd parties will write their plugins :-)
[09:31] <nerochiaro> mardy: however it's a bit useless i think, because if you call addWidget(0) Qt will not crash or do anything stupid, just not add any widget
[09:32] <mardy> nerochiaro: ah, I didn't know
[09:32] <mardy> nerochiaro: then let's forget about it
[09:32] <nerochiaro> mardy: actually I have not tried, but seems reasonable they do check their pointers
[09:32] <nerochiaro> ok
[09:32] <mardy> nerochiaro: let me have a quick look, I've the qt sources somewhere here...
[09:33] <nerochiaro> mardy: ok, i'm curious too now
[09:33] <nerochiaro> mardy: but i have faith in your former colleagues
[09:35] <mardy> nerochiaro: ah, actually that addWidget is a method of Unity2dPanel
[09:35] <mardy> nerochiaro: ok, it goes back to Qt anyways...
[09:37] <mardy> nerochiaro: oh, nope :-) *w is the widget being added, and they do w->parentWidget() before any checks :-)
[09:37] <mardy> nerochiaro: so better check for it in unity-2d
[09:39] <nerochiaro> mardy: ok, i'll add the check and the trolls lose a few points
[09:40] <mardy> :-)
[09:45] <nerochiaro> mardy: https://code.launchpad.net/~uriboni/unity-2d/panel-check-valid-applet/+merge/71834
[09:48] <ephan> I have a question
[09:48] <ephan> How can an applet be invalid?
[09:49] <nerochiaro> ephan: if the plugin just return NULL for whatever reason
[09:49] <ephan> Hm
[09:49] <ephan> Thanks, I sort of understand
[09:51] <mardy> ephan: it's not really something that we expect to happen, but if we don't do that check, the panel will crash and then enter a crash loop (because it's auto restarted)
[09:51] <andyrock> jaytaoko, hi
[10:05] <ephan> Anybody on Unity 4.8 can test something quick for me?
[10:10] <davidcalle> ephan, I can help
[10:11] <ephan> davidcalle, can you open the dash, enter some text, then do Ctrl+A and Ctrl+X (so select all, and then cut)
[10:11] <ephan> On Natty (Unity 3.8) I get a weird symbol
=== seb128_ is now known as seb128
[10:12] <davidcalle> Ctrl+A works, but Ctrl+X (or +C  or +V) inserts a symbol
[10:12] <ephan> Thanks davidcalle, I guess that's a bitesize bug
[10:12] <davidcalle> I guess it is.
[10:12] <ephan> reporting
[10:14] <ephan> I'll also check the code and see if I can fix it
[10:14] <davidcalle> ephan, thanks :)
=== rodrigo__ is now known as rodrigo_
[10:35] <andyrock> ephan, about ctrl+something
[10:36] <andyrock> is an already reported bug
[10:36] <ephan> yes?
[10:36] <ephan> Hm, when I reported it didn't warn me of anything, can you give me ID so I can mark it as duplicate?
[10:37] <andyrock> ephan, Omer already reported it as a duplicate
[10:37] <ephan> ah thanks
[11:00] <RAOF> DBO: I hope you're fixing bug #825029 in the next release, because it's driving me *insane*
[11:00] <ubot5> Launchpad bug 825029 in unity (Ubuntu) "When alt-tabbing to an application with multiple windows the focused window is not raised" [Undecided,New] https://launchpad.net/bugs/825029
[11:08] <om26er> smspillaz, Hi! around? Would it be fine if I close unitydialog related bugs for now?
=== m_conley_away is now known as m_conley
[14:57] <cdbs> njpatel: https://code.launchpad.net/~bilalakhtar/unity/software-center-integration-for-o/+merge/71905 Some parts are implemented, managed to workaround the tooltip trouble by adding a subroutine
[14:58] <cdbs> njpatel: Thanks a lot for helping! No need to get down and merge that branch straight away, take your time, it can land up till UI freeze
[14:58] <cdbs> which is 8 days away IIRC
[15:05] <njpatel> cdbs, thanks, DBO some coolness
[15:05] <njpatel> [up arrow]
[15:07] <DBO> ?
[15:07] <DBO> njpatel, ?
[15:09] <njpatel> DBO, cdbss branch linked above
[15:09] <DBO> OH
[15:10] <DBO> excellent
[16:45] <jjohansen> is there a design document to why unity alt-tab behaves the way it does?
[18:08] <smoser> i apologize for my ignorance
[18:08] <smoser> current(ish) oneiric in Unity. when i click on the date in the top left , it brings up a calendar
[18:09] <smoser> previously, in that calendar i could click the arrows to go see other months
[18:09] <smoser> that seems broken now
[18:09] <smoser> (and for quite some time for me inoneiric)
[18:09] <smoser> i'd open a bug, but i dont really even know what package that thing is , or what it is
[18:17] <jjohansen> smoser: yeah that one really bothers me too, https://bugs.launchpad.net/ubuntu/oneiric/+source/indicator-datetime/+bug/807509
[18:17] <ubot5> Ubuntu bug 807509 in indicator-datetime (Ubuntu Oneiric) "Cannot click on Calendar to select another day, month or year" [High,Triaged]
[18:19] <smoser> thanks for the link jjohansen
[18:48] <sbte> tedg, hi, is there anything else I have to do to get my indicator-messages patch committed?
[18:49] <tedg> sbte, Nope, actually going through code reviews right now :-)
[18:49] <sbte> tedg, ok, great
[18:55] <nhaines> tedg: when you're done with important things (like code reviews) I hope I can ping you to ping someone about Ubuntu business card artwork.  :)
[18:56] <tedg> nhaines, Hmm, did that never come out?  :-/
[18:56] <tedg> nhaines, I thought that doctormo made some, no?
[18:56] <nhaines> tedg: no, I never heard back from ivanka and then I waited for the release to settle and then she fled.  :)
[18:57] <tedg> nhaines, She's in SF right now, if you can catch her ;-)
[18:57] <nhaines> tedg: doctormo's design... let's just say isn't very consistent with the branding guidelines.
[18:57] <nhaines> Haha, the airport's just 30 minutes away. ;)
[18:57] <tedg> nhaines, Let me ask around and see who might have it.
[18:57] <nhaines> Less, actually.
[18:58] <nhaines> tedg: you're golden.  Thanks so much!  :)
[18:58] <tedg> nhaines, She's doing the whole thing on motorcycle... so airports aren't much use :-)
[18:58] <nhaines> tedg: they might be for me though!  I'd rather play 3DS for 45 minutes than drive for 7, hehe.
=== m_conley is now known as m_conley_away
=== yofel_ is now known as yofel