#instantbird log on 12 12 2010

All times are UTC.

00:08:30 --> GeekShadow has joined #instantbird
00:13:19 --> ircjimhall has joined #instantbird
00:16:45 --> micahg has joined #instantbird
00:35:39 <Mic> nn
00:44:15 <-- Ornthalas has quit (Quit: KTHXBYE)
00:49:05 <-- Mic has quit (Quit: Instantbird 0.3a1pre)
01:15:44 <-- Mook has quit (Connection reset by peer)
01:17:31 <-- GeekShadow has quit (Client exited)
01:32:33 --> Mook has joined #instantbird
01:43:39 <-- hicham has quit (Client exited)
02:05:50 <instantbot> New Instantbird (UI) bug 622 filed by mook.moz+bugs.instantbird@gmail.com.
02:05:51 <instantbot> mook.moz+bugs.instantbird@gmail.com added attachment 416 to bug 622.
02:05:53 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, ---, nobody, UNCO, Move menu.xul.inc into an overlay
02:08:29 <-- clokep has quit (Quit: Instantbird 0.3a1pre)
02:08:36 --> clokep has joined #instantbird
02:08:37 * ChanServ sets mode +h clokep 
03:29:16 --> mepine has joined #instantbird
04:03:08 <instantbot> New purplexpcom - General bug 623 filed by clokep@gmail.com.
04:03:11 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=623 nor, --, ---, nobody, NEW, Auto-Join option field is hard coded for certain protocols
04:05:05 * zz_auscompgeek is now known as auscompgeek
04:47:20 <-- zachlr has quit (Ping timeout)
04:55:42 <-- clokep has quit (Quit: Instantbird 0.3a1pre)
04:58:58 --> clokep has joined #instantbird
04:58:58 * ChanServ sets mode +h clokep 
04:59:44 <-- clokep has quit (Quit: Instantbird 0.3a1pre)
04:59:48 --> clokep has joined #instantbird
04:59:48 * ChanServ sets mode +h clokep 
05:00:42 * clokep needs to stop getting confused about whether he's closing his real Ib or his development version.
05:30:22 <instantbot> New Instantbird (UI) bug 624 filed by mook.moz+bugs.instantbird@gmail.com.
05:30:23 <instantbot> mook.moz+bugs.instantbird@gmail.com added attachment 417 to bug 624.
05:30:25 <instantbot> mook.moz+bugs.instantbird@gmail.com requested review from florian@instantbird .org for attachment 417 on bug 624.
05:30:26 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=624 enh, --, ---, nobody, UNCO, client.py should not require repos if not required
05:30:40 <Mook> .. huh, that summary is stupid
05:38:06 * clokep finds it very odd that the UI sends the auto join requests instead of the protocols.
05:41:04 <Mook> hmm. so. if I tried to just generally sort out the menu.xul.inc file to be saner, does that seem likely to be accepted?
05:41:26 <Mook> (move the preprocessed file into an overlay; move various platform-specific bits into overlays on top of that)
05:41:55 <clokep> I think flo would accept that, yes. :)
05:42:24 <clokep> Will definitely bit rot one of my patches. ;) But I need to rethink that one anyway.
05:48:06 <clokep> nn
05:48:19 <-- clokep has quit (Quit: Instantbird 0.3a1pre)
06:01:57 <Mook> hmm. what platforms does this target? XP_WIN, XP_MACOSX, and XP_UNIX? anything else?
06:03:23 <Mook> (trying to sort out the quit menu item, http://pastebin.instantbird.com/505 )
06:30:12 * auscompgeek is now known as zz_auscompgeek
07:28:26 --> tymerkaev has joined #instantbird
07:56:54 * tymerkaev is now known as tymerkaev_away
07:56:59 * tymerkaev_away is now known as tymerkaev
08:29:16 * tymerkaev is now known as tymerkaev_away
08:31:16 * tymerkaev_away is now known as tymerkaev
08:43:29 <-- tymerkaev has quit (Ping timeout)
09:11:04 <-- ircjimhall has quit (Ping timeout)
09:43:51 <-- Mook has quit (Quit: ChatZilla 0.9.86-rdmsoft [XULRunner 1.9.1.15/2010102600])
10:16:49 --> Ornthalas has joined #instantbird
10:48:59 <-- Jan has quit (Ping timeout)
10:50:21 --> Jan has joined #instantbird
10:50:41 * Jan is now known as IRCMonkey59713
10:56:18 --> GeekShadow has joined #instantbird
11:10:16 * IRCMonkey59713 is now known as Jan
11:10:51 --> Mic has joined #instantbird
11:10:51 * ChanServ sets mode +h Mic 
11:45:26 --> hicham has joined #instantbird
11:46:46 --> tymerkaev has joined #instantbird
11:47:38 <-- tymerkaev has quit (Quit: I must go. Good bye!)
11:47:58 --> tymerkaev has joined #instantbird
11:52:15 <-- mepine has quit (Ping timeout)
12:42:03 * tymerkaev is now known as tymerkaev_away
12:45:38 * tymerkaev_away is now known as tymerkaev
12:50:51 <-- micahg has quit (Ping timeout)
12:52:06 --> micahg has joined #instantbird
14:17:00 --> sadrick has joined #instantbird
14:17:12 <-- sadrick has left #instantbird ()
14:50:00 <-- Ornthalas has quit (Quit: KTHXBYE)
15:14:50 --> zachlr has joined #instantbird
15:35:27 --> clokep has joined #instantbird
15:35:27 * ChanServ sets mode +h clokep 
15:37:02 <instantbot> florian@instantbird.org added attachment 418 to bug 622.
15:37:06 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, ---, nobody, NEW, Move menu.xul.inc into an overlay
15:42:08 topic changed by Mic to "Ask questions about Instantbird here. Official website: http://www.instantbird.com. Latest release: 0.2. Read http://blog.instantbird.org/. Nightlies: http://nightly.instantbird.im/ (testing purpose only), IRC logs: http://log.bezut.info/. | ICQ troubles-> http://tinyurl.com/ICQError"
15:42:28 <Mic> argh, stupid topic bar bug
15:42:48 <clokep> :)
15:43:14 <-- Mic has quit (Quit: Instantbird 0.3a1pre)
15:45:42 --> Mic has joined #instantbird
15:45:43 * ChanServ sets mode +h Mic 
15:46:12 <Mic> This bug was a bigger pain than Nickserv ever was ;)
15:46:58 <clokep> Mic: Next time Mook is around we should ask him (re but 583) whether those will do what we need them to / if documentation exists.
15:47:16 <Mic> instantbot: bug 583
15:47:20 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=583 nor, --, ---, nobody, NEW, Need binary component to handle "native windows"
15:47:50 <clokep> flo: I made some magic happen last night...prefs work in JavaScript: https://hg.instantbird.org/experiments/file/IRC-JavaScript/modules/jsProtoHelper.jsm#l583
15:49:35 <flo> :)
15:50:30 <clokep> The list one doesn't /quite/ work yet,  but it mostly works. And I think it was a good way to do it.
15:51:17 <clokep> Also, flo....what exactly is a "UsernameSplit"?
15:51:45 <clokep> (Also I made a proxy object that doesn't do anything since the options menu was blowing up on me without one :P)
15:51:52 <flo> it's a way to split the username field in several text fields in the account wizard
15:52:32 <clokep> OK...I looked at the code from the Facebook override and I didn't really understand it.
15:52:34 <flo> look at XMPP or IRC for an example
15:52:48 <clokep> Hmm...OK. Thanks.
18:22:40 --> Mook has joined #instantbird
18:47:24 --> skeledrew-android has joined #instantbird
18:53:59 <-- Mic has quit (Quit: Instantbird 0.3a1pre)
18:54:06 <-- skeledrew has quit (Ping timeout)
18:59:46 --> skeledrew has joined #instantbird
19:10:18 <instantbot> mook.moz+bugs.instantbird@gmail.com added attachment 419 to bug 622.
19:10:19 <instantbot> mook.moz+bugs.instantbird@gmail.com requested review from florian@instantbird .org for attachment 419 on bug 622.
19:10:21 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, ---, nobody, NEW, Move menu.xul.inc into an overlay
19:24:25 <flo> Mook: hello :)
19:24:34 <Mook> hi!
19:24:40 <flo> have you seen the last point in https://bugzilla.instantbird.org/show_bug.cgi?id=622#c1 ?
19:24:42 * Mook is waiting for r- ;)
19:24:44 <instantbot> Bug 622 enh, --, ---, nobody, NEW, Move menu.xul.inc into an overlay
19:25:02 <Mook> oh, damn, forgot that bit
19:26:30 <flo> are the "<!-- vim: set sw=2 -->" really needed?
19:26:57 <Mook> no, not at all, it just helps my editor
19:27:55 <Mook> feel free to nitpick as much as you can, I need to figure out the conventions here :)
19:28:17 <flo> "preferencesCmdUnix" in menus-win.xul. Is this expected?
19:29:19 <flo> Not sure at first sight if this is a silly name we used to have, or a copy/paste mistake
19:29:42 <Mook> oops, no, it's not
19:30:00 <Mook> http://lxr.instantbird.org/instantbird/source/instantbird/content/menus.xul.inc?mark=117-118#103
19:30:47 <flo> ok :)
19:30:57 <Mook> (and the whole reason that bit was in the file was to change it to the windows variants... :p )
19:31:44 <Mook> oh, I guess it doesn't need a -win variant
19:32:45 --> Mic has joined #instantbird
19:32:45 * ChanServ sets mode +h Mic 
19:33:13 <Mic> The overlay id also contains unix for the Windows variant btw
19:33:34 <flo> is there a benefit (other than reducing the number of confusing ifdefs) to adding 3 separate overlays of OS-specific variations?
19:33:47 <Mook> no, just that
19:34:00 * Mook is also perfectly happy to drop that bit and just do the first part
19:34:38 <Mic> And "removelement" looks like a typo
19:34:45 <Mic> (same file)
19:35:17 <flo> Mook: I was asking mostly because this seemed like more work than absolutely needed for your use case and was wondering about the motivation ;)
19:35:27 <Mook> flo: I hate ugly code ;)
19:35:45 <Mook> Mic: good catch, thanks. at least it doesn't seem to have the same mistake in other files ;)
19:36:25 <Mic> Thanks for teaching me about removeelement this way, I didn't know it before ;)
19:36:32 <flo> the diff is messy (because of all the previous crap moving around), I guess it will be easier to apply it and review the resulting state ;)
19:36:51 <Mook> :)
19:37:29 <Mook> additional changes so far: http://sprunge.us/JiXU?diff
19:37:35 * flo didn't know removeelement either
19:38:10 <Mook> sadly, there's no way to remove attributes
19:39:30 <flo> I guess removeelement would complicate unapplying overlays if overlays are ever supported in restartless add-ons.
19:40:11 <Mook> haha, yes
19:43:35 <-- skeledrew-android has quit (Connection reset by peer)
19:49:44 <-- hicham has quit (Client exited)
19:50:09 --> hicham has joined #instantbird
19:53:25 --> Ornthalas has joined #instantbird
19:53:42 <-- hicham has quit (Client exited)
19:54:06 --> hicham has joined #instantbird
19:57:49 <-- tymerkaev has quit (Ping timeout)
20:00:24 <clokep> Bah I just found a bug in one of my scripts for class that just sped my code up by an order of magnitude.
20:00:32 <clokep> I had put a ) in the wrong place. :(
20:11:07 <-- skeledrew has quit (Ping timeout)
20:18:09 <-- micahg has quit (Ping timeout)
20:18:14 <clokep> flo: At one point should my jsProtoHelper stuff be merged? (Wondering if it'll affect your Twitter stuff or not?)
20:20:45 <-- GeekShadow has quit (Quit: The cake is a lie !)
20:26:24 --> skeledrew has joined #instantbird
20:30:14 <flo> clokep: yes, we should merge it.
20:30:29 <flo> I would like to figure out that inheritance issue before though.
20:30:59 <clokep> Right. I've also added a couple of nice things since the review (namely preferences).
20:31:26 --> sonny has joined #instantbird
20:32:14 <flo> clokep: ok. I wasn't really sure we wanted to keep that API, but if it "just works" with your code, I may procrastinate more redesigning the API ;)
20:32:54 <clokep> The list one still doesn't work (the first one isn't chosen automatically?) But I haven't had time to look at it more.
20:33:09 <clokep> I like the API, I think it's rather simple. :) But I'm sure it could use a  bit of redesign.
20:34:47 <-- Mic has quit (Quit: Instantbird 0.3a1pre)
20:38:30 <-- sonny has quit (Quit: ChatZilla 0.9.86-rdmsoft [XULRunner 1.9.2.13/20101211011801])
20:39:57 <-- skeledrew has quit (Ping timeout)
20:49:33 --> skeledrew has joined #instantbird
21:21:53 <-- chrisccoulson has quit (Client exited)
21:23:30 <-- hicham has quit (Client exited)
21:23:50 --> hicham has joined #instantbird
21:24:25 --> chrisccoulson has joined #instantbird
21:47:01 --> micahg has joined #instantbird
21:50:22 --> Mook_ib has joined #instantbird
22:00:34 * flo is trying Mook's patch
22:05:37 <Mook_ib> oh, then I should probably actually post that follow up I pastebinned
22:07:42 <flo> Mook_ib: yes. But don't worry, I applied it too before trying and looking at the code ;)
22:07:54 <Mook_ib> \o/
22:08:06 <flo> I wonder if we should rename blistMenubar to mainmenubar and how many add-ons that would break
22:08:33 <Mook_ib> yeah, that was sort of why I just kept everything the same...
22:08:48 <flo> I guessed that :).
22:08:54 <Mook_ib> on the bright side, it's 0.3 so it's not like it's _that_ big a deal right? :)
22:09:19 <Mook_ib> hmm, message styles didn't refresh in existing windows when I changed the pref. I wonder if I'm missing an observer topic...
22:09:20 <flo> we should increment the version number faster ;)
22:09:23 <clokep> flo: A better question is "how many add-ons that would break that aren't owned by someone sitting in this room right now + Mic"
22:10:02 <clokep> Bah, you want to be look Google Chrome ?! Aren't they on version like 42 or something?
22:10:20 <Mook_ib> nothing wrong with 0.10, 0.11, 0.12... :p
22:11:07 <flo> should we provide an add-on to add a preference for the numbering scheme? :)
22:12:32 <flo> clokep: the scary add-ons are the ones that aren't owned at all. ;)
22:13:18 <clokep> flo: I got the version for Instantbird bumped on Minimize-to-Tray, next time he releases we should upload a new version -- not sure who owns that?
22:13:27 * clokep realizes by saying that he probably does now. :-d
22:13:32 <flo> sure!
22:14:01 <flo> What's the list of authors on AIO?
22:14:04 <clokep> So...we can striek that off the list of "why people won't move to 0.3 ;)"
22:14:20 <clokep> Just Nils Maier.
22:14:27 <flo> but we recently added a ton of layout/painting issues :(
22:16:11 <clokep> Is that findbar one on Mac also, or just a Windows one?
22:16:17 * clokep goes to actually file that.
22:16:30 <flo> Mook_ib: was the stringbundle tag causing any problem, or do you just don't like it?
22:16:52 <flo> clokep: the findbar looks OK to me
22:16:52 <Mook> flo: I was just removing things that need to be overlaid all over the place
22:17:07 <Mook> (having ids makes things not work unless there's a target to go into...)
22:17:43 <flo> ah, right
22:19:11 * clokep wishes he could upload screenshots from a buffer
22:19:28 <flo> from a buffer?
22:20:02 <clokep> From my clipboard.
22:20:25 <flo> ah :)
22:21:05 <-- micahg has quit (Ping timeout)
22:21:46 <instantbot> New Instantbird (UI) bug 625 filed by clokep@gmail.com.
22:21:47 <instantbot> clokep@gmail.com added attachment 420 to bug 625.
22:21:49 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=625 nor, --, ---, nobody, NEW, Findbar broken on Windows in Conversation window
22:24:52 --> micahg has joined #instantbird
22:28:50 <Mook_ib> hmm, maybe whatever clokep is using is close enough and I don't really need a single-window UI...
22:29:03 <Mook_ib> (also: is tab completion of nicks supposed to work?)
22:29:09 <flo> no
22:29:16 <clokep> Mook_ib https://addons.instantbird.org/en-US/instantbird/addon/244/ I like using my own add-ons. ;)
22:29:37 <clokep> (Code is at http://code.google.com/p/vertical-tabs/ if you want to change stuff/etc.)
22:29:39 <flo> the reply-to-nick add-on helps a lot though
22:29:56 <clokep> Mook_ib there's an auto-complete page on the wiki wishlist I believe.
22:30:07 <Mook_ib> heh
22:30:29 <Mook_ib> that's why trying new software is fun - all sorts of things I want :p
22:30:50 <flo> Mook_ib: is it the first time you try Instantbird?
22:31:05 <Mook_ib> I think I've tried it a long time ago, once, but nothing recent
22:34:04 <flo> quitApplicationCmdMac is misnamed :(
22:34:27 <Mook_ib> yes, yes it is
22:35:51 <Mook_ib> http://i.imgur.com/7riy8.png still needs tab icons and d&d...
22:36:05 <flo> <popupset id="mainpopupset"/> in menus.xul is something you forgot to remove I assume?
22:36:30 <clokep> d&d was a pain to implement...which actually, not true. Getting the little arrow to line up was a pain. :)
22:37:15 <Mook_ib> flo: no, it's there for -mac to overlay into
22:37:32 <flo> Mook_ib: it's in menus.xul.inc too
22:37:50 <Mook_ib> oh, hmm, right, I guess I don't actually need it, then.
22:38:33 <clokep> Looking at Mook_ib's screenshot I was wondering what was wrong...then realized he doesn't have Show Nick installed. :)
22:38:51 <flo> clokep: ahah
22:39:14 <flo> clokep: the equally spaced bubbles look wrong to me too ;)
22:41:29 <-- chrisccoulson has quit (Quit: Ex-Chat)
22:41:37 --> chrisccoulson has joined #instantbird
22:44:14 <instantbot> florian@instantbird.org granted review for attachment 419 on bug 622.
22:44:18 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, ---, nobody, NEW, Move menu.xul.inc into an overlay
22:45:51 <Mook_ib> hmm, trying to restart the app (for addons install), it's telling me there's an unread conversation, but not which :|
22:45:55 <-- Mook_ib has quit (Quit: Instantbird 0.3a1pre)
22:46:18 --> Mook_ib has joined #instantbird
22:46:29 <Mook_ib> ... I guess typing in here made it think it's read
22:46:43 <flo> Mook_ib: without your add-on, it's usually rather obvious, the tab title is red and the conversation window where it is is focused before showing the message
22:46:53 <Mook_ib> ah, yes. :)
22:47:08 <Mook_ib> definitely an obvious case of I'm an idiot :D
22:48:16 <flo> are you going to attach a final version soon, or should I make the suggested changes locally and push it?
22:50:27 <flo> clokep: I don't see how changing the background could have made the text white in the findbar textbox :-S
22:51:07 <clokep> I'm not sure. I just figured since that change was made to the stylesheet on the conversation window, but not on the show logs window.
22:51:32 * clokep is updating his dev build so he can play w/ it.
22:51:42 <flo> clokep: do you have DOMinspector installed to check if there's a CSS rule causing trouble?
22:52:07 <clokep> On my other computer....it's updating (I'm on like a 3 week old build)
22:52:15 <instantbot> mook.moz+bugs.instantbird@gmail.com added attachment 421 to bug 622.
22:52:17 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, ---, nobody, NEW, Move menu.xul.inc into an overlay
22:52:27 <Mook> oh, sorry, didn't see you message because... I was busy attaching ;p
22:53:06 <flo> you could have obsoleted my attachment too by the way ;)
22:53:16 <Mook_ib> no, I couldn't - no editbugs :p
22:53:25 <flo> bah...
22:54:55 <-- Mook_ib has quit (Quit: Instantbird 0.3a1pre)
22:55:47 --> Mook_ib has joined #instantbird
22:57:57 <-- Jan has quit (Ping timeout)
22:58:28 --> ircjimhall has joined #instantbird
23:01:00 <flo> Mook_ib: about bug 624, do you want me to push the first hunk or have you found a solution that isn't excessively hackish to work around the issue?
23:01:05 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=624 enh, --, ---, nobody, UNCO, client.py should not require repos if not required
23:02:10 <Mook_ib> flo: up to you, really - I think all hunks make sense (if you're asking to be excluded, you should be), but I'm happy with just the first too. or doing hacks locally :)
23:02:37 * Mook_ib is just happy you're actually looking at patches; doesn't matter if it's r+, r-, wontfix, whatever :)
23:03:13 <clokep> Why would you want to build it without instantbird btw...?
23:03:56 <flo> clokep: he uses some magical git incantation to use our hg repository in git, so client.py is unable to pull the instantbird code, he has to update it by hand (if I understood correctly)
23:05:10 <clokep> Ah, I use some magical hg incantation when I need to pull git. :P
23:06:10 <Mook_ib> yeah. I just trust git more with rebasing stuff, given the, err, fun mozilla had
23:06:12 <flo> Mook_ib: I would rather not add pointless differences with the client.py in the comm-central repository. If skip_mozilla is true, I don't think the value of mozilla_repo is ever used.
23:06:29 <Mook_ib> ah, that's from c-c?
23:06:31 <flo> except if I missed something of course :)
23:06:41 <flo> yeah, our build system is a fork of comm-central
23:07:01 <flo> and we update it... not frequently enough, because it's a pain to update and merge
23:07:12 <Mook_ib> then yeah, just the first hunk is probably enough.
23:07:20 <Mook_ib> should I post a new patch?
23:07:48 <flo> if you really want. But the first hunk is a single line, I can copy/paste it in my editor ;)
23:07:56 <Mook_ib> hehe
23:08:10 <Mook_ib> that was my "start small" patch
23:08:21 <Mook_ib> (obviously didn't work well, given the size of the second patch...)
23:08:42 <flo> you mean the one that was reviewed faster? ;)
23:09:06 <flo> by the way, do you have any other itch to scratch?
23:09:46 <flo> Mook_ib: should I use m.ookm.oz+git@gmail.com for the commit, or your bugzilla address?
23:09:50 <Mook_ib> I will, soon. context menu is probably up next
23:10:02 <Mook_ib> flo: whichever, they all go to the same place. bugmail, I guess?
23:10:09 * Mook_ib should fixeup the commit stuff in that repo
23:10:32 <clokep> Context menu for what? :)
23:10:41 <flo> mook.moz+bugs.instantbird@gmail.com is a bit long ;)
23:11:33 <flo> clokep: I suppose he meant moving the content (conversation) context menu into an overlay to make it reusable by add-ons/separate conversation bindings. Not 100% sure though. :)
23:11:53 <Mook_ib> factoring out http://lxr.instantbird.org/instantbird/source/instantbird/content/instantbird.xul#123 to an overlay too
23:12:02 <Mook_ib> yeah, that was it, just looking for the link :)
23:12:45 <flo> Mook_ib: the content of <commandset id="conversationsCommands"> could benefit too
23:12:54 <flo> and <keyset id="conversationsKeys">
23:13:07 <Mook_ib> yeah, pretty much just everything :p
23:13:20 <flo> yeah :)
23:13:24 <flo> and maybe the log viewer could use it
23:14:07 <Mook_ib> I keep thinking that I'm just randomly jumping in with no context and would get all my patches rejected... but the experience has been oddly smooth so far :p
23:15:07 <flo> Mook_ib: you've been lucky.
23:15:13 <flo> or I've been maybe?
23:15:47 <Mook_ib> I guess. I hope my streak continues.
23:16:27 <flo> your patch is something I've been procrastinating for months, and the broken debug menu on all but the buddy list window hurt my workflow everyday (I need to remember to focus the buddy list before firing a magic key incantation to open DOMi/about:config/force online or a garbage collection...)
23:16:49 <Mook_ib> ah, that's what the debug menu is for
23:17:55 --> GeekShadow has joined #instantbird
23:19:00 <Mook_ib> hmm. I think I should change things to embedding instantbird.xul rather than using <tabconversation> directly - otherwise all the addons get all confused
23:19:33 <flo> Mook_ib: do you really need tabconversation?
23:19:43 <flo> I didn't use it in my all-in-one add-on
23:20:21 <Mook_ib> probably not; I do odd things to support it, in fact...
23:20:31 <flo> ah, right, the add-ons behave stupidly. We should fix them to not rely on the conversations being in tabs
23:21:01 <Mook_ib> and overlaying instantbird.xul :|
23:21:49 <flo> I guess the conversation binding should send notifications to the observer service at a few steps of creating a conversation / sending a message / whatever is needed by add-ons
23:22:36 <flo> in summary, we should do whatever is needed to make conversation.xml fully reusable/independent, not only "mostly reusable"
23:22:58 <Mook_ib> ah; in that case, probably just a "conversation view created" observer topic
23:23:05 <Mook_ib> then people can attach DOM listeners
23:23:50 <Mook_ib> though.. hmm, maybe not. not sure. (was thinking of nsIDOMDataContainerEvent)
23:23:50 <instantbot> florian@instantbird.org granted review for attachment 417 on bug 624.
23:23:53 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=624 enh, --, ---, nobody, UNCO, client.py should not require repos if not required
23:24:27 <Mook_ib> thanks for the r+s!
23:26:23 <flo> we should make code like https://hg.instantbird.org/addons/file/f58ea2b90918/colorize/content/colorize.js#l59 unnecessary
23:26:30 <flo> Mook_ib: np, ;)
23:28:01 <Mook_ib> oh wow. that's.. ick.
23:29:09 <flo> I know... I wrote that code ;)
23:29:17 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/45702da127b7 - Mook - bug 622 - move menus into overlay and clean up, r=fqueze.
23:29:18 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/d69795a2a97a - Mook - Bug 624 - client.py should not require an instantbird repository address when --skip-instantbird is used, r=fqueze.
23:31:20 <flo> Mook_ib: we should find a way to do it that's easy to use in restartless add-ons :-)
23:31:39 <Mook_ib> flo: heh, aiming high, are we? :)
23:32:02 <flo> I created my first restartless add-on a few days ago :)
23:32:19 <flo> it's so nice to install/update it without restart... :)
23:32:26 <Mook_ib> hmm, that means extensions won't always get to be around when the <conversation> gets created
23:33:02 <flo> for most add-ons that probably doesn't matter at all
23:33:27 <Mook_ib> which means a central thing, so going by the existing pattern probably observer service, topic-per-message
23:34:05 <flo> I think we have a list of all the <conversation> somewhere in imWindows.jsm
23:34:07 <flo> not sure
23:35:46 <clokep> I think we do. I haven't looked in there recently though.
23:35:50 <instantbot> florian@instantbird.org set the Resolution field on bug 622 to FIXED.
23:35:54 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=622 enh, --, 0.3a1, mook.moz+bugs.instantbird, RESO FIXED, Move menu.xul.inc into an overlay
23:36:03 <Mook_ib> hmm, new-conversation and new-text already exist as topics (for purplexpcom)
23:36:08 <clokep> And if we get the entire history story in the conversation binding somewhere...well...then it doesn't matter if they weren't there when it started. :)
23:37:32 <instantbot> florian@instantbird.org set the Resolution field on bug 624 to FIXED.
23:37:34 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=624 enh, --, 0.3a1, mook.moz+bugs.instantbird, RESO FIXED, client.py should not require repos if not required
23:38:39 --> Mic has joined #instantbird
23:38:39 * ChanServ sets mode +h Mic 
23:40:03 <clokep> Hey Mic.
23:40:09 <Mic> Hi
23:41:27 <-- Mook_ib has quit (Quit: Instantbird 0.3a1pre)
23:56:28 <flo> Good evening / night! :)
23:56:35 <clokep> 'night.
23:56:37 <Mic> Good night
23:56:55 <clokep> How do I bring up the DomI from short cuts? :)
23:59:09 <Mic> Should be Ctrl+Shift+I on Firefox but I think it doesn't work here
23:59:31 <clokep> Nah, any idea how I can use the one from AMO then? The one on AIO hasn't been updated for Mozilla 2, right?