Monday, 2011-09-26

*** Joins: soc42 (~soc42@g227016198.adsl.alicedsl.de)00:50
*** Joins: louisremi (~louisremi@mon69-3-87-90-54-237.dsl.sta.abo.bbox.fr)00:53
*** Joins: johnwards (~johnwards@95.177.127.123)00:55
*** Joins: aaronmcadam (~aaron@109.204.121.170)01:02
*** Joins: beevi7 (~manuel.bi@tim.7val.com)01:07
*** Joins: jaubourg (~jaubourg@109.128.110.145)02:16
*** Parts: louisremi (~louisremi@mon69-3-87-90-54-237.dsl.sta.abo.bbox.fr) ()02:30
*** Quits: sean` (~seankoole@D97A9E4C.cm-3-3c.dynamic.ziggo.nl) (Quit: Ik ga weg)02:36
*** Quits: soc42 (~soc42@g227016198.adsl.alicedsl.de) (Remote host closed the connection)02:36
*** Quits: temp02 (~temp01@unaffiliated/temp01) (Read error: Connection reset by peer)02:50
*** Joins: temp01 (~temp01@unaffiliated/temp01)02:52
*** Joins: scott_gonzalez (~scott_gon@cpe-069-134-000-200.nc.res.rr.com)02:58
*** Quits: gnarf (~gnarf@unaffiliated/gnarf) (Ping timeout: 260 seconds)03:41
*** Quits: parshap (~parshap@ip68-5-37-109.oc.oc.cox.net) (Ping timeout: 260 seconds)03:49
*** Joins: gnarf (~gnarf@unaffiliated/gnarf)03:51
*** Quits: scott_gonzalez (~scott_gon@cpe-069-134-000-200.nc.res.rr.com) (Quit: scott_gonzalez)03:58
*** Joins: sean` (~seankoole@D97A9E4C.cm-3-3c.dynamic.ziggo.nl)04:12
*** Joins: scott_gonzalez (~scott_gon@cpe-069-134-000-200.nc.res.rr.com)04:25
*** Joins: Rwhitbeck (~Rwhitbeck@cpe-67-253-202-235.rochester.res.rr.com)04:35
*** Quits: Rwhitbeck (~Rwhitbeck@cpe-67-253-202-235.rochester.res.rr.com) (Quit: Computer has gone to sleep.)04:40
*** Quits: Frippe (~Frippe@unaffiliated/frippe) (Quit: Computer has gone to sleep)04:45
*** Joins: kswedberg (~kswedberg@173-162-44-162-michigan.hfc.comcastbusiness.net)05:17
*** Joins: skamikaze (~basti@unaffiliated/skamikaze)05:26
*** Quits: mikeycgto (~mikeycgto@pdpc/supporter/active/mikeycgto) (Ping timeout: 260 seconds)05:30
*** Joins: Justin_o (~justin@205.211.169.2)05:36
*** Joins: erichynds (~ehynds@venkman.brightcove.com)05:39
*** Joins: mikeycgto (~mikeycgto@pdpc/supporter/active/mikeycgto)05:43
*** Quits: timmywil (~timmywil@host-68-169-154-67.WISOLT2.epbfi.com) ()05:45
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)05:47
*** Joins: DaveMethvin (~DaveMethv@pool-173-69-166-27.bltmmd.fios.verizon.net)05:51
*** Quits: Mannequin (~Mannequin@186.23.142.29) (Ping timeout: 248 seconds)05:52
*** Joins: darcyclarke (~darcyclar@rrcs-24-39-117-197.nyc.biz.rr.com)05:58
*** Joins: timmywil (~timmywil@sec1.epb.net)06:02
*** Joins: miketaylr (~miketaylr@206.217.92.186)06:06
*** Joins: danzik17 (~danzik17@164.55.254.106)06:10
*** Joins: Mannequin (~Mannequin@186.23.142.29)06:12
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)06:15
*** Joins: boaz (~boaz@li326-230.members.linode.com)06:16
jaubourgfuuuuuu06:16
bot-t(17 hours 50 mins ago) <tbranyen> tell jaubourg question for you about ajaxTransport when you come back06:16
*** Joins: kborchers (~kborchers@st-216-125-152-244.wcc.cc.il.us)06:16
jaubourgtbranyen: if you're around, so am I06:16
tbranyenjaubourg: oh i got it figured out.  was trying to determine the best way to "fall back" to a different transport06:32
tbranyenso instead of triggering a 404 in mine, fall back to the normal ajax06:32
tbranyenand i realized your brillance shown by just not returning an object :-p06:32
*** Joins: dunderline (~jpeterson@adsl-70-131-100-183.dsl.emhril.sbcglobal.net)06:37
jaubourghehe06:41
jaubourgyeah, transport selection is a "I'll give you something if I can, ask someone else if I can't" thingy06:42
tbranyenyeah i wasn't sure if i was going to have to make a custom dataType06:43
tbranyenthis works really well right now tho06:43
tbranyenhttps://github.com/tbranyen/vertebrae/blob/master/lib/plugins/jquery.js06:43
*** Quits: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net) (Ping timeout: 248 seconds)06:44
*** Quits: skamikaze (~basti@unaffiliated/skamikaze) (Quit: skamikaze)06:47
*** Joins: irae_ (~irae_@nat/yahoo/x-jnbfsxkhbcmecpst)06:52
gnarfmornin06:56
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)06:57
gnarfmeeting today?06:57
timmywilyup06:58
jaubourgguess who won't be able to attend?06:59
* jaubourg spanks himself06:59
timmywilthe belgian06:59
jaubourg\o/06:59
*** Quits: erichynds (~ehynds@venkman.brightcove.com) (Ping timeout: 256 seconds)07:04
*** Joins: Jon47 (~jonz@204.56.125.50)07:05
danheberdenjaubourg when are you flying out?07:11
jaubourgto or from boston?07:11
danheberdenwhen do you arrive in boston07:11
jaubourgfriday in the afternoon07:12
gnarfhrm07:12
jaubourgiirc07:12
gnarfthis unit failure is bugging me07:12
jaubourggnarf: the IE7 needing a closure one?07:12
gnarfi'm getting timers still running / leaking in cache after trying http://jqbug.com/p51307:12
jaubourgoh07:12
gnarfand i can't figure out what anything I did would have to do with that07:13
jaubourgyou're just running effects tests?07:13
gnarfeffects tests are the only ones failing07:14
gnarfbut if i ?filter=effects07:14
gnarfthey all pass07:14
gnarfif I don't07:14
gnarf....07:14
DaveMethvingnarf: all browsers?07:14
gnarfchrome atm07:15
DaveMethvini had a few probs unless I ran incognito07:15
DaveMethvinsome of my plugins screw up the dom i guess07:15
gnarfi've seen those failures07:16
gnarfi don't think this is the same07:16
gnarfhttp://cl.ly/2f3W3y103f2j1i2q1J2v07:16
DaveMethvinI think it's because you declared `parts` in a var statement. :P07:17
gnarfheh07:17
DaveMethvini wonder if some other test is leaving crap behind?07:17
DaveMethvini just almost did that with some propHooks tests07:17
gnarfYeah, my new tests seem okay07:19
DaveMethvingnarf, is it possible this is due to the qunit update?07:19
gnarfum, maybe07:19
DaveMethvinit's caused several issues07:19
gnarfbut that doesn't explain why it passes in the branch without my changes07:20
DaveMethvingnarf, the first test fail is the test above "animate with per-property easing", right?07:23
DaveMethvinsince it must leave timers running07:23
gnarfi would think that the error would be on the test thats failing07:23
gnarfsince its in the teardown that the check is done07:23
gnarfwtf.... they didn't fail this time07:24
*** Joins: erichynds (~ehynds@venkman.brightcove.com)07:25
DaveMethvinwell, that's convenient07:25
DaveMethvinbtw gnarf which version of chrome?07:26
gnarfdev07:26
gnarfbut same fails in ff07:27
DaveMethvinoh ok07:27
DaveMethvinstrange07:27
gnarfyeah07:27
DaveMethvinand it went away on a fresh run of the full unit tests?07:27
gnarfi mean i didn't mess with anything that should mess with that07:27
DaveMethvindid you take the window out of focus?07:27
gnarfit seems to fail about 80% of the time07:27
DaveMethvinor are you leaving it up on screen?07:27
DaveMethvinjust wondering if it has something to do with the way timers are treated in background07:28
gnarfmaybe?07:28
gnarfhrm07:29
DaveMethvinwas that a FIRM maybe? :P07:29
gnarfokay07:29
gnarfthat just like crept in on me07:29
gnarfi've seen the focus selector07:29
gnarfso07:29
gnarfnow there is another one failing07:29
gnarfoffset: scroll07:29
jaubourgguys, sometimes it's near stand-up comedy when you chit-chat about where a bug can possibly come from :)07:29
jaubourgcan't help but smile while being completely un-helpful07:30
gnarfno fails if i kept the window focused other than this offset thing07:30
DaveMethvinah, and ff/chrome are the only two that mess with timers07:31
jaubourggnarf: have you tried to setTimeout( start, 0 )? SetTimeout( __, 0 ) is my all-healing patch07:31
jaubourgattack of the timers07:31
DaveMethvinor the ever-popular .then( start, start ) // start damn you07:31
jaubourghehe07:32
DaveMethvini gotta tellya, i don't get no respect. just yesterday i set a timer for 100ms and it didn't fire until 1 second later07:32
timmywili still don't understand that07:32
jaubourgI could also switch to always( start ), silly me, dunno why I left this "then" it's sooo 1.507:32
DaveMethvinhmmm, gnarf, does that mean we'll leak timers when someone has a tab not visible? that sounds bad07:33
jaubourgtimmywil: the need for a closure? No idea... I guess start is now using this?07:33
timmywilit's not tho07:33
jaubourgDave: we won't leak them... they'll just stay longer07:33
timmywilhence my confusion07:33
DaveMethvinoh yeah07:33
jaubourgat the very least ff and chrome should honor setTimeout( _, 0 )07:34
DaveMethvinsetImmediate07:34
jaubourgat least until we have this setImmediate07:34
jaubourgheh07:34
gnarfI still think we should wrap that07:34
gnarfcuz we seem to use setTimeout 0 a bit07:34
jaubourgwe should, so that we can provide an imp with setImmediate07:34
jaubourg$.later07:34
DaveMethvincan we just shim window.setImmediate?07:35
timmywilyum07:35
DaveMethvinso how is everyone's stuff doing for 1.7?07:36
DaveMethvinrwaldron and i have been trying to get the propHooks stuff finished, but i'm stuck trying to figure out why the unit test won't work in IE907:37
gnarfaside from these tests failing07:37
*** Joins: colinclark (~colin@205.211.169.2)07:37
DaveMethvinyeah my prob is unit tests too07:37
DaveMethvini say we skip them :P07:37
gnarfI should be done with my blockers07:37
timmywilI'll work on getText tonight.  didn't have time over the weekend07:38
jaubourgDave: I still need a review... I have to make my prez and need to know if I can count on the terms I use in the code :P07:38
DaveMethvinfor callbacks?07:38
jaubourgyep07:38
DaveMethvink, i'll look at that by tonight07:38
jaubourgmethod names, flag names, etc07:38
gnarfhrm07:38
gnarfDaveMethvin: now this time they failed again07:38
gnarfwith the window active07:38
gnarf....07:39
timmywilDaveMethvin: word on hotel?07:39
DaveMethvinoh yeah looks like they are booked07:39
DaveMethvinfor sunday07:39
timmywilmmmmm07:39
DaveMethvinbut most likely we can find something07:39
jaubourghttp://joseoncode.com/2011/09/26/a-walkthrough-jquery-deferred-and-promise/ <= particularly like the explanation of pipe and the recursive use of it07:39
DaveMethvinwe can work it out there07:39
timmywilyea, or i'll be on the street07:40
DaveMethvinoh jaubourg btw the Windows 8 html-js stuff is using promises all over the place07:40
jaubourg:)07:40
DaveMethvindon't worry timmywil we'll find you a place07:40
jaubourgcoolness07:40
jaubourgyeah, I heard John's porch is quite nice07:41
* jaubourg hides07:41
jaubourgDave: any API doc anywhere?07:41
DaveMethvinyeah lemme get you a link07:42
timmywiljaubourg: better than my porch07:42
timmywilim sure07:42
jaubourgwish I had one myself... "come and squat my porch" sounds sooo cool ;P07:42
timmywillol07:43
jaubourgman, I so can't wait to be there already... except I need to do my prezo like yesterday :/07:43
jaubourgI'm wondering if I shouldn't talk about pipe a bit for people behind on deferreds07:43
timmywilyea, i still need to update my pres07:44
DaveMethvinhttp://gist.github.com/124238807:44
DaveMethvincheck out those crazy namespaces07:44
DaveMethvinand yeah jaubourg i think you should mention pipe07:44
DaveMethvinb/c they are crazy useful07:44
gnarfI hope my presentation isn't too dry/useless ;)07:45
DaveMethvinthe windows 8 promise impl doesnt' have a pipe-ish thing07:45
jaubourgDave: oh man, will I get sued if I say the namespacing is of a java-like epic proportion?07:45
jaubourgDave: they're probably Promise/A compliant and have then handle that07:46
DaveMethvinjaubourg: oh yeah i totally agree, they took their .net-ish namespacing and pushed it into javascript07:46
jaubourgin Promise/A, then === pipe, except it's not quite clear from the spec what happens if you return nothing, which is solved with pipe (you use pipe, whatever you return -- even nothing -- is the filtered value)07:47
gnarfso, the fact that this doesn't fail reliably is really f'n annoying07:48
DaveMethvingnarf: yeah, this is like when you have zoom set07:48
DaveMethvinthings just fail and it doesn't make sense07:48
gnarflol i got caught by that one recently too07:48
DaveMethvini get caught by it about once a month, just often enough to forget and it takes me about five minutes of saying , "damn this looks familiar"07:49
DaveMethvinbut it's really annoying that you have to leave the window in focus and visible else it will misbehave07:49
gnarfyeah07:49
gnarfespecially when it gets worse07:50
DaveMethvini suppose we could start using the page visiblity api and/or focus detection07:50
DaveMethvinmight be something to think about for the animation rewrite as wll07:50
jaubourgwheeeeeeeeee animation rewrite07:51
jaubourgDO WANT07:51
DaveMethvinyeah07:51
DaveMethvinjaubourg: here are the windows 8 samples, atm most of them include the javascript api wrappers in their zip files07:52
DaveMethvinhttp://code.msdn.microsoft.com/windowsapps/07:52
gnarfokay, this time through - it got caught up in the test07:52
jaubourgI would love it if nodejs would embrace promises07:53
DaveMethvinyeah it's the perfect application07:53
jaubourgthat would make working with its API so much easier07:53
gnarfOMG07:53
gnarfthis time it passed07:53
gnarfi changed NOTHING07:53
DaveMethvinbut gnarf did you keep it in focus this time?07:53
jaubourggnarf: yes you did, you need to type in the chat and talk to us... then everything works \o/07:54
DaveMethvinmaybe our unit tests just crave attention, yeah07:54
gnarfokay07:54
gnarfsorry for the delayed response07:54
gnarfi was re-running the unit07:54
gnarfthis time its failing07:54
jaubourgDave: are you implying our unit tests are attention whores?07:54
gnarfand all 3 times i kept the window front and focused07:55
DaveMethvinwell they certainly are demanding a lot of attention from gnarf and me07:55
jaubourgdamn, just got an image of blond chicks chewing bubble-gum with t-shirts saying "unit test this"07:55
DaveMethvinof course mine is an ie issue so that's par for the course07:55
DaveMethvineasy there big fella07:55
jaubourghehe07:55
*** Quits: jzaefferer (~jzaeffere@205.186.165.147) (Ping timeout: 240 seconds)07:56
DaveMethvinso gnarf you're saying that the three tests gave different results? please say that you're not saying that :/07:56
gnarfDaveMethvin: i just ran the test suite, 3 times, hitting refresh - waiting07:56
gnarffail - pass - fail07:56
DaveMethvintake #207:56
jaubourg:|07:56
*** Quits: paul_irish (~paul_iris@ve.hsh6wjwx.vesrv.com) (Ping timeout: 260 seconds)07:57
gnarfi'll screenshare this shit just to have someone else to freak out about it with me07:57
gnarflol07:57
DaveMethvinso is everything ok without this patch?07:57
gnarfno...07:59
gnarfi just checkout'ed master07:59
gnarflemme pull to newest08:00
gnarfmake sure i'm there08:00
DaveMethvinok well at least you know this patch didn't cause it ... but we don't know what did, ugh08:01
gnarfactually08:01
gnarfi think i tracked it down08:01
gnarfsomehow i managed to have a master that was ahead of upstream08:01
gnarfnow to rebase this pull08:02
DaveMethvinpull -f that sucker08:02
gnarfhrm08:02
gnarfdamnit!08:04
gnarfokay08:04
gnarfi know i'm on clean master now08:04
gnarfand if i run the test once - pass08:05
gnarfrefresh - run again - fail08:05
DaveMethvinand the same tests are failing?08:05
gnarfyup08:05
gnarfhttp://cl.ly/322a0q1U0m06131I062708:06
gnarfokay08:06
gnarfnoone else is getting these right?08:06
*** Quits: beevi7 (~manuel.bi@tim.7val.com) (Ping timeout: 276 seconds)08:06
gnarfi just got it on chrome stable08:06
gnarftoo08:06
DaveMethvinlemme see how chrome 13 fares08:06
DaveMethvinhmmm, i am getting them too08:09
DaveMethvinran incognito to be sure there was no interference08:10
gnarfhttp://cl.ly/2l3u273b0n0C082L1f2308:10
gnarfim seeing this one occasionally too08:10
*** Joins: anfernee (~Adium@218.79.89.161)08:11
gnarfalright08:11
gnarfi guess i'm debugging these tests for real now08:11
*** Joins: rwaldron (~rwaldron@75-150-66-249-NewEngland.hfc.comcastbusiness.net)08:11
gnarftime to try bisect :)08:13
jaubourgI *did* mention I won't be at the meeting, yes?08:13
timmywiljaubourg: you always miss the meeting08:14
jaubourgyeah, it's weird08:14
jaubourgworked like a charm for me before summer08:14
jaubourgsince then, I always end up with something at that very spot08:14
jaubourgalways something different too08:15
jaubourg:/08:15
gnarfwtf08:16
gnarfgit checkout 1.6.4 ; git submodule update ; make --- run tests --- its locking up....08:16
jaubourglike just hanging?08:16
DaveMethvinok, now i'm REALLY starting to suspect qunit08:16
gnarfstalling on Running manipulation: append the same fragment with events (Bug #6997, 5566)08:16
timmywilthat's expected08:17
gnarfstalling on 1.6.4 is expected?08:17
timmywilwith the new qunit yes08:17
DaveMethvinwith the new qunit08:17
gnarfi should be on the qunit that was with 1.6.4 right?08:17
timmywilit won't downgrade unless you force a reset08:17
gnarfgit submodule update checks it out08:18
gnarfgit status doesn't show (new commits)08:18
gnarfso i am assuming I'm on the version of qunit that was tagged with 1.6.408:18
timmywilah yes, you didn't do make08:19
timmywilwell then08:19
gnarfi did make, just after git submodule update08:19
timmywilhmm, ok, i did the same and mine doesn't hang08:20
gnarfwtfbbq08:20
timmywili also have cache disabled08:20
* gnarf clones fresh08:20
jaubourgfresh is a nice project name08:20
* jaubourg hides08:21
timmywilmchammer fresh08:22
jaubourgyaw dawg08:22
gnarfokay08:23
*** Quits: anfernee (~Adium@218.79.89.161) (Quit: Leaving.)08:23
gnarfwhatever was happening there isn't happening on a fresh clone...08:23
timmywili am getting effects and offset fails on the current master with the new qunit08:25
timmywilseems they all have to do with timers08:25
DaveMethvinyeah, seeing them as well with chrome 13 and firefox 608:25
timmywilrunning current master with old qunit08:27
timmywiland it seems everything passes08:27
timmywilso i vote we revert qunit and let jörn deal with adding those extra timers08:28
timmywilDaveMethvin, gnarf: what say you08:28
gnarfi say let me dig a second and see if i can't find why08:29
DaveMethvinand if  you can't find out i agree, let's revert ... there's another pull from AaronAsAChimp that exhibits fails with the latest qunit too08:30
jaubourg+1 on reverting08:34
timmywili might have a fix08:34
jaubourganyway, I gotta fly08:34
timmywilcyl jaubourg08:34
jaubourghave a nice meeting guys08:34
jaubourgbbl :)08:34
*** Joins: michaeldeol (~michaelde@wsip-70-168-157-254.oc.oc.cox.net)08:34
*** Joins: anfernee (~Adium@218.79.95.213)08:35
gnarfso - it seems qunit 7f292170fa1109f1355f3e96f8973c32fc553946 is solid08:37
gnarftimmywil: what qunit sha were you using?08:38
timmywilthe 1.6.4 one08:38
timmywilthat one08:38
*** Joins: stevecrozz (~stevecroz@173.227.0.2)08:40
gnarf61137fd6d1813b5f9ab2dfe7e4749ef1d51115d9 is the first bad commit08:45
timmywilyea nvm, i don't have a fix08:45
*** Joins: sonotos (~sonotos@unixboard/users/sonotos)08:45
gnarfhttps://github.com/jquery/qunit/commit/61137fd6d1813b5f9ab2dfe7e4749ef1d51115d908:45
socialhapy★ Commit on qunit by wwalser (1w, 1d ago): added parameter to start and stop allowing a user to increment/decrement the semaphore more than once per call08:45
timmywili figured08:45
timmywilit changed the api for stop08:46
DaveMethvinyeah and they say WE don't respect the API :P08:46
timmywilDaveMethvin: well, we fixed that08:46
DaveMethvinyeah08:46
timmywilmore importantly, the change adds timers so that our timer check fails08:46
timmywili don't know what the timers are for08:47
timmywilor how exactly they are getting added and they weren't before08:47
gnarfi think its to allow you to stop( 1000 );08:47
gnarfsaying if it doesn't finish in 1000ms consider it failed08:47
timmywilgnarf: that's what it used to do08:47
gnarfbut they changed it08:48
gnarf?08:48
timmywilnow it sets a timeout based on the config a certain number of times based on that "semaphore"08:48
timmywilwhich means we may have stray timers08:48
timmywilso stop(1000) would set 1000 timers08:48
*** Quits: anfernee (~Adium@218.79.95.213) (Quit: Leaving.)08:49
aaronmcadamis it an api decision to not allow the settings of { buttons : null } in a jquery ui dialog?08:50
DaveMethvinaaronmcadam: best to ask in #jquery08:51
aaronmcadami have done08:51
aaronmcadamI was just wondering, the api is confusing08:51
aaronmcadamno prob08:52
gnarf?channels08:52
bot-tFor support please go to #jquery or #jquerymobile. Development Channels: #jquery-dev - #jqueryui-dev - #jquerymobile-dev ; Meetings held in#jquery-meeting08:52
gnarf;)08:52
gnarfi'm not sure, and kinda in the middle of something08:52
aaronmcadamno prob08:52
*** Joins: mokush (~quassel@188.24.43.235)08:52
gnarfso i guess timmywil that my pull is okay then08:54
gnarfit seems to pass with the parent to that commit08:54
*** Quits: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net) (Ping timeout: 255 seconds)08:55
timmywilok, i'll roll out a revert08:59
timmywilhas something to do with the start/stop calls, but I don't think we're using them incorrectly08:59
*** Quits: johnwards (~johnwards@95.177.127.123) (Quit: johnwards)08:59
timmywiland we're never using arguments08:59
*** Joins: colinclark_ (~colin@205.211.169.2)09:00
*** Quits: colinclark (~colin@205.211.169.2) (Read error: Connection reset by peer)09:00
*** colinclark_ is now known as colinclark09:00
*** Joins: jqcommit (~jqcommit@sh1-ext.rs.github.com)09:01
jqcommit[jquery] timmywil pushed 1 new commit to master: http://git.io/S1Avrg09:01
jqcommit[jquery/master] Revert QUnit. Recent start/stop changes cause problems with some start/stop calls in effects and offset. Run git submodule update - timmywil09:01
*** Parts: jqcommit (~jqcommit@sh1-ext.rs.github.com) ()09:01
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)09:02
timmywil?tell jzaefferer http://git.io/S1Avrg  We reverted QUnit.09:04
bot-ttimmywil, Okay.09:04
*** Joins: bp0 (~bp@216.254.228.94)09:05
*** Quits: bp0 (~bp@216.254.228.94) (Read error: Connection reset by peer)09:05
kborcherspoor jzaefferer, first he's not wearing his own underwear, now this ;)09:09
*** Quits: aaronmcadam (~aaron@109.204.121.170) (Ping timeout: 248 seconds)09:11
rwaldronwait09:12
rwaldronwtf?09:12
rwaldroni thought stop() already expected an arg09:13
rwaldronfor timeout?09:13
rwaldron(or is that the issue)09:13
rwaldronsorry, was working on a popcorn.js bug09:13
*** Quits: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net) (Ping timeout: 255 seconds)09:14
*** Joins: anfernee (~Adium@218.79.85.14)09:17
*** Joins: henrikkok (~henrikkok@3306ds3-amb.0.fullrate.dk)09:19
timmywilrwaldron: they changed what the argument is, but that wasn't really the problem09:28
timmywilin making that change, they created an issue for us with their semaphore09:28
timmywili don't really understand it all09:29
rwaldronyeah... that sucks09:29
rwaldroni have a ticket for popcorn.js to upgrade qunit09:30
rwaldronwe use the timeout arg09:30
* rwaldron shrugs09:30
*** Quits: mokush (~quassel@188.24.43.235) (Remote host closed the connection)09:30
DaveMethvinhey rwaldron09:33
*** Joins: jaimerod (~jaimerod@dsl211-250-066.lax1.dsl.speakeasy.net)09:34
rwaldronDaveMethvin hey dude09:34
DaveMethvinoh man, i just cannot get this unit test to work on ie909:35
DaveMethvinthis fiddle works: http://jsfiddle.net/PECfF/09:35
*** Quits: anfernee (~Adium@218.79.85.14) (Quit: Leaving.)09:36
DaveMethvinthis unit test does not: https://github.com/dmethvin/jquery/blob/fix-8789-rwldrn-fix/test/unit/event.js#L240309:36
DaveMethvinthe events don't fire09:36
DaveMethvinworks fine in chrome and firefox natch09:37
*** Joins: jrburke (~jrburke@150.183.119.66.static.metrobridge.net)09:37
gnarfDaveMethvin: what are you trying to do?09:37
DaveMethvinfire a focus event using the native dom focus method09:38
gnarftriggering .focus() ?09:38
DaveMethvinyeah09:38
rwaldronerr09:38
gnarfhave you tried delaying the test to the next loop?09:38
rwaldronam i missing something in your jsfiddle?09:38
gnarfsetImmediately09:38
gnarf;)09:38
rwaldronhow would blurrinessLevel ever get added?09:38
DaveMethvinrwaldron: it's not added the first time, that fiddle is just a reduction of the unit test showing the event is/isn't fired09:39
rwaldronoh cool09:39
*** Joins: anfernee (~Adium@218.79.85.14)09:39
rwaldronso... dig this09:39
DaveMethvingnarf i see your point09:39
rwaldronhttp://jsfiddle.net/rwaldron/nxgVN/1/09:39
DaveMethvinof course then why does it work in fiddle09:39
rwaldronDaveMethvin ^09:39
rwaldron(run in IE9)09:39
gnarfDaveMethvin: because its fully async09:40
gnarfthe elem is getting removed before the focus happens?09:40
gnarfin your test09:40
gnarfthats my theory09:40
DaveMethvinand you're sticking to it :)09:40
DaveMethvini think i tried with it removed too09:40
DaveMethviner, not removed09:40
DaveMethvinlemme see09:40
rwaldronDaveMethvin did you see my fiddle?09:40
DaveMethvineyah what changed?09:40
rwaldronwell, put your cursor in the box09:41
rwaldronit fires with "focusin"09:41
* rwaldron shrugs09:41
rwaldronthen i realized thats not helpful at all09:41
rwaldronyou're trying to fire the thing programmatically09:41
rwaldroni'll shut up now09:41
DaveMethvinyeah sorry i need a differ b/c i don't see the code change09:41
rwaldronno worries09:42
DaveMethvinoh focusin!09:42
rwaldronyeah, forget it though09:42
rwaldronstill requires a person09:42
DaveMethvindamn people, screwing up our unit tests09:42
DaveMethvinjust a sec, gotta switch branches09:42
DaveMethvindamn 100 bytes ... that makes me sad09:43
rwaldronDaveMethvin http://jsfiddle.net/rwaldron/nxgVN/3/embedded/result/09:43
* DaveMethvin shakes fist at gnarf for taunting me with size info :)09:43
gnarfhey - timmywil requested it09:44
gnarfi just complied09:44
rwaldronDaveMethvin that is firing on it's own in IE909:44
DaveMethvinsorry rwaldron i prolly wasn't clear .. the fiddle works (fires the event) but the unit test doesn't09:44
DaveMethvinin ie909:44
rwaldrongotcha09:45
DaveMethvinand i just dont get it09:45
gnarfDaveMethvin: http://jsfiddle.net/nxgVN/4/ in ie9 says what?09:45
gnarfdoes OMG happen after or before ? :)09:45
DaveMethvinafter09:45
DaveMethvin:P09:45
gnarfhrm09:46
gnarfbut not in the unit?09:46
*** Joins: Frippe (~Frippe@unaffiliated/frippe)09:46
DaveMethvini just commented out the delete and unbind at the end, still not firing09:46
*** Quits: sean` (~seankoole@D97A9E4C.cm-3-3c.dynamic.ziggo.nl) (Quit: Ik ga weg)09:47
DaveMethvinof course now i get a memory leak reported since i didn't unbind ... :)09:47
*** Quits: Frippe (~Frippe@unaffiliated/frippe) (Client Quit)09:47
*** Joins: Frippe (~Frippe@unaffiliated/frippe)09:47
DaveMethvinbut interestingly enough the now-visible box did not get focus, the cursor isn't in it09:47
DaveMethvinoh yes it did09:48
DaveMethvini must have just clicked off it09:48
gnarfi really wish i had ie909:48
DaveMethvinnow THERE's something i've never heard anyone say before09:48
gnarf:)09:48
gnarfis it ie9 only? or ie8 as well?09:49
DaveMethvinwell since ie9 didn't work i didnt' bother to check09:49
gnarfwell after i finish cloning this ie8 vm so i can upgrade it to 909:49
gnarfi'll look again :)09:49
DaveMethvingnarf, same breakage in IE7 so they're all conspiring against me09:50
rwaldronso it looks like filter is never run?09:51
DaveMethvinright, i put a breakpoint and it isn't09:51
DaveMethvinalso the event never seems to be triggered09:51
rwaldronyeah09:51
DaveMethvinso that explains why filter isn't run :)09:51
gnarfmaybe its the blur that isn't working09:51
rwaldroni pulled the branch, so I'm poking around the source09:52
DaveMethvinif it was a failure in blur the first event woudl fire tho09:52
gnarfso neither event is firing?09:52
DaveMethvinright, neither one09:52
rwaldronhm09:53
gnarfwhat about a console.log( document.activeElement ) ?09:53
rwaldroni just changed the event name to something else and it works09:53
rwaldronyou probably knew that09:53
gnarfjust curious if its actually switching09:53
DaveMethvinrwaldron: like what event?09:53
rwaldronhahaha09:53
rwaldronthe word "fuck"09:54
DaveMethvinlol09:54
rwaldronand trigger("fuck")09:54
DaveMethvinoh sure, if you trigger09:54
DaveMethvinbut that's not a native event09:54
DaveMethvinalthough it woudl be sweet if it was09:54
DaveMethvindiv.onfuck = function()09:54
rwaldronrght, but it wasnt working by triggering focus either09:54
gnarfonfuck="alert( peen(5) );"09:54
rwaldronDaveMethvin: click works09:55
rwaldronbind("click")09:55
rwaldron[0].click()09:55
DaveMethvinyeah but try in chrome/ff09:55
rwaldronworks in chrome09:56
rwaldronworks in FF09:56
DaveMethvindamn, there was some reason why i didn't use click09:56
DaveMethvini thought it broke somewhere09:56
DaveMethvinlike, i tried click but it broke09:57
DaveMethvinlast night09:57
rwaldronclick is a mouseevent09:58
rwaldronwith mouseProps09:58
DaveMethvinoh right09:58
DaveMethvinwell let me clobber it and see if it works otherwise09:58
rwaldronfails your clobber test09:58
gnarfso hrm... .trigger("focus") wasn't working?09:58
*** Parts: stevecrozz (~stevecroz@173.227.0.2) ()09:58
rwaldronbut we could save the real hook, define our own and restore it09:59
*** Joins: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net)09:59
rwaldronwwwwwwwwwwooorrrks10:00
DaveMethvingnarf: appears so10:00
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)10:00
gnarfthats a little scary - why wouldn't .trigger() call the event bound?10:00
gnarfremoving the [0] right?10:00
DaveMethvinoh i think i know the reason for that10:01
rwaldronDaveMethvin https://github.com/rwldrn/jquery/commit/d11fcfac186d64457d54d7e80f7d06b97b1c344a10:01
socialhapy★ Commit on jquery by rwldrn (50.893s ago): Fixed tests to run in IE910:01
rwaldronIE7: pass10:02
DaveMethvinlooks good except the blur is still there ... guess it doesn't hurt tho10:02
rwaldroneh, looks like it wasnt doing anything?10:02
rwaldroni need to restore the hook10:02
rwaldronall set10:03
rwaldronDaveMethvin https://github.com/rwldrn/jquery/commits/fix-8789-rwldrn-fix10:03
*** Quits: anfernee (~Adium@218.79.85.14) (Quit: Leaving.)10:04
DaveMethvinyou want to put in a pr for that?10:04
rwaldronsure10:04
DaveMethvini'll get it from there10:04
rwaldronhttps://github.com/jquery/jquery/pull/51910:04
socialhapy★ Pull request on jquery by rwldrn (6.596s ago): Event Prop Hooks, #878910:05
rwaldroni should close the original10:05
DaveMethvineyah10:05
DaveMethvinyeah even10:05
rwaldronclosed10:05
rwaldronwith a note for the new one10:05
DaveMethvinbut anyway gnarf, if ie isn't firing the event on focus then we aren't calling the handlers10:05
DaveMethvinnow for as why ie isn't firing the event i don't know. i do know that ie will not let you focus a hidden element10:06
gnarfright10:06
gnarfbut it passes in the fiddle10:06
DaveMethvinyeah and in the unit i speicfically put it in the body and had a chekc in there to make sure it was visible10:06
DaveMethvinso it had no excuse for failing afaik10:07
rwaldronNo Excuse!10:07
DaveMethvinie don't need no stinking excuse10:07
rwaldronaaaaaaahahahaha10:08
rwaldronok, i need food10:08
DaveMethvineat!10:08
*** Joins: imp3rator (~sonotos@unixboard/users/sonotos)10:11
gnarfhttp://jsfiddle.net/3JmEd/1/ mini qunit :)10:12
*** Quits: sonotos (~sonotos@unixboard/users/sonotos) (Ping timeout: 256 seconds)10:14
*** Joins: Connor_ (~Connor@host-78-149-97-213.as13285.net)10:15
DaveMethvingnarf: so what's strange is that it works there -- the error is related to changes that are only on the branch, but the first event fired10:16
*** Joins: Krinkle (~Krinkle@wikimedia/Krinkle)10:20
gnarfhey DaveMethvin10:20
gnarfgot it to work with focus10:20
DaveMethvinso what did you do?10:21
gnarfwrapped everything starting from unbind to the end with setTimeout(...,0) and added start() to the second event handler - made it an asyncTest10:21
gnarfand it works in ie10:21
DaveMethvinso you were right all along ...10:22
gnarfyeah10:22
DaveMethvini always underestimate ie's stupidity10:23
gnarfbut10:23
gnarfwhat doesn't make sense10:23
gnarfis why http://jsfiddle.net/nxgVN/4/ appears correct10:23
DaveMethvinyeah that is why i didn't think it was an async prob10:24
DaveMethvini had done something similar last night10:24
*** Quits: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net) (Quit: Leaving...)10:24
DaveMethvinoh well i guess we'll leave it with click, i am SOOO tired of this patch10:25
DaveMethvinbut this is good to know10:25
gnarfaha10:25
*** Joins: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net)10:25
gnarfhttp://jsfiddle.net/nxgVN/8/ in ie10:25
gnarfapparently - focusin was working10:26
gnarf:)10:26
gnarf"focus" isnt10:26
*** Joins: jmotta (~jmotta@200-225-197-223.static.ctbctelecom.com.br)10:26
DaveMethvinugh10:26
jmottaI have a string that was rendered in my html as a return of a db query, but there are double white spaces on it and when I obtain these strings using jquery the double white spaces disappear (e.g. the original string was "something  here" and when I obtain the value I get "something here")10:26
gnarfsupport in #jquery ? :)10:26
DaveMethvinjmotta: try #jquery although i don't see jquery in there anywhere10:27
jmottahm10:27
jmottathanks10:27
*** Parts: jmotta (~jmotta@200-225-197-223.static.ctbctelecom.com.br) ("Leaving")10:27
gnarfDaveMethvin: yeah, i ran into some weirdness with .focus() in my mask unit tests10:27
gnarfDaveMethvin: only reason i knew to suggest delaying looking for it10:27
DaveMethvinactually let me ask you about a change i made to fix a different bug .. now i'm starting to worry about it10:28
DaveMethvinin 1.7 i have focus/blur fire the native focus/blur method, and *that* fires the focus/blur event and we run handlers off that10:29
DaveMethvinin 1.6 we ran the handlers directly, then fired the method and blocked the handlers from running a second time10:29
DaveMethvinwhich ensures the handlers fire even if ie doesn't focus or blur10:29
DaveMethvinso it seems like this test case is saying that might not be a good idea10:30
gnarfwell apparently neither is your "blocking"10:30
gnarfmight cause a dupe fire right?10:30
DaveMethvinthat's a result of how trigger() has always fired events, it's backwards of native events10:31
DaveMethvinnormally you run the DOM .click() or whatever method, and that fires onclick, and runs handlers10:31
DaveMethvinbut in jquery we run the handlers then run the DOM .click() method but  block any event that comes out of it10:32
DaveMethvinso for focus/blur I was trying to do it the "right" way  and run the dom method, letting that fire events10:32
DaveMethvinwhich solves a few sticky ui problems with the fact that focus doesn't bubble for example but focusin does10:33
gnarfswitching that order around might cause regression?10:33
DaveMethvinyeah technically it could10:33
DaveMethvinthe main problem is that .trigger("blur") is nothing like a native blur event10:34
DaveMethvinbut now it's a lot more like a native blur event in 1.710:34
DaveMethvinit runs the blur method, then triggers blur on the target, then triggers focusout on the parents10:35
timmywilgit st10:36
timmywilwoops10:36
timmywilyou're not my terminal!10:36
DaveMethvinrm -rf /10:37
timmywilha10:37
*** Joins: jqcommit (~jqcommit@sh1-ext.rs.github.com)10:40
jqcommit[jquery] dmethvin pushed 3 new commits to master: http://git.io/Ws9UtA10:40
jqcommit[jquery/master] Merge branch 'fix-8789-rwldrn-fix' of git://github.com/dmethvin/jquery into fix-8789-rwldrn-fix - Rick Waldron10:40
jqcommit[jquery/master] Fixed tests to run in IE9 - Rick Waldron10:40
jqcommit[jquery/master] Restore original click prop hook - Rick Waldron10:40
*** Parts: jqcommit (~jqcommit@sh1-ext.rs.github.com) ()10:40
*** Joins: AzaToth (~azatoth@wikipedia/AzaToth)10:42
DaveMethvindamn shoulda squashed10:43
gnarfyup10:43
timmywilpulley < no love10:43
DaveMethvinbtw there seems to be NO working precompiled node.js that works with pulley10:43
DaveMethvinthat's what was causing the crash i get10:43
DaveMethvinbut the older one crashes too10:43
timmywilhmmmmmm?10:43
timmywilwhat version?10:43
DaveMethvini am looking for the version of the one i'm using, no luck10:44
DaveMethvinnode -v ftw10:45
DaveMethvin0.4.710:45
DaveMethvinthat's the one that works well enough for make10:45
timmywilhmm, ok, well i'm on 0.4.810:45
timmywilcan't imagine pulley wouldn't work with 0.4.7 tho10:46
DaveMethvinwell, something is broke with it10:46
*** Joins: bp0 (~bp@unaffiliated/bp0)10:50
*** Joins: sean` (~seankoole@D97A9E4C.cm-3-3c.dynamic.ziggo.nl)10:54
*** Joins: colinclark_ (~colin@205.211.168.17)11:02
*** Joins: jzaefferer (~jzaeffere@205.186.165.147)11:02
*** Quits: colinclark (~colin@205.211.169.2) (Ping timeout: 245 seconds)11:04
*** colinclark_ is now known as colinclark11:04
*** Quits: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net) (Ping timeout: 256 seconds)11:14
*** Joins: johnwards (~johnwards@host81-141-9-62.wlms-broadband.com)11:18
*** Quits: colinclark (~colin@205.211.168.17) (Ping timeout: 256 seconds)11:18
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)11:19
*** Quits: johnwards (~johnwards@host81-141-9-62.wlms-broadband.com) (Client Quit)11:20
*** Joins: colinclark (~colin@205.211.168.17)11:23
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)11:33
*** Joins: paul_irish (~paul_iris@ve.hsh6wjwx.vesrv.com)11:34
paul_irishHaxe ships with jQuery nw11:35
paul_irishthat's great news11:35
paul_irish:D11:35
*** Joins: boaz (~boaz@li326-230.members.linode.com)11:35
*** Joins: eddiemonge (~eTiger13@70-36-226-152.dsl.static.sonic.net)11:40
*** Parts: eddiemonge (~eTiger13@70-36-226-152.dsl.static.sonic.net) ()11:40
*** Joins: parshap (~parshap@ip68-5-37-109.oc.oc.cox.net)11:43
*** Quits: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net) (Ping timeout: 258 seconds)11:45
*** Joins: kriszyp (~kriszyp@71-38-225-51.slkc.qwest.net)11:46
*** Quits: colinclark (~colin@205.211.168.17) (Quit: colinclark)11:49
*** Quits: ajpiano (~ajpiano@li98-57.members.linode.com) (Excess Flood)11:49
rwaldronpaul_irish somehow i've never heard of that beore11:49
rwaldronbefore*11:49
*** jrburke is now known as jrburke|away11:49
rwaldronbut, awesome!11:49
paul_irishyah. real crazy hackers use HaXe11:49
*** Quits: miketaylr (~miketaylr@206.217.92.186) (Read error: Connection reset by peer)11:49
rwaldronlooks very interesting11:50
*** Joins: miketaylr (~miketaylr@206.217.92.186)11:50
*** Joins: ajpiano (~ajpiano@li98-57.members.linode.com)11:51
*** Quits: ajpiano (~ajpiano@li98-57.members.linode.com) (Excess Flood)11:51
*** Joins: ajpiano_ (~ajpiano@li98-57.members.linode.com)11:52
*** ajpiano_ is now known as ajpiano11:52
*** Quits: imp3rator (~sonotos@unixboard/users/sonotos) (Remote host closed the connection)11:52
*** Joins: danzik171 (~danzik17@164.55.254.106)12:02
*** Quits: danzik17 (~danzik17@164.55.254.106) (Ping timeout: 248 seconds)12:05
*** boaz is now known as benalman12:20
*** benalman is now known as boaz12:21
*** tbranyen is now known as tbranton12:21
*** Quits: ajpiano (~ajpiano@li98-57.members.linode.com) (Excess Flood)12:21
*** Joins: ajpiano (~ajpiano@li98-57.members.linode.com)12:22
DaveMethvin"boaz is now known as benalman"  <-- I KNEW IT!  And Superman is Clark Kent12:25
*** Joins: colinclark (~colin@205.211.169.2)12:34
*** Quits: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net) (Quit: Leaving...)12:41
*** Joins: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net)12:43
*** Quits: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net) (Read error: Connection reset by peer)12:43
*** Joins: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net)12:43
boazlol DaveMethvin!12:45
*** Quits: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net) (Ping timeout: 260 seconds)12:48
*** jrburke|away is now known as jrburke12:59
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)13:07
*** Quits: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com) (Quit: Leaving...)13:08
*** Joins: boaz (~boaz@li326-230.members.linode.com)13:09
*** Joins: colinclark_ (~colin@205.211.168.17)13:15
*** Quits: colinclark_ (~colin@205.211.168.17) (Remote host closed the connection)13:16
*** Joins: colinclark_ (~colin@205.211.169.2)13:16
*** Quits: colinclark (~colin@205.211.169.2) (Read error: Connection reset by peer)13:16
*** colinclark_ is now known as colinclark13:16
*** Quits: Jon47 (~jonz@204.56.125.50) (Quit: Leaving.)13:24
*** Quits: erichynds (~ehynds@venkman.brightcove.com) ()13:29
*** Joins: rworth (~rworth@pool-72-83-231-22.washdc.fios.verizon.net)13:29
*** Quits: Connor_ (~Connor@host-78-149-97-213.as13285.net) (Quit: Leaving)13:31
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)13:42
*** Joins: boaz (~boaz@li326-230.members.linode.com)13:44
*** Quits: miketaylr (~miketaylr@206.217.92.186) (Quit: miketaylr)14:01
*** Quits: rwaldron (~rwaldron@75-150-66-249-NewEngland.hfc.comcastbusiness.net) (Ping timeout: 248 seconds)14:09
*** Joins: wesbos (~wesbos@CPE0017c521d8a2-CM001225d72fa4.cpe.net.cable.rogers.com)14:13
*** Quits: kswedberg (~kswedberg@173-162-44-162-michigan.hfc.comcastbusiness.net) (Ping timeout: 248 seconds)14:16
*** Joins: Jon47 (~jonz@204.56.125.50)14:22
*** Quits: wesbos (~wesbos@CPE0017c521d8a2-CM001225d72fa4.cpe.net.cable.rogers.com) (Ping timeout: 260 seconds)14:23
*** Quits: Mannequin (~Mannequin@186.23.142.29) (Ping timeout: 248 seconds)14:24
*** Quits: kborchers (~kborchers@st-216-125-152-244.wcc.cc.il.us) (Quit: Back to work code monkey!)14:25
*** Quits: timmywil (~timmywil@sec1.epb.net) (Ping timeout: 248 seconds)14:29
*** Joins: timmywil (~timmywil@sec1.epb.net)14:30
*** Quits: timmywil (~timmywil@sec1.epb.net) (Read error: Connection reset by peer)14:32
*** Quits: irae_ (~irae_@nat/yahoo/x-jnbfsxkhbcmecpst) (Quit: irae_)14:32
*** Joins: timmywil (~timmywil@sec1.epb.net)14:32
*** Quits: timmywil (~timmywil@sec1.epb.net) (Ping timeout: 258 seconds)14:51
*** Quits: henrikkok (~henrikkok@3306ds3-amb.0.fullrate.dk) (Quit: Leaving.)14:53
*** Joins: Mannequin (~Mannequin@186.23.142.29)14:55
*** Joins: simonleojung (~simonleoj@141.117.180.89)14:57
*** Quits: Justin_o (~justin@205.211.169.2) (Quit: Justin_o)14:59
*** Joins: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com)15:00
*** Joins: sonotos (~sonotos@unixboard/users/sonotos)15:05
*** tbranton is now known as tbranyen15:11
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)15:13
*** Joins: boaz (~boaz@li326-230.members.linode.com)15:14
*** Quits: Jon47 (~jonz@204.56.125.50) (Quit: Leaving.)15:15
*** Quits: jaimerod (~jaimerod@dsl211-250-066.lax1.dsl.speakeasy.net) (Quit: HydraIRC -> http://www.hydrairc.com <- Chicks dig it)15:29
*** Quits: Frippe (~Frippe@unaffiliated/frippe) (Quit: Computer has gone to sleep)15:32
*** Quits: sonotos (~sonotos@unixboard/users/sonotos) (Remote host closed the connection)15:37
*** Quits: ben_alman (~ben_alman@web126.webfaction.com) (Excess Flood)15:40
*** Joins: ben_alman (~ben_alman@web126.webfaction.com)15:40
*** Joins: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net)15:54
*** Quits: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net) (Client Quit)15:57
*** Joins: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net)16:04
*** Quits: colinclark (~colin@205.211.169.2) (Ping timeout: 258 seconds)16:10
*** Quits: darcyclarke (~darcyclar@rrcs-24-39-117-197.nyc.biz.rr.com) (Quit: Linkinus - http://linkinus.com)16:14
*** Quits: danzik171 (~danzik17@164.55.254.106) (Ping timeout: 260 seconds)16:18
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)16:23
*** Joins: boaz (~boaz@li326-230.members.linode.com)16:24
*** Joins: timmywil (~timmywil@host-68-169-154-67.WISOLT2.epbfi.com)16:56
*** Joins: Rwhitbeck (~Rwhitbeck@cpe-67-253-202-235.rochester.res.rr.com)17:00
*** Quits: simonleojung (~simonleoj@141.117.180.89) (Quit: simonleojung)17:03
*** Quits: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com) (Ping timeout: 260 seconds)17:05
*** Joins: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com)17:05
*** Quits: jrburke (~jrburke@150.183.119.66.static.metrobridge.net) (Quit: jrburke)17:06
*** Quits: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net) (Quit: Leaving.)17:14
*** Quits: AzaToth (~azatoth@wikipedia/AzaToth) (Remote host closed the connection)17:14
*** Joins: darcyclarke (~darcyclar@cpe-98-14-132-12.nyc.res.rr.com)17:17
*** Joins: rubyruy (~ruy@76.77.91.41)17:19
*** Quits: rubyruy (~ruy@76.77.91.41) (Client Quit)17:20
*** Joins: rubyruy (~ruy@76.77.91.41)17:21
*** Joins: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com)17:21
*** Quits: bp0 (~bp@unaffiliated/bp0) (Remote host closed the connection)17:34
socialhapy★ Tweet from jquery: If you are attending the jQuery Conference this weekend don't forget to follow @jqcon for conference updates. #jqcon ★ http://bit.ly/pmjgL717:47
*** Joins: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net)17:53
*** Quits: Krinkle (~Krinkle@wikimedia/Krinkle) (Quit: Too bad a day only lasts 24 hours. See you on another one!)17:57
*** Joins: miketaylr (~miketaylr@24.42.93.245)18:00
*** Quits: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com) (Quit: Colloquy for iPad - http://colloquy.mobi)18:10
*** Quits: michaeldeol (~michaelde@wsip-70-168-157-254.oc.oc.cox.net) (Ping timeout: 260 seconds)18:13
*** Quits: darcyclarke (~darcyclar@cpe-98-14-132-12.nyc.res.rr.com) (Quit: Bye!)18:14
*** Quits: scott_gonzalez (~scott_gon@cpe-069-134-000-200.nc.res.rr.com) (Quit: scott_gonzalez)18:14
*** Quits: rubyruy (~ruy@76.77.91.41) (Quit: Bye!)18:21
*** Quits: dunderline (~jpeterson@adsl-70-131-100-183.dsl.emhril.sbcglobal.net) (Quit: dunderline)18:31
*** Quits: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com) (Quit: Linkinus - http://linkinus.com)18:32
*** Joins: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com)18:32
*** Joins: simonleojung (~simonleoj@CPE687f7488464b-CM00137189dc6c.cpe.net.cable.rogers.com)18:43
*** Joins: anfernee (~Adium@210.13.73.174)18:44
*** Quits: boaz (~boaz@li326-230.members.linode.com) (Excess Flood)18:47
*** Joins: boaz (~boaz@li326-230.members.linode.com)18:48
*** Joins: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com)18:58
*** Quits: miketaylr (~miketaylr@24.42.93.245) (Quit: miketaylr)19:01
*** Quits: temp01 (~temp01@unaffiliated/temp01) (Read error: Connection reset by peer)19:04
*** Quits: bot-t (~bot-t@unaffiliated/temp01/bot/bot-t) (Read error: Connection reset by peer)19:04
*** Joins: michaeldeol (~michaelde@ip68-106-223-180.oc.oc.cox.net)19:04
*** Joins: temp01 (~temp01@unaffiliated/temp01)19:05
*** Joins: bot-t (~bot-t@unaffiliated/temp01/bot/bot-t)19:07
*** Joins: miketaylr (~miketaylr@24.42.93.245)19:07
*** Quits: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com) (Quit: Colloquy for iPad - http://colloquy.mobi)19:12
*** Joins: danzik17 (~danzik17@ool-4a59e57e.dyn.optonline.net)19:17
*** Joins: Corey (~KB1JWQ@freenode/staff/kb1jwq)19:18
*** Joins: danzik171 (~danzik17@ool-4a59e57e.dyn.optonline.net)19:18
DaveMethvintimmywil: do the unit tests run ok for you on master in ie7?19:28
timmywilthey did earlier today19:29
timmywili'd check now, but i'm in the middle of getText19:29
DaveMethvinthat's fine19:29
DaveMethvinlet me back up a bit and see if it gets better19:29
DaveMethvinseems to be in support.js19:29
timmywilDaveMethvin: so I rewrote barberjoy's test and it seems his is slower19:32
DaveMethvinugh19:32
DaveMethvinhow about using textContent/innerText?19:32
timmywili did it on a selections of divs rather than a selection of the body element19:32
timmywilhaven't gotten that far19:33
DaveMethvinif we could use them i think that would be the fastest way19:33
DaveMethvinbut of couse you need the recursive stuff for xml19:33
DaveMethvinstill19:33
DaveMethvinthere was another implementation in the related ticket19:33
timmywilyea, looking into that one now19:34
DaveMethvindocs#3144 ... took me that long to load the page19:34
bot-t[#3144] Inconsistent cross-browser results from .text() method (open bug) - http://jqbug.com/314419:34
timmywilDaveMethvin: so you like his solution for the bug fix?19:36
DaveMethvini liked it when i played with it, seemed good to me19:37
timmywilcool19:37
DaveMethvinjohn mentioned that we'd had probs with textContent/innerText but i think those probs are gone19:37
DaveMethvinand the biggest pains are from ie, but this makes them better19:38
DaveMethvintimmywil: here's what the new code looks like in ie7: http://i.imgur.com/DZzjc.png19:40
DaveMethvinand for reasonable browsers there's no change19:40
timmywilsounds good to me19:41
timmywili have an idea for improving getText performance too.  I think I can work the bugfix in and still keep the code pretty short.19:42
timmywilwe'll see19:43
*** Joins: bp0 (~bp@unaffiliated/bp0)19:46
*** Quits: michaeldeol (~michaelde@ip68-106-223-180.oc.oc.cox.net) (Read error: Connection reset by peer)19:48
*** Joins: r3vd (~michaelde@ip68-106-223-180.oc.oc.cox.net)19:48
*** Quits: Mannequin (~Mannequin@186.23.142.29) (Ping timeout: 260 seconds)19:50
*** Quits: danzik171 (~danzik17@ool-4a59e57e.dyn.optonline.net) (Read error: Connection reset by peer)19:51
*** Quits: simonleojung (~simonleoj@CPE687f7488464b-CM00137189dc6c.cpe.net.cable.rogers.com) (Quit: simonleojung)19:51
*** Joins: colinclark (~colin@65.93.202.185)19:56
*** Quits: colinclark (~colin@65.93.202.185) (Client Quit)19:57
*** Quits: anfernee (~Adium@210.13.73.174) (Ping timeout: 276 seconds)20:04
timmywilDaveMethvin: according to my tests, the performance loss is minor with the bugfix and barber's getText is noticeably slower20:05
timmywilmy idea didn't pan out20:05
DaveMethvintimmywil: which one is barber's gettext?20:06
timmywilthe PR20:06
timmywilwith nextSibling20:06
DaveMethvink found it20:06
DaveMethvinmaybe this one instead? http://bugs.jquery.com/ticket/3144#comment:220:07
timmywilyea, that's the bugfix i'm talking about20:07
timmywilperformance loss is minor20:07
DaveMethvinhmm i would have expected it to be faster if anything20:08
DaveMethvinaltho ... maybe it should not query the text properties twice and keep them in a local var20:08
DaveMethvinpoor browser prolly does what we do, traverse the tree and collect the strings :)20:09
*** Quits: kswedberg (~kswedberg@c-71-205-160-80.hsd1.mi.comcast.net) (Quit: Leaving.)20:10
timmywilDaveMethvin: ok, my test was bad20:13
timmywilbut wow20:13
timmywilusing textContent is a boost20:13
DaveMethvinhey, C code has got to be faster than recursive javascript20:14
timmywilcertainly20:14
timmywilbarber's is better than 164, but the bugfix is faster than barber's20:15
timmywil:P20:15
DaveMethvinhope it's not too much bigger20:16
DaveMethvinbut also it's good that it fixes some issues with .text() on IE20:16
*** Joins: Mannequin (~Mannequin@186.23.142.29)20:22
*** Quits: Rwhitbeck (~Rwhitbeck@cpe-67-253-202-235.rochester.res.rr.com) (Quit: Computer has gone to sleep.)20:45
timmywilDaveMethvin: need to sleep.  I'll pick it up again tomorrow.20:47
timmywilthe bugfix causes some problems in manipulation20:47
DaveMethvinyeah, i'm tired too and my patches aren't behaving20:47
timmywilI think it's because innerText allows newlines20:47
DaveMethvinamazing how tough these things can be20:47
timmywilbut i'll look into it more tomorrow20:47
DaveMethvinok, well maybe tomorrow, gnite20:48
timmywilhmm, btw, it's only a problem in IE with appending elements for some reason20:50
*** Quits: miketaylr (~miketaylr@24.42.93.245) (Quit: miketaylr)21:06
*** Quits: DaveMethvin (~DaveMethv@pool-173-69-166-27.bltmmd.fios.verizon.net) (Quit: Leaving.)21:07
*** Joins: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com)21:11
*** Quits: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com) (Remote host closed the connection)21:11
*** Joins: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com)21:12
*** Joins: anfernee (~Adium@210.13.73.174)21:26
*** Quits: danzik17 (~danzik17@ool-4a59e57e.dyn.optonline.net) (Quit: Leaving)21:27
*** Joins: jrburke (~jrburke@S01060016cbc69317.vc.shawcable.net)21:31
*** Quits: kborchers (~kborchers@d53-64-212-190.nap.wideopenwest.com) (Quit: Colloquy for iPad - http://colloquy.mobi)21:42
*** Quits: timmywil (~timmywil@host-68-169-154-67.WISOLT2.epbfi.com) ()21:53
*** Quits: anfernee (~Adium@210.13.73.174) (Quit: Leaving.)22:15
*** Quits: wesbos (~wesbos@24-246-54-116.cable.teksavvy.com) (Quit: Leaving...)22:24
*** Joins: eddiemonge (~eTiger13@c-24-6-243-111.hsd1.ca.comcast.net)22:30
*** Quits: socialhapy (~socialhap@46-38-167-162.static.cloud-ips.co.uk) (Ping timeout: 258 seconds)22:34
*** Quits: jrburke (~jrburke@S01060016cbc69317.vc.shawcable.net) (Quit: jrburke)22:43
*** Quits: Jakey (~Jacke@unaffiliated/vienzzion) (Ping timeout: 260 seconds)22:52
*** Joins: Jakey (~Jacke@li192-128.members.linode.com)22:54
*** Quits: Mannequin (~Mannequin@186.23.142.29) (Ping timeout: 260 seconds)23:11
*** Quits: eddiemonge (~eTiger13@c-24-6-243-111.hsd1.ca.comcast.net) (Quit: Leaving...)23:34
*** Joins: Mannequin (~Mannequin@186.23.142.29)23:37
*** Joins: Frippe (~Frippe@unaffiliated/frippe)23:38
*** Joins: beevi7 (~manuel.bi@tim.7val.com)23:52
*** Joins: mokush (~quassel@188.24.43.235)23:54

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