Attendees: jacine, mortendk, webchick, jerryitt, MediaDoneRight, aspilicious, Bojhan, ericduran, timplunkett, Crell, Gumpo, jhood, cosmicdreams, jbeach, davidwatson [3:59 PM] hey guys :) [4:00 PM] jacine: hey [4:00 PM] OMG it's the fabulous JACINE [4:00 PM] wassup!? [4:00 PM] webchick: haha :) [4:02 PM] So… looking at all the issues tagged in this sprint: http://drupal.org/project/issues/search/drupal?status[]=Open&version[]=8.x&issue_tags_op=and&issue_tags=html5,+sprint [4:02 PM] hey jacine [4:02 PM] Is there anything in there that people need help with? [4:02 PM] hi jerryitt :) [4:02 PM] aspilicious: hey [4:03 PM] aspilicious: do you know what happened with this one? http://drupal.org/node/1360994 [4:03 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 9 IRC mentions [4:03 PM] yay toolbar.tpl.php [4:03 PM] ok. today is the day. how does this typically work. look at issue queue, assign yourself an issue, get to work? [4:04 PM] MediaDoneRight: yep, look at the stuff in the sprint first, if nothing in there tickles your fancy go for anything tagged html5 [4:04 PM] sprint list is here: http://drupal.org/project/issues/search/drupal?status[]=Open&version[]=8.x&issue_tags_op=and&issue_tags=html5,+sprint [4:04 PM] jacine now [4:04 PM] nop* [4:04 PM] aspilicious: so weird. [4:04 PM] aspilicious: should we get this contextual mofo outtahere this week ;) [4:04 PM] aspilicious: is there anything in there you need help with? [4:04 PM] mortendk, probably [4:04 PM] jacine, sweet. thx. I'm looking for something easiest to start... [4:05 PM] MediaDoneRight: sure :) [4:05 PM] jacine, I'm busy with ds this week [4:05 PM] aspilicious: oh fun :) [4:05 PM] and the gf wants attention [4:05 PM] LOL [4:05 PM] that happens :) [4:05 PM] aspilicious: he he just tell her … oooh wait … mnah dont do that ;) [4:05 PM] What do we need to do to finish up what is in this sprint? [4:06 PM] Or at least go forward with other issues if some of these are stalled. [4:06 PM] Bojhan tunes in, anything UI up for review? [4:06 PM] Is there anything you guys are working on that you would like to get more attention? [4:06 PM] Bojhan: Hey!! :) Not that I know of…. [4:07 PM] phew im just trying to get up to speed [4:07 PM] jacine, srry I can't help this week :) [4:07 PM] aspilicious: No problem :) [4:07 PM] aspilicious: Have fun with DS and your gf :D :D [4:08 PM] ericduran: / davereid Do you think you two could pair up on http://drupal.org/node/1174640 ? [4:08 PM] http://drupal.org/node/1174640 => Add new HTML5 FAPI element: number => Drupal core, forms system, normal, needs work, 76 comments, 14 IRC mentions [4:08 PM] What is DS? To me that is a handheld gaming system but I'm sure I'm wrong. :D [4:08 PM] Display Suite :) [4:08 PM] hahaha [4:08 PM] Oh, ha. ;) [4:08 PM] http://drupal.org/node/1183042 looks like it just needs a quick review of the change notice at http://drupal.org/node/1430892 [4:08 PM] http://drupal.org/node/1183042 => Change notice: Add WAI-ARIA roles to Core blocks => Drupal core, markup, critical, needs review, 81 comments, 8 IRC mentions [4:08 PM] http://drupal.org/node/1430892 => WAI-ARIA roles added to core blocks => 0 comments, 1 IRC mention [4:08 PM] mortendk taking notes for an upcoming talk about DS ;) [4:09 PM] Looks like everett is the best person to do so, probably? [4:09 PM] http://drupal.org/node/1183250 also just needs a change notice. It's already committed (yay!) [4:09 PM] webchick: yep probably [4:10 PM] http://drupal.org/node/1183250 => Change notification for: Add a theme_datetime() function to consistently theme dates and datetimes => Drupal core, theme system, critical, active, 214 comments, 33 IRC mentions [4:10 PM] webchick: I plan on reviewing the ARIA change notice. I looked at it earlier, but need to double-check it against the patch. [4:10 PM] webchick: yep [4:10 PM] Oh is meeting time [4:10 PM] I totally missed that [4:11 PM] ericduran: hey :) [4:11 PM] HEY [4:11 PM] ericduran: oi [4:11 PM] hi [4:12 PM] From the looks of things it appears we should just leave the issues that are in the current sprint and not add any more right now. What do you guys think? [4:12 PM] the number issue can be tagged novice, it just needs some clean up [4:13 PM] ericduran: oh rly? can we add another element then? [4:14 PM] jacine: the email might be good [4:14 PM] ericduran: perfect :) thanks! [4:14 PM] I uploaded a patch last week that should be pretty close [4:14 PM] ericduran++ [4:14 PM] I think I just missed the css with the patch [4:14 PM] but that should be it, /me looks [4:16 PM] jacine tagged it. [4:16 PM] I totally lied, I must had upload a patch in my head because there's nothing in the issue queue [4:16 PM] I just commented on it I guess [4:16 PM] LOL [4:18 PM] jacine is feeling sick today. [4:18 PM] hey timplunkett, are you satisfied with sun's response to this: http://drupal.org/node/1221718 [4:18 PM] http://drupal.org/node/1221718 => Refactor block-admin-display-form.tpl.php => Drupal core, block.module, normal, needs review, 24 comments, 6 IRC mentions [4:19 PM] I *think* you are :) [4:19 PM] jacine: yes [4:19 PM] timplunkett: ok, great thanks :) [4:20 PM] jacine: i didn't rtbc because i had almost the identical patch written myself, although in retrospect that sounds like a GREAT reason to rtbc :) [4:20 PM] timplunkett: haha :) It all happened so fast. [4:23 PM] jacine marked it RTBC [4:23 PM] jacine: http://drupal.org/node/1382350#comment-5540810 - Has anyone from HTML5 weight in on this thread? [4:23 PM] http://drupal.org/node/1382350 => [meta] Theme/render system problems => Drupal core, theme system, normal, active, 21 comments, 8 IRC mentions [4:23 PM] It may be useful to offer feedback there. [4:23 PM] Crell creeps back to the shadows. [4:24 PM] Crell: Not really, but it would be helpful. [4:24 PM] Crell puts that on jacine's radar and runs to his next meeting. [4:24 PM] Crell++ [4:25 PM] Hey everybody [4:25 PM] We're relatively new to all this but have some very experienced HTML5 developers [4:25 PM] Gumpo: yay! [4:26 PM] Hey Gumpo :) [4:26 PM] welcome [4:26 PM] Unfortunately I'm not one of them but I'll be asking them to participate in one way or another [4:26 PM] :) [4:26 PM] thanks [4:26 PM] my gosh ... some of these threads are really deep [4:26 PM] One of my devs has been writing a base theme for d7 in HTML5 [4:26 PM] MediaDoneRight just sayin' [4:27 PM] I'll maybe ask for some feedback on any issues he had along the way [4:27 PM] jhood gets off conference call, reads the scrollback and starts taking notes [4:27 PM] as he's a drupal beginner but is getting to love it [4:27 PM] Gumpo: that's great! [4:27 PM] back! [4:27 PM] MediaDoneRight: yes, HTML5 is trick because it touches a lot of different aspects of the system: form API, theme system, CSS, markup, themes, etc. [4:27 PM] MediaDoneRight: yeah, we know all about it. We need people to jump and do reviews to get some of these issues back on track. [4:32 PM] mortendk: you gonna review this one this week? http://drupal.org/node/1216978 [4:32 PM] http://drupal.org/node/1216978 => Clean up the CSS for Contextual module => Drupal core, contextual.module, normal, needs review, 97 comments, 18 IRC mentions [4:32 PM] jacine: ai ai captain' [4:32 PM] mortendk: ok awesome :) [4:33 PM] need to get that fucker outta my system [4:33 PM] yeah, i thought it was ready. [4:33 PM] jacine: so did i ;) [4:34 PM] I have a patch nearly there for dashboard [4:34 PM] http://drupal.org/node/1216980 [4:34 PM] http://drupal.org/node/1216980 => Clean up the CSS for Dashboard module => Drupal core, dashboard.module, normal, needs work, 12 comments, 2 IRC mentions [4:34 PM] webchick: can you get someone to help with the upgrade issue here: http://drupal.org/node/1360994#comment-5505104 [4:34 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 10 IRC mentions [4:34 PM] cosmicdreams: cool! [4:34 PM] Haven't posted it yet, but will tonight if I can clean it up [4:35 PM] dashboard is somewhat confusing because the javascript is binded to the id naming of a few of the divs [4:35 PM] I've cleaned that up in my sandbox and I'm just looking for more to do with it. [4:35 PM] yeah… :( [4:35 PM] that's great to hear [4:35 PM] cosmicdreams++ [4:35 PM] I still use the primary #dashboard id [4:36 PM] but I'm trying to keep the number of ids used to just that one [4:36 PM] I think is we wanted an easy win we can go for http://drupal.org/node/1138258 [4:36 PM] cosmicdreams: it's fine [4:36 PM] http://drupal.org/node/1138258 => Drop textarea.js in favor of CSS3 resize => Drupal core, forms system, normal, needs work, 25 comments, 2 IRC mentions [4:36 PM] ericduran: awesome [4:36 PM] ericduran: oh, haven't seen that one. there is another similar issue that i've seen. [4:37 PM] the patch shouldn't be that bad [4:37 PM] ericduran: reading it [4:37 PM] jacine: you can skip to #24 [4:38 PM] jacine: everything else is from a different approach [4:38 PM] ericduran: would it require the js to support IE8? [4:38 PM] cosmicdreams: not on that issue, that issue is about ripping it out [4:39 PM] cosmicdreams: if you want ie8 support we can do a contrib module or something [4:39 PM] ok....but aren't we supporting IE8 in D8 core? [4:39 PM] ericduran: so IE8 and 9 get no more resizing? [4:39 PM] yea that's the idea [4:39 PM] lol I guess it won't be as simple then [4:40 PM] ericduran: I dunno about that TBH. [4:40 PM] the idea is if an ie8 user really wants to support it, there's 100 other ways to do it, core can use the simple version by default [4:40 PM] ericduran: That doesn't sound very good. I suppose testing will show if that property (or something close) is supported [4:40 PM] jacine: IE 9 doesn't support css resize? [4:40 PM] jacine: lol [4:40 PM] sorry was distracted. checking. [4:40 PM] http://www.quirksmode.org/css/contents.html [4:40 PM] As noted in one of those various issues, "if the browser doesn't support it, so-sorry, no-resize-for-you!" does seem to be the HTML5-ish convention for how to handle such things. [4:40 PM] I didn't test it manually but ^^ says it's not supported [4:41 PM] jacine: http://drupal.org/node/1360994#comment-5505104 <— sorry, I don't understand that? The behaviour described there is correct. [4:41 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 11 IRC mentions [4:41 PM] jacine: Or, maybe not correct, but "as-designed" [4:41 PM] webchick: upgrades are supposed to use bartik? [4:41 PM] jacine: People with really well-branded sites want a really well-branded maintenance page. [4:41 PM] jacine: it's supposed to use the front-end theme, yes [4:41 PM] AFAIK? [4:41 PM] http://en.wiktionary.org/wiki/as_far_as_I_know [4:41 PM] botsmack! [4:41 PM] Yea, but not-resizing in IE is not really a feature blocker or anything crazy, is just a simple you can't make your text box bigger [4:41 PM] so helpful, Druplicon [4:41 PM] All right, all right, I'll chill. Sheesh, some people can't take a joke. [4:42 PM] webchick:talking about update.php - i always see seven. [4:42 PM] Oh, on update.php. Hm. [4:42 PM] jacine: do we have something like Modernizr in core? [4:42 PM] Ok, let me take a closer look. [4:42 PM] webchick: the issue is happening on a major upgrade because of the theme hook change. [4:42 PM] Drupal 7 > 8 [4:43 PM] cosmicdreams: no we do not, but this sounds like something that needs feature detection to me [4:43 PM] ericduran: Right, that's my point. We put the onus on the browser to support resize, don't waste time with JS, and go have a beer. [4:43 PM] yep [4:43 PM] I'll leave that up to you guys. [4:43 PM] Crell doesn't drink but is all in favor of development practices that increase available time for its consumption. [4:43 PM] hihi [4:43 PM] same [4:43 PM] jacine: ericduran: yea detect if the browser supports and if so use native implementation, if not use js implementation. [4:43 PM] hehe [4:43 PM] I think it would be annoying to entire content in a content management system, and not be able to resize the textarea. [4:43 PM] ericduran, jacine, hi hi [4:43 PM] hi jbeach :) [4:44 PM] cosmicdreams: simpler, just use css, No detection [4:44 PM] jacine: Only because you're used to Drupal, which has had it since Drupal 5 or so. :-) [4:44 PM] jbeach: HEYYYYYY [4:44 PM] Crell: haha, yeah probably :) [4:44 PM] jacine: How would you feel about a contrib module called ie9 that provides the shivs that matter to absolutely no one but IE? :-) [4:44 PM] jacine: FYI, the default input is pretty big [4:44 PM] I would like to tackle some form of feature detection though. [4:44 PM] I mean, the only other legacy browser we even think about is Firefox 3.6... and we're not thinking about it. [4:44 PM] also lets remember some of these things can be done in contrib :-p [4:44 PM] ericduran: would the css approach work for resizable textareas? [4:45 PM] Crell: if you guys are all good with a CSS solution, so am I [4:45 PM] cosmicdreams: On everything except IE8 and maybe IE9. [4:45 PM] jacine: It's more a question of "can we leave out a shiv for this one". [4:45 PM] I'm okay with contrib modules dealing with it [4:45 PM] ericduran: Crell: well that's what I'm talking about [4:45 PM] I propose yes. [4:45 PM] we can name our module "sucka" and it can have ie8 support [4:45 PM] yes [4:45 PM] Done! [4:45 PM] LOL [4:45 PM] I will tag it for this sprint [4:45 PM] jbeach: it was nice to meet you FINALY [4:46 PM] How many people will be using IE8 when D8 is a stable release? [4:46 PM] ericduran, agreed [4:46 PM] Gumpo: hopefully a lot less than now [4:46 PM] Gumpo: quite a few (at least in the first few years) [4:46 PM] Gumpo, too many [4:46 PM] http://drupal.org/node/1138258#comment-5572232 [4:46 PM] http://drupal.org/node/1138258 => Drop textarea.js in favor of CSS3 resize => Drupal core, forms system, normal, needs work, 25 comments, 3 IRC mentions [4:46 PM] MS could solve that if they gave Windows 8 away for free [4:47 PM] It's all about priorities [4:47 PM] If people can't upgrade their browsers, I can't see them upgrading the OS :D [4:47 PM] Hm. About http://drupal.org/node/1360994 [4:47 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 13 IRC mentions [4:47 PM] I get that hook_maintenance_page is confusing [4:47 PM] Gumpo: IE8 is the maximum browser available for WinXP users [4:47 PM] but should we maybe just make that entire template page--update.tpl.php? [4:48 PM] or something that doesn't require peopel to learn new things? [4:48 PM] Yeah, it's a travesty [4:48 PM] webchick, love that [4:48 PM] webchick: well it is maintenance… so you would get maintenance--update.tpl.php as an option [4:48 PM] cosmicdreams: No, Firefox 10 and Chrome are. :-) [4:48 PM] Gumpo: in this case they do need to upgrade their OS to upgrade their browser [4:48 PM] jacine: Oh, good point. [4:48 PM] all the maintenance styles are grouped together [4:48 PM] Crell: THATABOY [4:48 PM] the hook is too ideosyncratic [4:48 PM] page—maintenance would also work [4:48 PM] And probably their hardware too [4:48 PM] jbeach: you don't like maintenance.tpl.php [4:48 PM] webchick: no that is the problem [4:48 PM] but there is a fair chance "maintenance" collides with an existing system path [4:48 PM] webchick: we CANNOT do that. [4:48 PM] yep [4:49 PM] jacine, I don't like the hook function [4:49 PM] jbeach: which one? the one we are trying to change it to? [4:49 PM] Ok, just making sure this approach made sense before I get someone to debug why it's not working. :) [4:49 PM] Jumping into the fray in the middle, so apologies if this has already been covered, but what is the consensus on progressive enhancement as a development guideline? For embellishment functionality that we can handle with CSS in modern browsers, is shoehorning in JS support worth the bloat? [4:49 PM] jacine, *reads backscroll before she puts foot in mouth* [4:49 PM] It'd be nice to really make D8 as streamlined as possible [4:50 PM] davidwatson: no js support [4:50 PM] Gumpo: That's the goal. [4:50 PM] davidwatson: at least that's what we're going for. Or trying to go for* [4:50 PM] The issue makes sense, because page.tpl.php no longer includes the HTML stuff [4:50 PM] so it's confusing that one [4:50 PM] jbeach: the issue is that page-maintenance.tpl.php implies a relation to page, and 1/2 of the stuff in page.tpl.php is not available ad will never be. Its not an extension of page.tpl.php in any way and the code is sloppy as hell and mimics all the preprocess functions. It's totally stupid. [4:50 PM] "page" template does, and the other does not [4:50 PM] right [4:51 PM] So maybe leave that decision to the developers, just create a module to add support for IE8 [4:51 PM] there are a few use cases but they are all minimal, so a different template makes sense. [4:51 PM] I would go for maintenance-mode.tpl.php personally but that's a bikeshed issue that is irrelevant to the bug. :D [4:51 PM] jacine, right, my $0.02 is that there's a lot of code duplication associated with the maintenance page that should be eliminated [4:51 PM] webchick: it's not just used for maintenance mode though: http://drupal.org/node/1360994#comment-5373076 [4:51 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 14 IRC mentions [4:52 PM] ericduran: That's what I figured; the matter of feature detection threw me for a bit, but that's probably just me being addled from teaching. :] Thanks for the clarification. [4:52 PM] I cede the details to folks that are invested here :) [4:52 PM] jbeach: ok, that is exactly what we are doing, and we are taking "page" out of the hook and template name. [4:52 PM] this comment sums it all up: http://drupal.org/node/1360994#comment-5373076 [4:52 PM] http://drupal.org/node/1360994 => Change hook_maintenance_page to hook_maintenance => Drupal core, theme system, normal, needs work, 15 comments, 15 IRC mentions [4:52 PM] davidwatson: yea, feature detection is hard to do for css. And we really don't want to do browser sniffer [4:53 PM] gr8 [4:53 PM] you-no-got-a-database-connection.tpl.php [4:53 PM] ericduran: Agreed 100%. [4:54 PM] there will always be maintenance--update.tpl.php, maintenance--install.tpl.php, maintenance--offline.tpl.php [4:54 PM] oh touche. [4:54 PM] that's there already [4:54 PM] webchick did not know that. damn drupal people are smart. ;D they think of everything! [4:54 PM] haha :D [4:55 PM] timplunkett prefers the name you-no-got-a-database-connection.tpl.php [4:55 PM] LOL [4:55 PM] Crell votes for got-no-database-now-its-gone.tpl.php [4:55 PM] smells like a bikeshed [4:56 PM] where-u-go-database-sniff.tpl.php [4:56 PM] LOL [4:56 PM] ok ok I'll stop now. ;) [4:56 PM] jacine: where's the list of issues targetted for the sprint? [4:56 PM] Does anyone have anything else they want to talk about? [4:56 PM] html5 sprint? [4:56 PM] html5 sprint is http://bit.ly/html5-sprint [4:56 PM] cosmicdreams: ^^ [4:57 PM] i should probably put that in the IRC room message. [4:57 PM] jacine: http://dvcs.w3.org/hg/webcomponents/raw-file/tip/explainer/index.html some really awesome stuff here [4:57 PM] i always forget how to do that though [4:57 PM] jacine: sorry if I've already sent you that link [4:57 PM] I've seen that before :) [4:57 PM] it is cool [4:57 PM] cosmicdreams: Oh, thanks, I had forgotten where that link went. [4:57 PM] there's a lot of these popping up [4:57 PM] jacine: but a few years down the line (once this is a standard) it will change the way we handle smart componenets [4:57 PM] No idea if it will ever see the light of day, of course. [4:58 PM] It's nice to see, TBH. [4:58 PM] ericduran thanks shadow dom should stay out of drupal [4:58 PM] Shadow DOM is already in Chrome Canary [4:58 PM] that's more for javascript plugins developer [4:58 PM] thinsk* [4:58 PM] Yeah. It means people defining HTML and CSS are finally realizing that 90% of pages today are built dynamically. :-) [4:58 PM] damn it. lol [4:59 PM] ericduran: yes, but if we can couple it with JSON calls we wouldn't have to do so much in the theme layer [4:59 PM] cosmicdreams: that's not what theme layer is for [4:59 PM] well ajax callbacks i mean [4:59 PM] is different [4:59 PM] like a form with dynamic children [5:00 PM] cosmicdreams: yes, I get that. But that belongs in a jquery plugin for example [5:00 PM] cosmicdreams: well is just my opinion :-p [5:00 PM] lol [5:00 PM] ericduran stop enforcing his opinion on other people now [5:00 PM] ericduran: somewhat recent work as an example http://www.targetportraits.com/offers/birthday-club [5:02 PM] There's a lot going on still with standards [5:04 PM] Crell is excited about EventSource recently. Although how the server-side is supposed to work I'm still unclear. [5:04 PM] Crell: link? [5:05 PM] http://groups.drupal.org/node/192173#comment-682573 - And following comments. [5:05 PM] http://groups.drupal.org/node/192173 => WebSockets => 22 comments, 1 IRC mention [5:06 PM] jacine: ok, I left a comment on that issue with what I *think* needs to happen. [5:06 PM] jacine: I have to duck out and go work on some distro stuff but I'll try and get some eyeballs on it later [5:06 PM] webchick: thank you thank you :D [5:06 PM] np! :)