[09:07:02] hi there [09:07:07] topics ? [09:07:46] I'm updating the doc [09:08:17] k [09:08:52] (did I mention you rock Rick?) [09:10:08] I try [09:10:10] thanks [09:10:12] hahaha [09:10:16] Ok, who's here? [09:10:22] Jerry Seinfeld agenda! What's the deal with attr(foo)? WHATS THE DEAL WITH THAT? [09:10:24] * jaubourg raises his hand [09:10:34] here [09:10:43] ding [09:10:45] Hey mikesherov [09:10:49] hey ajpiano [09:10:53] woah, april 9 beta [09:10:54] sup [09:11:09] I'll so been in domnican republic at that time :/ [09:11:11] Ok, I updated the doc with Dave's notes from the ML [09:11:12] * timmywil hi [09:11:18] hi timmywil! [09:11:36] please open and turn to page 0 [09:11:39] rwaldron: :) [09:11:40] ...index [09:11:42] :D [09:12:03] one sec, I'm updating the triage link [09:12:08] what's this about attr? [09:12:21] timmywil: you broke the interweb!!!!1111 [09:12:24] looks like I dont have to [09:12:28] eeeeeek [09:12:32] ok, 1.7.2 [09:12:39] it rocks [09:12:41] next? [09:12:44] you rock [09:12:57] this is the issue [09:12:58] http://bugs.jquery.com/ticket/11516 [09:12:58] dave said he couldn't reproduce. where the ticket at [09:13:00] yeah, I mean, 1.7.2 is good, but only until we get more reports of attr being busted [09:13:05] timmywil see above [09:13:14] mike: shush you [09:13:17] :P [09:13:23] oh input value? [09:13:28] I suspect the ticket was filed by the same person [09:13:32] anonymous [09:13:45] yes, it was anonymous [09:13:45] I don't see a test case [09:13:51] there isnt [09:14:06] http://bugs.jquery.com/ticket/11518 [09:14:13] we have an email addy at least [09:14:17] don't make me start on missing test cases [09:14:19] ziabzu@… [09:14:23] I've set to pending [09:14:35] jaubourg yeah I think you laid down the law [09:14:41] i tried being halpful on that one [09:14:47] Ooh, you boilerplated him! [09:14:50] maybe wasn;t a good idea to do so [09:15:17] timmywil dont sweat that ticke [09:15:18] ticket [09:15:28] and mixed with autocomplete? [09:15:32] I have a feeling the entire test suite would be crippled if it was legit [09:15:41] amitrite? [09:15:43] most probably [09:15:47] tests are awesome [09:15:54] could be some document type issue [09:15:54] except for the ones they don't cover [09:15:59] could be anything really [09:16:08] I think it might be quirks mode [09:16:20] mikesherov attr is used a whole lot [09:16:26] I'm JK [09:16:33] mike: indeed... biggest thing being xhtml... didn't we talk about testing for it or did we give up btw? [09:16:35] duh [09:16:37] we currently have a hook for value that returns the property. If the value property is set to an object, it will return an object. The same is not true for id, so that wouldn't make sense. [09:16:43] well, i mean.. it's kind of true [09:17:00] timmywil let's wait it out [09:17:08] guy is probably setting the property to an object [09:17:20] I'm willing to bet OP comes back with "i cant reproduce" [09:17:24] case closed [09:17:28] sure [09:17:39] kk [09:17:43] ok, nextttt [09:17:49] let's bikeshed this git thing [09:17:55] BLUE! [09:18:00] mikesherov +1 [09:18:07] but it has to be "sky blue" [09:18:39] hmmm, I like royal blue the best, but in all seriousness, someone pick the answer that requires the least work [09:18:49] and I'll blindly follow [09:18:58] jaubourg can you close this [09:18:59] https://github.com/jquery/jquery/pull/716/files [09:18:59] ★ Pull request on jquery by BigBlueHat (32m, 26s ago): added Shiv check so show/hide work with Modernizr [09:19:04] or timmywil [09:19:09] I'm gonna fucking scream [09:19:40] No, but jquery should anticipate other libraries existing.... it should also anticipate Object.prototype meddling [09:19:50] :-P [09:20:16] no kidding [09:20:20] god [09:20:24] jQuery sucks sooooo much [09:20:29] Rick: how come you stll don't have commit rights? [09:20:36] why doesnt guess what I'm thinking and just do that [09:20:41] jaubourg the hell if I know [09:20:44] wait, what doesn't he like? It should go to block if the stylesheet says block [09:20:47] the hell if I know too [09:21:04] do we have someone in the room with enough priviledges to give rick's some? [09:21:31] I mean, everytime I see you comment, I'm like "why the f can't he close this?" [09:21:35] no, see, what happens is the defaultDisplay creates a sandboxed iframe without modernizr [09:21:45] ah, it's shiv'd with js [09:21:47] ? [09:22:08] jaubourg: i think john and dave are the only ones who can do it [09:22:21] defaultDisplay has no clue about modernizr... it relies on creating that element in an iframe and returning the style if it can, and block if it cant [09:22:34] ajpiano: k, thanks [09:22:40] and hi \o/ [09:23:21] jaubourg I'm flattered that you'd think of me :) [09:23:25] ok, I've stopped confusing myself [09:23:26] but it's a bunch of nonsense, isn't "issues with other libs" on our permanent wontfix list? [09:23:43] mikesherov you're right but it's such a corner case to begin with [09:24:16] mikesherov yes, exactly [09:24:22] rwaldron: are you kidding? I don't even know why I got commit priviledges when you didn't [09:24:25] right, it has to be something that is already display none for some reason, or disconnected node, or some shit that trigggers defaultDisplay [09:24:43] mikesherov you got it sir [09:24:45] NEXT [09:24:50] I'm looking at this pull request and I'm wondering if it's not deeper than what it seems? [09:24:57] so basically, you have to be doing something dumb to begin with [09:25:08] like, won't we get weird behaviour for html5 elements in old IEs? [09:25:17] jaubourg this is like, my area of expertise *barf* [09:25:17] YES [09:25:24] we will [09:25:33] and so, fuck you! [09:25:34] yeah, thought so [09:25:46] hm. [09:25:51] mike? [09:25:52] we could shim the iframe [09:25:56] NO [09:25:59] it only gets created once [09:26:07] what if I want my articles to be inline [09:26:12] ugh, yeah mikesherov thank you for stopping me [09:26:15] FU modernizr. next [09:26:18] jk [09:26:27] Then don't call show on them [09:26:39] I dunno [09:26:43] like honestly, you have a billion ways to do this [09:26:58] the whole getting HTML5 elements working on non HTML5 browser business always baffled me [09:26:58] just call hide before show, or show before hide [09:27:09] there are a dozen workarounds [09:27:20] this patch is wrong [09:27:21] mikesherov: we should somehow document those then [09:27:22] "canvas,video"? [09:27:35] when would you ever use these in oldIE? [09:27:38] ...hint... [09:27:40] never. [09:27:40] that [09:27:45] that's the whole point [09:27:50] audio? [09:27:52] nope [09:27:52] HTML5shiv is just that [09:27:53] a shiv [09:28:04] in that it's like being stabbed in the face [09:28:11] agreed. [09:28:22] like that time someone tried to shim web workers [09:28:24] there are html5 elements that get used in IE with display: block. His problem is a problem, but I don't think it's worth the code to fix it. [09:28:31] that gave me lots to laugh at for weeks [09:28:36] no, but's it's deeper [09:28:40] its a big edge case [09:28:44] yep [09:28:50] he has default style of none on article [09:29:09] right [09:29:14] that's what causes jQuery to say, oh shit, I gotta make a whole iframe to cover some bullshit [09:29:17] if he used a class - non issue [09:29:33] because then the iframe crap never happens [09:29:43] exactly [09:29:51] people are going to do it, we should document it, but we shouldn't fix it [09:29:51] jQuery can make a node, get the display and move on [09:30:05] +1 documenting [09:30:09] this is a clear case of "you're doing it wrong" [09:30:14] but yeah, let's document [09:30:22] or browser vendors can finally listen to us and give us a way to directly query the default styles they use [09:30:24] and end the behaviour once and for all [09:30:32] I can see how this does not make sense to the user. all he had to do was #somearticle { display: none; }. so documenting makes sense [09:30:38] mikesherov, I'll see you in 2025 [09:30:38] it's a clear case of "you're doing it wrong... given how things are implemented" so to document [09:30:40] ;) [09:30:46] timmywil: false [09:30:55] he needs to make All articles display:none [09:30:56] mikesherov: false what [09:30:59] jaubourg yes [09:31:09] not just a specific element [09:31:10] oh sure [09:31:11] are you trying to be kind to make up for your brutal PR comment? [09:31:12] :D [09:31:17] errr no [09:31:17] w/e [09:31:21] sure [09:31:22] i mean ticket comment [09:31:25] anyway, we all agree [09:31:29] yes [09:31:34] nice work everyone [09:31:35] document, and move on [09:31:40] rwaldron? Who? me? [09:31:42] ok, back to 1.8 [09:31:51] jaubourg yes, was just teasing [09:31:55] oh [09:32:09] In favor of 1.8 feature branch: vote now [09:32:13] +1 [09:32:23] +1 [09:32:30] well, the ticket was like " test case? it's easy to reproduce test case? it's easy to reproduce" [09:32:32] had to stop [09:32:37] +1 [09:32:55] I gotta run gents, it's been real. [09:33:02] mikesherov thanks for input! [09:33:06] later [09:33:09] thanks for leadering! [09:33:12] cya mike [09:33:18] ok, 1.8 on master: vote now [09:33:30] -1 [09:33:31] ehhh, that might take more time [09:33:38] +1 (proxy for jzaefferer) [09:33:57] +1 (proxy for rworth) [09:34:13] I think rworth said -1 [09:34:17] did he? [09:34:30] he stated he expected 1.8pre to be master no? [09:34:44] sure, if there is no 1.7.3pre. did we decide that? [09:34:53] are we having a 1.7.3? [09:34:55] that's the thing I'm tacky about [09:35:01] i think we'd be imprudent to not leave open the possibility of a 1.7.3 [09:35:03] ruh-roh [09:35:09] ajpiano ftw [09:35:13] we don't know what shit could hit the fan [09:35:19] and again [09:35:20] and I also like to be able to develop a feature in a branch without getting bothered with too early integration problems [09:35:26] then I don't see the harm in keeping 1.8 off of master for now [09:35:37] +1000 [09:35:44] we can always merge, amirite? [09:35:48] yep [09:36:02] timmywil is that +1 for feature? [09:36:05] ajpiano same q [09:36:16] yea [09:36:31] i'm in favour of the jzaefferer approach [09:36:49] it contradicts your previous statement, doesn't it? [09:36:53] it does [09:37:06] ajpiano: yea, not sure what you're for then [09:37:12] the jets [09:37:17] except that [09:37:35] the jqueryui approach leaves open the possibility for 1.7.3, does it not? [09:37:45] yes and no [09:37:51] 1.7.3 would have to be done in a branch [09:38:00] (the 1.7 branch) [09:38:08] definitely not for that [09:38:19] does not compute for me either [09:38:22] so, let's get a complete picture of what UI does, in case it helps [09:38:28] oh hai! [09:38:40] once a branch seems stable, like 1.7, we create a 17-stable branch [09:38:47] and call master 1.8 [09:38:57] then we always commit everything to master [09:39:18] and if something merges clean (and isn't risky/outside of scope) for 1.7, we merge from master to 17-stable [09:39:42] yes, this is what I assumed [09:39:43] ok [09:39:51] I'm for the github approach [09:39:57] master is the stable [09:40:01] features on a branch [09:40:09] merge when proven stable [09:40:11] agreed [09:40:12] once in a blue moon we have to do an isolated change in 17-stable, whether it didn't merge clean from master or whether it is specific to only that code branch [09:40:14] I'm all for 1.x branches but right before we go for a 1.x+1 if you see what I mean [09:40:15] tag for release [09:40:48] I'm more like rick myself [09:41:03] so, we tag 1.7.x releases from 17-stable, and we tag 1.8 preview releases (right up through alpha and beta and rc) from master [09:41:13] question is to know who will code features for 1.8 and know their opinions, these are the ones who matter [09:41:19] * jaubourg pokes gnarf [09:41:37] I'll be doing most of my work on the sizzle repo before it goes to jQuery [09:41:41] thanks jaubourg, I pinged gnarf earlier, hopefully he shows [09:41:48] we also have feature branches, to keep master clean - only contained what is confirmed to be in the next major release already [09:42:05] rworth thats essentially what we want to do [09:42:09] and hi richard btw [09:42:11] keep master clean [09:42:13] ;) [09:42:16] hey :) [09:42:17] work in a branch [09:42:29] land when ready [09:42:46] alright then, sounds like core team has a consensus [09:42:48] we just don't branch 1.7 yet in case we did something terribly wrong [09:43:03] seems like it [09:43:33] k, I gotta fly too [09:44:04] cya all around guys [09:44:31] later [09:45:24] ok, i think that's all [09:45:26] thanks everyone [09:45:34] ajpiano can you close the gates? [21:20:33] -verne.freenode.net- [freenode-info] if you're at a conference and other people are having trouble connecting, please mention it to staff: http://freenode.net/faq.shtml#gettinghelp