#instantbird log on 05 30 2011

All times are UTC.

10:14:04 --> instantbot has joined #instantbird
10:14:04 topic changed by sand.mozilla.org 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/"
10:14:04 * ChanServ sets mode +v instantbot 
10:14:10 <flo> ok to commit that? Not sure who's the author of the changes :-S
10:14:15 <Mic> In short: it fixes all the borders around the conversation in the middle
10:14:19 <Mic> Wait
10:14:28 <flo> "this.base.getAccount" I prefer not duplicating, as it's something I would like to make disappear soon after the release
10:15:24 <Mic> Shall the remove-unnecessary-options-from-facebook-account-preferences-change go in the same patch?
10:16:15 <Mic> It also adds the missing border on top of the input field.. this sort of things
10:16:33 <flo> Mic: "It also adds the missing border on top of the input field.. this sort of things" which is a Win7 only issue, isn't it?
10:16:52 <flo> so yeah, I guess the question is "what was broken before these changes?" :)
10:17:07 <Mic> bah :P
10:17:12 <Mic> I'll look into it
10:17:28 <Mic> I'm back later
10:17:29 <-- Mic has left #instantbird ()
10:18:00 <flo> so basically, none of the facebook options make any sense?
10:26:35 <instantbot> florian@instantbird.org added attachment 664 to bug 532.
10:26:37 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=532 nor, --, ---, nobody, REOP, Disable "Join chat" for protocol overrides that don't support it (Facebook)
10:33:11 <-- chrisccoulson has quit (Ping timeout)
10:39:26 <instantbot> New Instantbird (UI) bug 808 filed by florian@instantbird.org.
10:39:28 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=808 enh, --, ---, nobody, NEW, UI to add/remove tags
10:40:21 --> Mic has joined #instantbird
10:40:21 * ChanServ sets mode +h Mic 
10:41:28 <instantbot> New purplexpcom - General bug 809 filed by florian@instantbird.org.
10:41:30 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=809 nor, --, ---, nobody, NEW, Handle removing a tag from a buddy even when the server ignores our request
10:44:22 <instantbot> New Instantbird (UI) bug 810 filed by florian@instantbird.org.
10:44:24 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=810 enh, --, ---, nobody, NEW, Add a 'Favorite' special tag
10:45:42 <instantbot> New Instantbird (UI) bug 811 filed by florian@instantbird.org.
10:45:44 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=811 enh, --, ---, nobody, NEW, List the tags a contact have in the UI
10:55:41 <instantbot> New Instantbird (UI) bug 812 filed by florian@instantbird.org.
10:55:42 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=812 enh, --, ---, nobody, NEW, Display the character count while typing a tweet
11:01:29 <flo> I'm trying to file everything that's left on the 0.3 roadmap but has no bug, so that we can track what's holding the 0.3 releases by looking only at bugzilla.
11:06:28 --> chrisccoulson has joined #instantbird
11:07:58 <-- chrisccoulson has quit (Client exited)
11:08:12 --> chrisccoulson_ has joined #instantbird
11:09:38 <-- chrisccoulson_ has quit (Client exited)
11:09:42 --> chrisccoulson has joined #instantbird
11:13:31 --> sonny has joined #instantbird
11:17:03 --> Even has joined #instantbird
11:17:03 * ChanServ sets mode +o Even 
11:23:04 --> waynenguyen has joined #instantbird
11:25:48 --> skeledrew has joined #instantbird
11:27:27 <-- Mic has quit (Quit: Instantbird 0.3a3pre)
11:29:29 <-- varuna has quit (Ping timeout)
11:36:10 <-- mepine has quit (Ping timeout)
11:43:39 --> varuna has joined #instantbird
12:14:21 <-- waynenguyen has left #instantbird ()
12:15:37 <flo> anybody has strong feelings for/against integrating the "Reply to Nick" add-on by default?
12:38:47 --> clokep has joined #instantbird
12:38:47 * ChanServ sets mode +h clokep 
12:40:41 <flo> clokep: hello :)
12:40:52 <clokep> Good morning flo.
12:41:19 --> igorko has joined #instantbird
12:41:55 <clokep> deOmega: You cannot set privacy per person for chat in Facebook AFAIK. You're either offline or online for all your buddies.
12:41:58 <-- igorko has left #instantbird ()
12:45:39 <flo> clokep: have you ever noticed any useful advanced option in either the facebook or twitter protocol override?
12:46:02 <clokep> Every option in Facebook should be removed I believe. I don't even think you can choose to connect over SSL or not.
12:46:29 <clokep> You can't change the server, you can't use custom smilies, etc.
12:46:33 <clokep> All the options are bogus.
12:46:42 <clokep> Which is reply-to-nick? ;)
12:46:51 <clokep> (I also missed a bunch of log since instantbot left. :P)
12:47:06 <flo> double click on a bubble to add the <nick>: at the beginning of the text box.
12:47:27 <flo> hmm, I can pastebin what I have :)
12:48:08 <flo> clokep: http://pastebin.instantbird.com/707
12:48:13 <clokep> Ah OK. I'm OK w/ that. I don't personally use it (I have Mook's tab complete add-on in.)
12:48:23 <clokep> Ah, thank you. :)
12:48:47 <flo> if someone said something earlier today, I haven't seen it either ;)
12:56:55 --> Mic2 has joined #instantbird
12:57:00 <Mic2> Hi
12:57:44 <Mic2> I'm not using reply-to-nick either
12:57:56 <Mic2> I wouldn't mind tab-completion though :P
12:57:59 <clokep> In terms of bug 532, the patch looks good, some of the options ofr GTalk might be valid though.
12:58:01 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=532 nor, --, ---, nobody, REOP, Improve Facebook override protocol: disable "Join chat" option, remove advanced options
12:58:29 <clokep> (You might be able to toggle SSL or not?)
12:59:55 <flo> I wonder if the "connect server" field is still useful there
13:00:11 <-- rikki has quit (Connection reset by peer)
13:00:18 --> rikki has joined #instantbird
13:00:52 <-- clokep has quit (Ping timeout)
13:02:38 <flo> hmm, we will probaby need to remove/fix the facebook/gtalk specific code there: http://lxr.instantbird.org/instantbird/source/instantbird/content/accountWizard.js#316
13:07:02 <flo> probably better to set these proto specific prefs in the getAccount function
13:07:31 <flo> gtalk supports SSL so I don't see why one would want to connect without encryption
13:17:18 <Mic2> 29 bugs are tagged with "0.3" on the whiteboard..
13:17:46 <Mic2> ..11 of them are blocking either beta or final.
13:17:46 <-- skeledrew has quit (Ping timeout)
13:19:09 <flo> bug 718 and 783 are blocking-beta but don't block the string freeze
13:19:13 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=718 nor, --, ---, nobody, NEW, Error console links to libpurple source code are broken
13:23:19 <instantbot> New Instantbird (UI) bug 813 filed by benediktp@ymail.com.
13:23:20 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=813 enh, --, ---, nobody, NEW, Allow to copy status text from the "conv-top-info"
13:24:35 <Mic2> Judging by this bug summary I've been digging through the CSS of this bar for too long already :D
13:25:08 <-- rikki has quit (Connection reset by peer)
13:25:16 --> rikki has joined #instantbird
13:25:40 <flo> I'm trying to integrate time bubbles
13:26:00 <flo> and struggling with the localization ;)
13:26:44 <Mic2> clokep and you are using it already, aren't you?
13:27:00 <flo> I've been using it for over a year, but only in english
13:27:23 <Mic2> What needs localization there?
13:27:37 <flo> and as message themes are not localizable, I need to have the localizable strings come from the theme system
13:27:37 <Mic2> I thought it only changes the spaces between bubbles?
13:27:43 <flo> the time units
13:34:16 <flo> hmm, should I be surprised that it works at the first attempt? :-S
13:37:43 --> clokep has joined #instantbird
13:37:43 * ChanServ sets mode +h clokep 
13:38:23 <clokep> Yes I'm using time bubbles flo.
13:39:47 <clokep> flo: Although GTalk supports SSL, there's an overhead so some people might connect w/o it since they'll be able to maintian a connection better?
13:40:38 <clokep> (And I meant Mic2: Yes I'm using Time Bubbles.)
13:40:56 <flo> "they'll be able to maintian a connection better" really? Or is it just what they believe?
13:41:07 <clokep> In terms of the account Wizard stuff, we should definitely change the protocol to use SSL instead of automatically checking it. :)
13:41:21 <clokep> flo: Maybe it's what they believe. I know i Have a better connection to moznet without SSL than with it.
13:41:54 <flo> that may be a server issue
13:43:37 <clokep> A touche.
13:43:41 <clokep> *Ah
13:44:34 <flo> by the way, I maintain a better connection if I add some encryption overhead with an SSH tunnel to a stable and reliably connected server ;)
13:45:16 --> waynenguyen has joined #instantbird
13:45:56 <clokep> :) Fair enough.
13:46:03 <clokep> I withdraw my objection!
13:46:16 <clokep> Is there a bug about having links in status messages? (Is that even possible?)
13:46:31 <flo> we can still revert the change between the beta and final if there are too many upset people
13:46:52 <flo> yes. I think it contains the word "interactive" in its summary
13:46:56 <clokep> Breakfast bbl
13:54:02 <Mic2> flo: are you going to replace the Bubble theme by Time Bubbles theme?
13:54:10 <Mic2> I thought you'd just add it as extra theme
13:54:12 <flo> yes
13:54:31 <flo> I would like to add an option for the feature
13:54:46 <flo> but the option system for message themes is currently more or less inexistant (+ it would require localization :())
13:56:09 <Mic2> If you'll make it a separate theme you'd be able to fix these problems later
14:05:28 <flo> whatever we do, we can fix it later :)
14:11:59 <Mic2> I meant: you can have both already without having to fix localization or options first.. ;)
14:14:45 <flo> I don't need the old one
14:15:11 <flo> the option would be mostly for people with the "OMG change" reaction
14:17:11 <Mic2> https://wiki.instantbird.org/File:IbTags.PNG
14:17:45 <-- clokep has quit (Ping timeout)
14:31:59 <Mic2> hmm, I think I can turn this into a patch later .. currently it lists all the tags between the top of the contact and the buddies and breaks them into new lines if the width is too low to put them all into one line
14:32:08 <Mic2> If there's interest to do it this way
14:45:07 <-- waynenguyen has quit (Ping timeout)
15:10:52 <-- GeekShadow has quit (Quit: http://www.mibbit.com ajax IRC Client)
15:13:14 <flo> it would be nice to be listed on https://www.facebook.com/sitetour/chat.php
15:18:24 --> clokep has joined #instantbird
15:18:25 * ChanServ sets mode +h clokep 
15:18:35 --> GeekShadow has joined #instantbird
15:20:36 <Mic2> Do you think so..? It's only clients that have no 'Facebook Chat' protocol in their lists.. we'd be out-of-place there ;)
15:31:01 <flo> it's still facebook promoting clients, with a nice download link ;)
15:36:23 <instantbot> florian@instantbird.org added attachment 665 to bug 532.
15:36:25 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=532 nor, --, ---, nobody, REOP, Improve Facebook override protocol: disable "Join chat" option, remove advanced options
15:36:28 <flo> feedback welcome on that patch ;)
15:42:48 <Mic2> bye
15:42:54 <-- Mic2 has quit (Quit: http://www.mibbit.com ajax IRC Client)
15:54:44 --> igorko has joined #instantbird
16:00:42 <-- clokep has quit (Ping timeout)
16:01:13 <flo> arg, I didn't even respect the coding style on that file for the space between the color components :( http://lxr.instantbird.org/instantbird/source/instantbird/themes/blist.css#268
16:07:06 <-- Even has quit (Quit: Instantbird 0.3a3pre)
16:17:23 --> DetroitLibertyPenguin has joined #instantbird
16:17:26 --> clokep has joined #instantbird
16:17:27 * ChanServ sets mode +h clokep 
16:55:56 <-- clokep has quit (Ping timeout)
17:19:31 <-- sonny has left #instantbird ()
17:22:22 --> sonny has joined #instantbird
17:22:39 <-- sonny has left #instantbird ()
17:58:19 <instantbot> florian@instantbird.org set the Resolution field on bug 504 to FIXED.
17:58:23 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=504 enh, --, 0.3a3, florian, RESO FIXED, Use CSS3
17:59:20 <instantbot> florian@instantbird.org set the Resolution field on bug 532 to FIXED.
17:59:23 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=532 nor, --, 0.3a3, florian, RESO FIXED, Improve Facebook override protocol: disable "Join chat" option, remove advanced options
18:02:36 <-- varuna has quit (Quit: ChatZilla 0.9.86.1 [Firefox 3.6.16/20110323163108])
18:04:14 <-- flo has quit (Quit: Instantbird 0.3a3pre)
18:04:17 <-- igorko has quit (Quit: Instantbird 0.3a3pre)
18:04:33 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/b4caa7cf3f8b - Florian Quèze - Port 'Bug 517097 - make enabling debug symbols more sane' and 'Bug 554024 - switch mozilla-central Linux builds to use -gdwarf-2 instead of -gstabs' changes to mozconfig
18:04:34 <instantbot> files.
18:04:35 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/5e3af713ee10 - Florian Quèze - Make sound for contact events rather than buddy events to reduce pointless noise for contacts with several buddies.
18:04:36 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/1023849b0d65 - Florian Quèze - Bug 532 - Disable advanced options for the Facebook and Google Talk protocol overrides, and disable 'Join Chat' for Facebook.
18:04:37 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/b06de022100f - Florian Quèze - Integrate Time Bubbles' features into the default Bubbles message theme.
18:04:38 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/07fab1b2715e - Florian Quèze - Follow-up to bug 649 - purple proxies should not be in the way of JavaScript protocols: override protocols should have the same proxy behavior as their base.
18:04:39 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/3a3d0e64df1b - Florian Quèze - Bug 504 - Use the CSS3 name (without the '-moz' prefix) of the border-radius, box-shadow and background-origin properties.
18:04:40 <instantbot> Check-in: http://hg.instantbird.org/instantbird/rev/0cade9d0f74e - Florian Quèze - [Mac] Disable history in the hidden window to avoid a warning.
18:38:40 <-- micahg has quit (Connection reset by peer)
18:39:33 --> micahg has joined #instantbird
18:46:44 --> Even has joined #instantbird
18:46:44 * ChanServ sets mode +o Even 
18:54:30 --> mokush has joined #instantbird
19:21:23 --> clokep has joined #instantbird
19:21:23 * ChanServ sets mode +h clokep 
19:29:14 --> sonny has joined #instantbird
19:29:52 <-- sonny has left #instantbird ()
19:32:49 <clokep> Uhh...should I be disabling Time Bubbles then? :P
19:32:54 --> linuxwizard has joined #instantbird
19:34:58 <-- clokep has quit (Quit: Instantbird 0.3a3pre)
19:35:04 --> clokep has joined #instantbird
19:35:04 * ChanServ sets mode +h clokep 
19:54:44 --> flo has joined #instantbird
19:54:44 * ChanServ sets mode +qo flo flo 
20:00:13 <flo> clokep: you can switch the preference back to Bubbles, it should continue to display the times correctly tomorrow :)
20:02:21 <clokep> flo: Oh true! I forgot I was technically adding another theme.
20:02:30 <clokep> You guys got a lot done this weekend!
20:03:34 <flo> "weekend"... pfff
20:03:58 <flo> Thursday is a holiday in France though :-P
20:04:41 <clokep> I've been busy. :P
20:04:49 <clokep> Off today though. Gonna start looking at some stuff.
20:06:27 <flo> I cleaned up my todo list, hence the small unimportant commits
20:11:39 <flo> triaging the bugs, especially deciding which things we shouldn't hope to have time do any more is frustrating :-S
20:16:49 <-- mokush has quit (Client exited)
20:39:30 <-- Even has quit (Ping timeout)
20:44:43 --> EionRobb has joined #instantbird
20:48:14 <-- GeekShadow has quit (Quit: The cake is a lie !)
20:50:23 --> sonny has joined #instantbird
20:56:57 <-- clokep has quit (Quit: Instantbird 0.3a3pre)
20:57:21 --> clokep has joined #instantbird
20:57:21 * ChanServ sets mode +h clokep 
21:08:17 <instantbot> clokep@gmail.com added attachment 666 to bug 618.
21:08:18 <instantbot> clokep@gmail.com requested review from florian@instantbird .org for attachment 666 on bug 618.
21:08:20 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=618 enh, --, ---, clokep, ASSI, Basic Jump List support on Windows 7
21:14:17 <flo> clokep: what happens with the default action if the user has both -status <something> and -preferences on the command line?
21:14:42 <clokep> flo: To start Instantbird or with Instantbird already running?
21:14:52 <flo> to start it
21:15:16 <clokep> flo: I get a core not initialized error. :-/
21:15:25 <clokep> I didn't try that one. :(
21:15:38 <flo> ahah, that's not what I expected :-D
21:16:14 <flo> where do you see that error by the way?
21:16:28 <clokep> http://pastebin.instantbird.com/708 is the error
21:16:56 <flo> the error is good :)
21:17:12 <flo> have you added -jsconsole to the command line too or something to see it?
21:17:34 <clokep> C:\Users\clokep\instantbird\objdir-ib-release\mozilla\dist\bin>instantbird.exe -
21:17:34 <clokep> P dev -no-remote -status offline -jsconsole -preferences
21:17:56 <flo> ok :)
21:18:09 <clokep> Is this something I need to change or...?
21:18:37 <clokep> (As in, is it an error in my code or somewhere else.)
21:18:50 <flo> I was wondering if in a case where something else prevented the default action already, your cmdLine.preventDefault =(cmdLine.state != Ci.nsICommandLine.STATE_INITIAL_LAUNCH); would accidentally restore the default action or not
21:19:34 <clokep> Ah, OK.
21:19:48 <clokep> So it's working as it should! :)
21:20:02 <flo> it's failing before that line :-D
21:20:14 <flo> because the 2 options I mentioned are incompatible
21:20:24 <clokep> Right.
21:20:42 <flo> but if someday we add an option to start minimized to the tray (which is not the default action), it may make sense to specify the status at the same time
21:21:24 <flo> I was basically wondering if we should change that line to if (cmdLine.state != Ci.nsICommandLine.STATE_INITIAL_LAUNCH)\ncmdLine.preventDefault = true;
21:22:03 <clokep> Ah, so that way you leave it alone if it's already set?
21:22:44 <clokep> I originally had that and changed it since it seemed simpler the way I have it now, I didn't realize there could be other issues.
21:23:00 <flo> yes
21:23:26 <flo> I'm not sure of how this value is handled
21:24:45 <flo> you put 2 spaces instead of the tab on the new line in jar.mn
21:25:36 <clokep> Bah I just fixed that I thought.
21:25:54 <clokep> Bleh apparently I need to copy and paste a tab to put it into that file.
21:26:02 <clokep> Sorry about that.
21:26:06 <flo> you used a getter for the first description property, and forgot for the next 2
21:26:59 <clokep> Dammit.
21:27:08 --> Mic has joined #instantbird
21:27:08 * ChanServ sets mode +h Mic 
21:27:23 <-- Mic has left #instantbird ()
21:27:24 <clokep> Fixed all of that.
21:27:31 --> Mic has joined #instantbird
21:27:31 * ChanServ sets mode +h Mic 
21:27:53 <Mic> clokep: thanks for taking the localization on bug 620
21:27:57 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=620 enh, --, ---, benediktp, ASSI, Use Glass on Aero (Windows Vista and 7)
21:28:09 <Mic> hmm, the other bug.. not this one :D
21:28:12 <flo> should it be tested on XP to check that the "Return early if we can't use the jumplist service" code works without causing trouble, or are you already sure of it? :)
21:28:15 <clokep> bug 618?
21:28:19 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=618 enh, --, ---, clokep, ASSI, Basic Jump List support on Windows 7
21:28:28 <clokep> flo: I believe Mic tested on Windows XP.
21:31:13 <Mic> Maybe with a very early version
21:31:20 <Mic> I certainly haven't in the last week
21:31:29 * clokep doesn't have access to an XP machine anymore.
21:31:33 <clokep> I don't even have a disk with it. :-/
21:31:49 <Mic> I think I'll have access to one tomorrow in a library ;)
21:32:16 <Mic> Do you have a build with jumplist support that you could upload somewhere?
21:32:16 <flo> http://mxr.mozilla.org/mozilla-central/source/widget/public/nsIWinTaskbar.idl#89
21:33:08 <Mic> I think I copied the "available-check from Firefox jumplist handling module
21:33:57 <flo> it's not clear to me why there's an available property on both the nsIWinTaskbar and the nsIJumpListBuilder interfaces
21:34:12 <Mic> http://mxr.mozilla.org/mozilla-central/source/browser/components/wintaskbar/WindowsJumpLists.jsm#203
21:35:15 <clokep> Maybe in case only one is supported? I'm not sure when that would occur though.
21:35:29 <clokep> Mic: I can upload a build if need be. :)
21:35:36 <flo> I'm looking at the code
21:36:06 <flo> the first one is implemented at http://mxr.mozilla.org/mozilla-central/source/widget/src/windows/WinTaskbar.cpp#348
21:36:25 <Mic> Would be great if it's not too inconvenient for you
21:37:17 <clokep> Mic: Of course, just need to find somewhere to do it. :)
21:39:12 <Mic> You can always send it to my ymail adress
21:39:27 <Mic> Yahoo mail can receive attachments upto 25MB if I'm not mistaken
21:40:28 * flo opens the XP VM
21:40:36 <Mic> Or any of the file hosting services .. into a dropbox .. whatever ;)
21:40:39 <flo> it will be faster to just try the code in the error console than to understand it
21:41:17 <clokep> Mic: Yup! Gonna wait until flo's comments first.
21:42:48 <flo> Warning: reference to undefined property Components.classes['@mozilla.org/windows-taskbar;1']
21:42:56 <flo> Source File: javascript:%20Components.classes["@mozilla.org/windows-taskbar;1"].getService(Components.interfaces.nsIWinTaskbar).createJumpListBuilder();
21:43:13 <flo> Error: Components.classes['@mozilla.org/windows-taskbar;1'] is undefined
21:43:45 <flo> of course the Win7 SDK is not installed in my VM... :-/
21:47:21 <flo> Firefox has a check for that: http://mxr.mozilla.org/mozilla-central/source/browser/components/nsBrowserGlue.js#371
21:48:01 <clokep> Hmm...I see.
21:48:08 <flo> clokep: I think you need a similar code
21:48:46 <clokep> flo: Yes, probably. In the module or in ibCore?
21:49:53 <flo> I would put it in ibCore and move down the Cu.import call (inside the test) so that there's a single ifdef
21:50:01 <flo> but if you prefer in the module, it's fine too
21:51:07 <clokep> OK.
21:53:47 <clokep> flo: http://pastebin.instantbird.com/709 is what I have for ibCore now.
21:54:34 <flo> Nit: for consistency, add a name (WJL_<property name>) to the init and reset methods, or remove it on _getSeparatorItem/_getHandlerAppItem
21:55:34 <clokep> OK.
21:55:43 <clokep> Those have two __ in them, is that for a reason Mic?
21:56:37 <flo> clokep: it's WJL + "_" + name (which starts with an _)
21:56:52 <clokep> Ah, :) Right. I had figured that out last time haha.
21:57:11 <Mic> I'd rename the init method to "set" or something..
21:57:14 <flo> clokep: the pastebined code looks good.
21:57:27 <Mic> (btw)
21:57:41 <clokep> Mic: It's "init" to match the other methods in ibCore.
21:57:47 <flo> wasn't it named like this so that it looks similar to the other methods in ibCore?
21:57:53 <clokep> Yes.
21:58:35 <flo> but Mic has a good point. If reset is supposed to be used by add-ons, you need another method for them
21:59:01 <flo> you need the createJumpListBuilder call only once
21:59:09 <flo> but add-ons may need the following code several times
21:59:24 <clokep> So there should be an init which calls set. ;)
21:59:31 <Mic> That's what I was thinking
21:59:41 <flo> exactly
22:01:03 <clokep> Got it. :)
22:01:24 <clokep> And I should probably sav ethe output of !this.winJumpListBuilder || !this.winJumpListBuilder.available so I can check in both methods?
22:02:56 <flo> you can put the result of winTaskbar.createJumpListBuilder() in a local variable and put it in this.winJumpListBuilder only if winJumpListBuilder.available is true
22:03:23 <flo> and then you need to test only that this.winJumpListBuilder is not null, in both set and reset
22:04:28 <clokep> :) Much cleaner than how I was going to do it!
22:04:34 <flo> there are lots of comments that are sentences that could benefit from receiving a point at their end.
22:04:43 <flo> and after that, I'm out of improvement ideas
22:05:45 <clokep> I'll read over the comments again then and add periods.
22:06:14 <flo> the è in my name doesn't seem to be in UTF8, but I assume you tested it and it compiles, so it really doesn't matter :)
22:07:02 <clokep> In splash.rc?
22:07:07 <Mic> good night
22:07:18 <clokep> 'night Mic.
22:08:12 <-- Mic has quit (Quit: Instantbird 0.3a3pre)
22:08:24 <flo> clokep: yes
22:08:41 <clokep> Yeah, the file wasn't UTF-8, just changed it and it still compiles. :)
22:10:24 <instantbot> florian@instantbird.org denied review for attachment 666 on bug 618.
22:10:26 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=618 enh, --, ---, clokep, ASSI, Basic Jump List support on Windows 7
22:10:48 <clokep> Posting another one now. just trying to make the diff from the latest to show my changes.
22:11:27 <clokep> Which, isn't Bugzilla able to do that? :P
22:11:54 <-- sonny has quit (Quit: Leaving.)
22:12:03 <flo> because Even still hasn't fixed that
22:12:47 <clokep> Mmhmm.
22:13:02 <clokep> Is there an easy way to make a diff like that?
22:14:20 <flo> interdiff patch1 patch2
22:14:50 <clokep> I'm guessing that's a *nix command? :)
22:15:05 <clokep> Or just that I would need to have committed in between?
22:15:09 <flo> it's part of diffutils I think
22:15:11 <clokep> Wait no...
22:15:13 <clokep> Ah. OK.
22:16:38 <flo> hmm, if you commit the first patch, then unapply it (patch -R) then apply the second one, you can hg diff and see it :-D (and then you hg rollback the useless commit)
22:20:35 <-- linuxwizard has left #instantbird ()
22:22:16 <instantbot> clokep@gmail.com added attachment 667 to bug 618.
22:22:17 <instantbot> clokep@gmail.com requested review from florian@instantbird .org for attachment 667 on bug 618.
22:22:19 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=618 enh, --, ---, clokep, ASSI, Basic Jump List support on Windows 7
22:23:38 <instantbot> clokep@gmail.com added attachment 668 to bug 618.
22:25:48 <flo> that's not an interdiff
22:25:58 <flo> it's a diff of 2 patch files
22:29:09 <flo> you have not changed the cmdLine.preventDefault = line
22:29:31 <flo> another nit: helpInfo : <-- no space between helpInfo and ':'
22:30:12 <clokep> I know it's not a real interdiff. I don't have diffutils.
22:30:23 <clokep> Ah, I didn't realize we had decided to change that one.
22:31:49 <flo> I don't understand your init/set split
22:32:42 <flo> http://log.bezut.info/instantbird/today#m307 it's not like I suggested at least
22:35:24 <clokep> Ahhh, that shouldn't work. I fixed it.
22:36:18 <clokep> Sorry for that. :-/
22:37:24 <flo> what I suggested would look like http://pastebin.instantbird.com/710
22:38:40 <clokep> Yeah what I had changed it to mostly.
22:38:50 <clokep> I read it wrong, sorry.
22:39:00 <clokep> (Plus wrote code that doesn't even make sense.)
22:39:03 <flo> if you feel that we need to avoid double initialization (I don't think so), you can add if (this.winJumpListBuilder) throw Cr.NS_ERROR_ALREADY_INITIALIZED;
22:40:12 <clokep> I had thought of that, although i wasn't sure what to throw exactly. But I don't thik there's really any code that will die if you run it twice.
22:41:46 <flo> and throwing may be destructive if for some reason the second code is ibCore and the first one was a broken add-on ;)
22:42:21 <clokep> Yeah, I think it's fine to just re-init. :) Should I add a comment saying that this is called by the core and add-ons shouldn't call it?
22:49:17 <flo> doesn't hurt :)
22:50:27 <clokep> :)
22:55:38 <flo> I'm attempting to fix the "display character count in twitter conversations" bug
22:56:29 <clokep> Ah, I see. Protocol specific UI is very....bleh. :(
22:58:15 <flo> that part already works
22:58:37 <flo> I'm stuck on handling keyboard events :-D (yes, this is stupid)
22:59:48 <clokep> So it updates when you type? That sounds like an important part. :P
23:00:05 <flo> yeah
23:00:34 <flo> the problem is... it updates before the typed character is taken into account, because adding the character is the default action for the event... so custom handlers are processed first
23:01:25 <flo> we already had the same issue when trying to send typing notifications. We handled that with a setTimeout (bleh... x-()
23:08:16 <clokep> Oh. :-/ Yes. And there's too many ways to edit the text.
23:08:39 <clokep> (Backspace, pasting, adding text, deleting, deleting multiple characters at once.)
23:09:53 <flo> dragging!
23:14:54 <clokep> :(
23:15:00 <clokep> Sounds like set timeout is the way to go.
23:16:07 <flo> O_o why do I have to add a .bind(this) to make something that used to already work work again :-S
23:16:51 <clokep> That's probably not good. :-/
23:18:01 <-- chrisccoulson has quit (Ping timeout)
23:24:10 <flo> bah, it works. Too good to be true :)
23:30:24 <flo> clokep: are you going to attach another diff soon (= before I go to bed)?
23:31:06 <clokep> flo: I can, yes. :) I was reheating some salmon and asparagus. ;)
23:32:06 <clokep> I need to test it. I'll be back.
23:33:12 <-- clokep has quit (Quit: Instantbird 0.3a3pre)
23:33:23 --> clokep has joined #instantbird
23:33:23 * ChanServ sets mode +h clokep 
23:33:24 <-- clokep has quit (Connection reset by peer)
23:34:18 --> clokep has joined #instantbird
23:34:18 * ChanServ sets mode +h clokep 
23:34:53 --> sonny has joined #instantbird
23:41:16 <instantbot> clokep@gmail.com added attachment 669 to bug 618.
23:41:17 <instantbot> clokep@gmail.com cancelled review?(florian@instantbird .org) for attachment 667 on bug 618.
23:41:18 <instantbot> clokep@gmail.com requested review from florian@instantbird .org for attachment 669 on bug 618.
23:41:19 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=618 enh, --, ---, clokep, ASSI, Basic Jump List support on Windows 7
23:49:06 <flo> clokep: want to review that twitter thing? :)
23:49:11 <instantbot> florian@instantbird.org added attachment 670 to bug 812.
23:49:13 <instantbot> Bug https://bugzilla.instantbird.org/show_bug.cgi?id=812 enh, --, ---, florian, ASSI, Display the character count while typing a tweet
23:50:56 <-- flo has quit (Input/output error)
23:51:19 <clokep> flo: Sure.
23:51:41 --> flo has joined #instantbird
23:51:41 * ChanServ sets mode +qo flo flo 
23:53:41 <clokep> flo: Where does this._statusTextEnd = left < 200 ? left.toString() : ""; come from?
23:53:44 <clokep> The 200 I mean.
23:53:53 <flo> magic :)
23:54:12 <flo> I don't want to display huge numbers in the status bar
23:54:20 <flo> and I would like in the future if we could set the values for other protocols too
23:54:38 <flo> (on IRC I think you receive it from the server at connect-time)
23:55:10 <flo> a comment above/around it may be a good idea
23:56:09 <flo> clokep: when should this.reset be called exactly?
23:56:14 <clokep> Yes, there's one for IRC...although for IRC it's the command + user name + a few spaces + what you wrote. ;) So it's awfully complicated.
23:56:51 <clokep> flo: I call this.reset to delete any current jump list that is set for Instantbird (which generally will just clean up any garbage left from a previous run, etc. if we ever change what the jump list does)
23:57:17 <flo> shouldn't it be called before each set call?
23:57:34 <clokep> Hmmm...let me think.
23:57:40 <flo> in my pastebined example the .reset call was inside the set method rather than init
23:58:16 <clokep> Well, you can actually add multiple jump lists, so not necessarily. But I guess it should probably be in the set method.
23:59:20 <clokep> flo: I'd r+ that with a comment saying the 200 is magic to prevent large numbers from showing. :) (And I think I'd increase it to at least a number over 255 since I think that's the default IRC one.)
23:59:57 <flo> ah, I thought it was much more on IRC