[09:00:53] arschmitz fnagel jasperdegroot jperrault jzaefferer mikesherov rxaviers tj_vantoll [09:00:58] hello [09:01:00] Hello [09:01:00] ‘allo! [09:01:01] Hi [09:01:07] yo [09:01:07] hi all [09:02:06] It looks like today will be a short meeting. [09:02:21] hey [09:02:32] arschmitz: Anything that needs to be discussed for classes or button today? [09:02:45] i have a couple things [09:03:08] First i mentioned this to jzaefferer and he already said he was ok with it but want to mention here [09:03:22] changing how position works in button [09:03:38] to not use a class for if you want it at beginning or end only if its top or bottom [09:03:50] and just changing the widget to append/prepend accordingly [09:04:08] this is how most frameworks do it [09:04:16] and it makes a lot more sense semanticly [09:04:32] We discussed this in -dev the other day, right? [09:04:37] yeah [09:04:38] This is what I was confused about at first? [09:04:52] Ok, yeah, I'm good with that change (after understanding what you were saying). [09:05:06] I think it makes a lot of sense [09:05:48] so unless anyone has an objection im going to make that change [09:05:55] still sounds good to me [09:06:26] awesome [09:07:03] also semi related to buttons iv been working with rending perf tests for chassis on buttons [09:07:15] and the suite does a lot of metrics around js too [09:07:28] so i think we should look into that side of it for ui and mobile as well [09:07:49] http://104.236.81.132:5984//css-perf/_design/site/index.html#/page-select [09:08:01] there is no js on those pages but you can see the metric's it looks at [09:08:30] we could use this both for regression testing to make sure we dont in advertantly effect things with expensive event handlers or things like that [09:08:32] Let's wait for it to be used in Chassis for a while, in case there are any kinks that need to be ironed out over time. [09:08:47] But setting this up sounds good. [09:08:51] scott_gonzalez: well many of the kinks woudl likely be different [09:09:05] since we will be looking at different things [09:09:32] but yeah as far as the general setup it will be very similar [09:09:41] so probably best to wait until that part is ironed out [09:10:24] the reson i mention here is i wanted to see if in general we are interested because ill keep it in mind as i set up things for chassis [09:10:36] to make it as re-usable for both as possible [09:10:58] yeah [09:11:06] I think it's good to have eventually. [09:11:25] ok [09:11:52] Anything else for classes or button? [09:12:02] sort of [09:12:24] i went through the learn article last night and i wonder if we want to either drop or expand the section on custom widgets [09:12:45] i think it either needs a lot more detail or we should just skip it and focus on the option [09:12:51] It’s open for improvements for sure :) [09:13:11] Let's discuss that in the PR. [09:13:16] ok [09:13:39] thats all then im updating classes pr right now [09:13:47] great [09:13:51] waiting for feedback on a few comments before i completely finish [09:14:02] The datepicker/calendar split has finally landed in the datepicker branch! [09:14:08] Woo! [09:14:14] I'm sure fnagel was super happy to merge that. [09:14:14] Yeaah! [09:14:14] \o/ [09:14:54] Very happy, indeed! [09:15:10] Merged to more PRs for datepicker as well [09:15:14] *two [09:15:33] Unit Test run in Phantom now [09:15:43] Yeah, not worrying about rebasing again means that fnagel is able to work on things faster :-) [09:16:03] but its awesome to rebase between branches [09:16:14] its even better between branches between projects! [09:16:47] hehe [09:17:35] jzaefferer rxaviers jasperdegroot: Anything that needs to be discussed about the theme update? [09:17:38] scott_gonzalez: what does that mean exactly? [09:17:45] not worrying about rebasing again [09:17:55] do we go back to merging master into datepicker? [09:18:12] because when I tried to rebase datepicker against master, it immediately failed with some conflict [09:18:26] I've updated https://github.com/jquery/download.jqueryui.com/pull/248 to reflect the new jzaefferer updates (https://github.com/jquery/jquery-ui/pull/1420) [09:18:47] rxaviers: we should merge the Globalize Update PR soon, too [09:19:02] jzaefferer: I meant that fnagel had to keep rebasing his branch with edits to commits, so he didn't want to have more commits in front which might conflict from the edits. [09:19:04] re: theme update - I think all that needs to be done is https://github.com/jquery/download.jqueryui.com/issues/249 [09:19:07] yeap, I must add it back to my list now [09:19:09] jzaefferer: it removes a step instead of master->datepicker->calendar its just master->datepicker [09:19:26] I can do a rebase of datepicker on master and resolve the conflicts. [09:19:58] scott_gonzalez: okay; do you have rerere enabled? Recording the resolutions should make that a lot more effective [09:20:08] No, rerere scares me. [09:20:12] scott_gonzalez: That would be a great help, thanks! [09:20:15] oh [09:20:22] well [09:20:25] jasperdegroot, has created 249, which is a feature request for better UI [09:20:37] if you want to do that despite it... [09:21:00] right now, let's focus on getting 248 landed [09:21:52] rxaviers: I'll review 248 later [09:21:58] I don't think we need to discuss it here [09:22:05] ok for me [09:22:57] o [09:22:58] ok [09:23:08] I'll be sending out a doodle today for the RTL meeting. [09:23:18] I'm going to send dates for the first week of February. [09:23:46] We've got a decent list of people, so hopefully we'll have a good turn out and figure out a path forward. [09:24:14] And that brings us to our last item. [09:24:27] jzaefferer just landed the menu change to use ui-state-active instead of ui-state-focus. [09:25:52] Anyone have anything else to discuss? [09:25:55] Which looks stupid until the theme update lands... [09:26:19] pep... [09:26:34] I've asked for some help on intern in #PEP, but it hasn't been forthcoming. [09:26:43] arschmitz: when you rebase classes, you'll likely get a conflict in tooltip, need to deal with the added ui-widget-shadow class [09:27:06] jzaefferer: thats fine [09:27:11] i really dont care about conflicts [09:27:33] jzaefferer: they don't bother me ill resolve it [09:27:48] its easier when you know what the change is about :-) [09:27:51] jperrault: I'll ping snover. [09:28:28] jzaefferer: thats why we have commit history :-) [09:28:30] jperrault: I've just joined #pep, too; I haven't used intern, but might be able to help anyway [09:28:48] k. we can take that to #PEP then. [09:29:10] looks like you already got your answer [09:29:39] see it, thx [09:29:59] I think that's all for today then [09:30:09] Thanks everyone. See you back in -dev. [09:30:12] k. Bye guys!