[00:00:05] yes yes, I just, for some reason, felt typing out "brandon (black)" was easier than non-tab-completing his irc nick... [00:00:08] go with it [00:03:25] kaldari: http://meyerweb.com/eric/tools/css/reset/ [00:05:01] jdlrobson: kaity you two coming to the VE discussion? [00:05:04] jdlrobson: hey you available for the meeting now? [00:06:54] (03CR) 10Jdlrobson: [C: 04-1] "On a tablet if I flick from portrait to landscape, when I go back to portrait the viewport is broken. I haven't quite worked out why.." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107519 (owner: 10Kaldari) [00:14:53] rmoen: did you just lose video from the office? [00:15:23] jdlrobson: head's up i, maybe we, lost video from the office [00:15:29] i think rmoen may be frozen [00:16:14] all better! and now in hd :) [00:16:16] mhurd: tfinc: want to try installing the iOS app enterprise-signed build via test flight? [00:16:25] brion: ya! [00:16:43] mhurd: ok https://www.testflightapp.com/install/ee820b2d209d83848bd39225161885dc-ODg1MTI2MQ/ <- should only let you if i've enabled ya and you're logged in [00:17:23] (03CR) 10Kaldari: Fixing CSS resets for tables in reset.css and hacks.less (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107494 (owner: 10Kaldari) [00:17:36] in theory this should let us just sign people up through testflightapp and not have to deal with the device whitelist anymore (as long as it's in-house people) [00:18:01] i'm just hoping i pushed all the right knobs and buttons :D [00:19:46] brion: WOOHOO!!!! it worked :) [00:20:01] huzzahhhhhhhh [00:20:03] brion: just ran it on new retina ipad mini [00:20:53] sweet [00:21:10] brion: thanks! [00:21:27] ok i'll push it to a few folks like the designers and we'll set up a new invite group probably later [00:21:39] this one's kinda labeled under 'wikimedia commons' still which i notice is probably wrong ;) [00:21:48] hehe [00:29:39] mhurd: ok next build we'll maybe put a proper icon on it and try to get more internal testers :D [00:30:11] brion: ya! [00:31:09] ok i've got a pile of devices to charge, bbiab [00:32:44] (03PS5) 10Kaldari: Fixing CSS reset for border-spacing. Making table headers bold. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107494 [00:37:45] jdlrobson: Let me know if this is better: https://gerrit.wikimedia.org/r/#/c/107494/ [00:40:23] Hi I just came across https://github.com/wikimedia/apps-ios-wikipedia and looks pretty good so far (issues I had with the old/original one seem to fixed). I wouldn't mind making some contributions. I saw in the readme it mentioned about pull requests through GH, is the bot active? or is it better for me to register and use Gerrit? [01:00:04] ScrimpyCat: If you can use gerrit, it's probably better [01:00:54] ok [01:02:11] I can't remember if code review comments are pushed back onto the pull request on github [01:08:16] (03PS1) 10Kaldari: Adding revisions to BeforeSpecialMobileDiffDisplay hook params [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107754 [01:15:59] (03PS1) 10Dr0ptp4kt: Remove deprecated mozmarket.js call. [apps/firefox/wikipedia] - 10https://gerrit.wikimedia.org/r/107755 [01:34:32] (03PS1) 10Jdlrobson: Fix broken messages on account creation / login for watchlist [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107756 [01:38:48] legoktm: Good idea on the Thanks. I rewrote it with half as much code :) https://gerrit.wikimedia.org/r/#/c/107633 [01:48:40] (03CR) 10Kaldari: [C: 032] Regression: fix hamburger and left menu icon alignment [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107625 (owner: 10JGonera) [01:49:14] (03Merged) 10jenkins-bot: Regression: fix hamburger and left menu icon alignment [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107625 (owner: 10JGonera) [02:09:27] (03PS1) 10Reedy: Add COPYING [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107767 [02:09:48] (03PS1) 10Reedy: Add COPYING [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/107768 [04:55:46] hey vbamba [04:55:52] around? [07:16:33] jdlrobson: hey, when can i expect to see my stuff deployed on en.wikipedia.org? [07:17:44] hey pcc1 can't talk for long but let me summarize quickly now :) so a branch gets cut every thursday so as of tomorrow MediaWiki.org should have sections open by default in settings on alpha [07:17:53] then next monday that will be on wikipedia.org [07:18:24] i forget when things filter out to other projects exactly but they will do so between those dates [07:19:21] hope that helps - will be around tomorrow if you want to get more details. Next step beta! :-) [07:19:26] jdlrobson: cool, thanks [07:19:49] goodnight! (or morning depending on where you are!) [07:22:43] (03PS1) 10Yuvipanda: Simplify sections to be not nested [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107808 [07:39:31] vbamba: around? [13:54:27] (03CR) 10Theopolisme: "Maybe this would be better displayed on the preview page? (Perhaps above the license text...)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/106865 (owner: 10Legoktm) [17:09:41] brion: hey! [17:21:23] (03CR) 10Yurik: Add COPYING (031 comment) [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/107768 (owner: 10Reedy) [17:45:13] mornin [17:47:30] ihello brion [18:03:08] brion: code review? [18:09:02] (03CR) 10Kaldari: Fixing CSS reset for border-spacing. Making table headers bold. (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107494 (owner: 10Kaldari) [18:14:39] moment, been caught up in mp4-land ;) [18:14:42] ah [18:14:47] hello vbamba [18:19:03] (03PS6) 10Kaldari: Removing CSS reset for border-spacing. Making table headers bold. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107494 [18:43:34] (03PS1) 10Kaldari: Story 1565: Tweak infobox styling for tablet [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107882 [18:44:30] (03Abandoned) 10Kaldari: Story 1565: Improve Infobox styling for tablet on beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/107519 (owner: 10Kaldari) [18:50:08] * YuviPanda pounces on mhurd [18:50:14] brion: YuviPanda: heya! sry didn't realize irc hadn;'t connected [18:50:15] just as a general principle :) [18:50:20] hehe [18:50:25] mhurd: hah! [18:50:28] mhurd: what client are you using? [18:50:38] colloquy [18:51:15] mhurd: ah, hmm. [18:51:20] mhurd: tried Textual or LimeChat? [18:51:42] dfoy, yurik: faidon said he can't make it (too late), and mark is away. brandon accepted. faidon's going to be here next week. i think we should still talk with brandon to get on the same page, and then next week have another discusssion. [18:51:53] YuviPanda: no but i think i'll give them a peek later :) [18:52:00] mhurd: please do :) [18:52:15] dr0ptp4kt, sounds good [18:52:44] YuviPanda: i got distracted refactoring [18:53:02] mhurd: best kinda distracted [18:53:05] I spent yesterday doing that too [18:53:21] evening :) [18:53:51] hello krakjoe [18:53:58] YuviPanda: for sure! some was a bit overdue in the ios code base [18:54:07] mhurd: oh, I saw a couple of pull requests to the iOS app on github [18:54:19] YuviPanda: commons? [18:54:27] mhurd: no, https://github.com/wikimedia/apps-ios-wikipedia/pulls [18:54:34] mhurd: do comment there, and let me know if it is ok to merge. [18:54:38] I'll migrate them to gerrit and merge [18:54:52] if you tell him 'hey now learn this other thing that sucks, unlike gerrit' he'll probably go away :) [18:55:08] err [18:55:11] unlike github I mean [18:56:53] YuviPanda: whoa! so cool. they look pretty good, but i'll have to test 'em a bit later today :) [18:57:09] yurik, what day and time you get here, approx? [18:57:22] mhurd: yeah, but can you leave a comment on those saying 'hey, cool! I will test them later today'? :) [18:57:32] (03CR) 10Brion VIBBER: "There seems to be some sort of problem with internal links with non-ASCII chars in them -- they consistently cause crashes for me when I c" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107626 (owner: 10Yuvipanda) [18:57:32] YuviPanda: ya! [18:58:10] YuviPanda: so we really need better error handling in LeadSectionFetchTask [18:58:14] (03CR) 10Yuvipanda: "Really? They were causing problems for me, and then this patch fixed them. Are you sure this patch is applied? Can you send me a crash rep" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107626 (owner: 10Yuvipanda) [18:58:21] it seems to crash on me like A LOT [18:58:39] pageProperties = new PageProperties(Utils.parseMWDate(result.asObject().optJSONObject("mobileview").optString("lastmodified"))); [18:58:41] as always [18:58:42] brion: it crashed on me on non ascii internal links, but that patch seems to fix it [18:58:46] brion: exactly that line [18:59:07] brion: I prefer it crashing on us now so we can isolate those easier and handle it graciously before we release. [18:59:15] so that line crashing basically means "something somewhere returned a bad response of some kind" [18:59:24] with no way to actually see what was wrong [18:59:43] if i could see the returned API result that might help [18:59:52] lemme try adding a log line [19:00:43] brion: there's already a log line in PageViewFragment, line 250 [19:00:49] i might've added it in a later patch [19:01:13] yeah thats in the next patch [19:01:16] right [19:01:21] should've added it to this one [19:01:26] since that is what I used to test [19:01:34] dr0ptp4kt, 21-29 [19:01:41] it told me invalid title because we were sending it without decoding the URL [19:01:46] yurik thx [19:04:20] agh now it's being super flaky... maybe it's just my bad wifi :P [19:04:33] brion: heh, office liefi [19:05:06] well now this is interesting [19:05:27] YuviPanda: it's crashing inside URLDecoder.decode with a null pointer sexception [19:05:33] wat [19:05:41] brion: goddamit [19:05:45] I know what's wrong there [19:05:46] let me fix [19:05:49] :D [19:06:07] dr0ptp4kt: ok thx [19:08:04] anybody know if juliusz's tdd workshop is streaming? [19:08:12] i can't join the hangout because it's full [19:08:22] (03PS2) 10Yuvipanda: Simplify sections to be not nested [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107808 [19:08:22] brion: there! should be fixed now. [19:08:24] (03PS2) 10Yuvipanda: Handle URLEncoded internal page links properly [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107626 [19:08:27] whee [19:09:47] tfinc: ^^^ [19:10:03] awjr: their working on it [19:10:07] thanks tfinc [19:13:07] (03CR) 10Brion VIBBER: [C: 032 V: 032] "Ok this fixes when there's no namespace. ;)" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/107626 (owner: 10Yuvipanda) [19:16:39] tfinc: i see the stream, but there is no audio [19:16:58] awjr: message chip and let him know [19:17:28] tfinc: i have but he hasn't been responding to me [19:17:42] awjr: i just let him know [19:17:46] thnx [19:19:07] got it tfinc, thanks for your help