[00:21:09] (03CR) 10JGonera: [C: 032] Document $wgMFAnonymousEditing [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110420 (owner: 10Jdlrobson) [00:21:43] (03Merged) 10jenkins-bot: Document $wgMFAnonymousEditing [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110420 (owner: 10Jdlrobson) [00:22:45] (03CR) 10JGonera: [C: 032] Beta Feature: Register mobile skin, offer users the ability to change it [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102866 (owner: 10Jdlrobson) [00:23:22] (03Merged) 10jenkins-bot: Beta Feature: Register mobile skin, offer users the ability to change it [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102866 (owner: 10Jdlrobson) [00:42:27] rmoen|away: you still away? [00:43:04] (03CR) 10JGonera: [C: 04-1] "Breaks titles in editor overlay in stable (they now have underscores instead of spaces)." (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 (owner: 10Jdlrobson) [00:44:49] (03CR) 10JGonera: [C: 032] QA: Allow browser tests in alpha mode [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109451 (owner: 10Jdlrobson) [00:45:28] (03Merged) 10jenkins-bot: QA: Allow browser tests in alpha mode [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109451 (owner: 10Jdlrobson) [00:54:18] (03CR) 10JGonera: [C: 04-1] QA: Add browser tests for ajax pages (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109453 (owner: 10Jdlrobson) [01:02:15] (03CR) 10JGonera: [C: 032] Hygiene: Don't use content_wrapper [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109319 (owner: 10Jdlrobson) [01:03:26] (03Merged) 10jenkins-bot: Hygiene: Don't use content_wrapper [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109319 (owner: 10Jdlrobson) [01:07:05] mhurd: hey! [01:07:10] I've proper internet now! :) [01:11:00] YuviPanda: shouldn't you be sleeping now ? [01:11:07] tfinc: I had slept and woken up [01:11:25] tfinc: I also had an interesting auto ride after, so well woken up :P [01:11:35] interesting good ? [01:11:52] tfinc: interesting interesting :P [01:12:22] tfinc: this entire trip I have paid more for autos than for the intercity buses themselves :) [01:12:46] YuviPanda: where are you heading that is so costly ? [01:13:12] tfinc: I went to Chennai to start the process to pick up my college certificates [01:13:16] tfinc: just came back to Bangalore [01:13:32] tfinc: the entire trip was about 20$ more expensive than it should be, but oh well. [01:13:56] worth it if it gets you the visa [01:14:12] tfinc: heh, true [01:14:39] YuviPanda: you're gonna love me more for what im about to show you. https://trello-attachments.s3.amazonaws.com/52e98a603e6d08a53861025b/52e98cbf8ef5bbc824a0e978/d1245cd998479551615a59042b0cc7e8/Screenshot_2014-01-30_17.16.34.png [01:15:06] YuviPanda: yay! [01:15:10] violetto: I like the header nav better! :) [01:15:20] violetto: were there any other changes? [01:15:20] YuviPanda: https://trello-attachments.s3.amazonaws.com/52e98a603e6d08a53861025b/52e98cbf8ef5bbc824a0e978/5414994afc79021d928e57b0e2b69e28/Screenshot_2014-01-30_17.18.49.png [01:15:49] YuviPanda: took out the bottom action cus we'd have a double if the warnings are short [01:16:01] violetto: hmm, ok! let me take it out as well [01:16:13] in the top nav now there is a call to action "Ignore or review edit" [01:16:16] basically bottom action is brought to top and will be persistent [01:16:32] violetto: makes sense [01:16:37] violetto: I'll adjust code to match [01:16:44] YuviPanda: but sorry, back filled button stay for other workflows taht needs it! [01:16:59] so i guess you love me still the same amount before [01:17:05] violetto: :P :D :) [01:17:08] im just gonna have to deal with it [01:17:58] violetto: me too :P [01:18:19] mhurd: legoktm didn't get the iOS builds that were sent out :( any idea why that might be happening? [01:19:08] hmmm have him ping brion... maybe he's not been added to the acct properly? [01:22:24] violetto: did you add an asset for the 'x' [01:22:56] YuviPanda: i did. but it's not set. I'm awaiting Pau's reply on that. [01:22:57] violetto: also, I think we should make them 144px instead of 150 - to match Android's 48px grid [01:23:16] violetto: where did you add it to? [01:23:27] YuviPanda: 144 is good [01:23:33] violetto: ok! [01:23:49] YuviPanda: https://trello.com/c/NVw38u0P/8-assets [01:24:15] violetto: ah ok! [01:24:24] violetto: yay for svgs :) [01:24:49] violetto: I'll also do an open sans run today, and convert all the things [01:25:11] YuviPanda: perfcet! [01:25:11] violetto: are you ok with using roboto serif for the places we use serif in? [01:25:19] violetto: which is so far just the page title [01:25:28] YuviPanda: that's currently what we're using right [01:25:28] violetto: or do you want to pick out another font? [01:25:30] violetto: ya [01:25:36] YuviPanda: that's great then [01:25:40] violetto: ok [01:25:44] cus i thought it looked good [01:25:54] violetto: oh can you attach the latest to the card? [01:26:22] violetto: remember we tried a build that did 'When you start scrolling search results keyboard should hide (is weird, May's playing with it now) ' [01:26:33] violetto: it still feels weird, should I just cross it off and abandon the patch? [01:27:43] YuviPanda: yeah, we've added more search results though right? [01:27:49] violetto: yeah, it's 12 now [01:28:57] violetto: awesome!!!! [01:29:30] YuviPanda: let me look at it again. the keyboard should hide though [01:29:44] violetto: the keyboard hides when you tap, but not when you scroll [01:29:44] violetto: YuviPanda: see ya tomorrow! [01:30:12] violetto: since if you start scrolling, and say item 1 is hidden, and then we show keyboard, it just comes right back, since there is now enough room to display everything [01:30:19] mhurd: cya! happy climbing! [01:30:27] violetto, you around? [01:31:08] YuviPanda: :) [01:32:05] YuviPanda: okay we'll just let it stay and see how things go [01:32:08] dr0ptp4kt: i am! [01:32:10] violetto: yeah [01:32:16] violetto: I crossed it off for now [01:32:20] violetto: wanna come over here for a mini demo? [01:32:21] YuviPanda: koo! [01:32:33] dr0ptp4kt: /going [02:08:58] Krinkle, I was kind of wrong, the elements do exist, but they're not attached, anyway I know how to fix this [02:09:41] Krinkle, do I have to specify the surface in MobileViewTarget constructor or could I do it later? [02:31:05] (03PS1) 10JGonera: Fix VE error with latest VE master [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110498 [02:49:20] hi violetto [02:50:41] YuviPanda: hey! [02:50:50] violetto: hey! [02:50:56] violetto: I can't point out why it feels odd [02:50:58] violetto: but it does! [02:52:15] violetto: do you want links to the screenshot again? [02:52:25] YuviPanda: my android batts died, can you screenshot an article? [02:52:29] sure [02:52:45] violetto: https://www.dropbox.com/s/asuk30tdlo67wgq/2014-01-31%2002.24.09.png and https://www.dropbox.com/s/2bi1j7uxkgjwns7/2014-01-31%2002.22.54.png [02:52:52] some fonts have really bad kerning that's why it's hard to find really good fonts to use [02:52:56] violetto: an article is coming up [02:53:33] violetto: [02:53:35] violetto: https://www.dropbox.com/s/w8wxkywo4w177ko/2014-01-31%2002.53.11.png [02:54:05] it's definitely the kerning [02:54:13] it's rendering fine on the article but elsewhere not [02:54:15] violetto: right. [02:54:21] violetto: it looks fine in the article, tho? [02:54:28] it looks great on the article [02:54:32] violetto: it is the same font as the article, right? [02:55:31] it's the same yea [02:55:59] violetto: tell me a font that you *know* has good kerning? I'll put it on and try it out to see if it is the font or the renderer? [02:56:06] violetto: doesn't have to be open source, is just for testing [02:56:10] the kerning is not as great as helvetica neue im thinking in the history view it's looking bad possibly because it's larger? [02:56:36] if you changed the article copy to the same as what's used on the history view lets see if it has the same problem [02:56:38] violetto: the history view and the sidebar/search stuff is exactly the same size [02:57:02] helvetica neue is a very refined set of font you can try that [02:57:22] history/sidebar vs article copy? [02:57:40] violetto: hmm, I'm not sure. let me check. [02:59:53] violetto: okay, it is 18 [03:00:22] violetto: although it is 18sp, which is a slightly different unit than 16px [03:00:29] I am trying to find a way to convert [03:01:00] YuviPanda: oki [03:03:18] violetto: let's assume for now that it is 18px [03:03:27] violetto: do you want me to make body text 18px or the UI text 16px? [03:03:42] body text 18 first [03:05:18] violetto: ok [03:05:52] violetto: https://www.dropbox.com/s/zz27y1bcpnj6eyx/2014-01-31%2003.05.34.png [03:06:39] YuviPanda: yup! [03:06:45] turns out it's just a bad kerned font [03:06:48] arghhh [03:06:58] bigger the text bigger the kerning problem [03:07:57] violetto: sigh. [03:09:02] YuviPanda: let me investigate this after my presentation tmr [03:09:17] let's see what i can do [03:09:23] violetto: I am setting UI font to Helvetica Neue to verify that it isn't the rendering engine [03:09:30] YuviPanda: okay [03:13:16] violetto: nevermind, I can't find a proper version of Helvetica Neue that'll work on Android. Licensing issues I guess [03:13:16] grr [03:13:26] violetto: so I just stash this patch for now until we find a better font? [03:13:51] YuviPanda: yeah [03:14:24] violetto: hmm, actually, I am going to send up this patch anyway. [03:14:34] violetto: will mean we are reminded constantly of the ugliness :P [03:14:39] YuviPanda: im going to start preparing for my presentation more than i have for tmr, do you need anything from me? i'll be more available after tomorrow [03:14:47] YuviPanda: lol [03:15:00] violetto: nah, that's cool! :) Good luck for the presentation! [03:15:27] YuviPanda: thanks! im going to be presenting wikitrails [03:15:38] violetto: ooooh very nice! :) [03:15:40] (03PS1) 10Yuvipanda: Make the UI use OpenSans too [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110504 [03:15:53] violetto: there, patch uploaded :) [03:16:02] violetto: where are you presenting? [03:17:08] ux brown bag tomorrow at lunch [03:17:14] on 6th floor [03:17:53] violetto: oh, wait, I forgot tomorrow was not saturday [03:17:59] violetto: my days are messed up. ignore me [03:18:10] YuviPanda: hahahah [03:18:27] violetto: :) have fun! I'll consider taking a nap or doing some other cleanup work [03:18:36] violetto: do consider linking me to the presentation when you are done :) [03:20:20] YuviPanda: i will! [03:20:25] violetto: sweet! [03:22:08] YuviPanda: http://techcrunch.com/2014/01/30/facebook-paper/ [03:22:23] violetto: yeah, saw that. too bad I can't try it out (not being in the US and iOS) [03:22:50] YuviPanda: the interactions are so bad ass [03:23:03] violetto: I could guess :D [03:23:11] violetto: you should see if there's things we could steal [03:23:54] YuviPanda: see that's their first release [03:23:56] ;P [03:24:10] violetto: :) we still have about 2 months for our first release [03:24:11] wouldn't you want to be on tech crunch on first release! [03:24:19] violetto: we should stop saying 'MVP'. It is nowhere near an MVP [03:24:36] what would it be [03:24:45] plus I personally don't think the MVP paradigm fits us - we have a lot of ideas / concrete data that we know works, just needs implementation [03:24:49] violetto: 'first release'? :P [03:24:54] violetto: '1.0'? [03:25:02] YuviPanda: :) [03:25:18] YuviPanda: i know right [03:25:50] violetto: yeah. MVPs are useful in startupland when you have a super limited amount of time and have no clue at all as to what works and what doesn't [03:25:52] we dont wanna be like… oh yea that's just mvp we still have a lot of stuff to tweak [03:25:56] not true for us [03:25:58] violetto: yeah, that too [03:26:14] violetto: an MVP is not going to get pushed out to 20 million people :P [03:26:45] YuviPanda: i know right! [03:26:55] YuviPanda: https://www.facebook.com/paper [03:26:59] hot damn! [03:27:26] YuviPanda: heck i wanna be so excited i want to make a video like that [03:28:12] violetto: nice re-imagining of how their app could look [03:28:31] violetto: the current fb app is just 'oh we can reuse all our designs from our mobile website, and just make it a little faster!' [03:28:34] violetto: agreed :) [03:28:38] YuviPanda: can't let yahoo and facebook highlight 2014 [03:28:42] it's time for wikipedia [03:29:13] violetto: +1 :) [03:29:19] YuviPanda: fb app is kinda shitty [03:29:25] yahoo's been doing super awesome things [03:29:27] violetto: 'kinda' is putting it lightly, IMO [03:29:41] violetto: hmm, interesting. I've zero yahoo apps on my phone and also use zero yahoo products. [03:29:57] YuviPanda: airbnb is also doing great things [03:30:35] violetto: well, in a way so are we (Wikipedia as a whole) [03:30:39] violetto: just growing pains :) [03:31:09] violetto: I hope betafeatures will speed up now that zz_prtksxna is around to implement things [03:31:26] violetto: also I am immensely happier to be having this conversation on IRC than fb chat or hangout :P [03:33:38] YuviPanda: :) [09:28:56] (03PS2) 10Yuvipanda: Make the UI use OpenSans too [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110504 [10:07:03] YuviPanda: What exactly is bugging you with the fonts? [10:07:12] prtksxna: I don't know :P [10:07:19] lol [10:07:24] prtksxna: violetto (May) doesn't like using Roboto [10:07:35] prtksxna: and Open Sans feels a bit weird on the UI text (looks fine for body text) [10:07:54] YuviPanda: And you guys are sure you want to use a sans serif? [10:08:09] prtksxna: I suppose so? [10:08:42] I see, Roboto Slab could've been an option otherwise [10:09:01] prtksxna: yeah, I don't think it was even considered [10:09:06] prtksxna: we could use any Open Source Font though [10:09:23] YuviPanda: oh, ok [10:09:33] prtksxna: I liked Arimo [10:09:38] YuviPanda: Hm. [10:09:41] * prtksxna google [10:09:43] prtksxna: but I should still continue not having any opinion on fonts [10:09:54] prtksxna: so I am just going to let violetto pick it and not say much about it [10:10:11] YuviPanda: That sounds like a good plan too :) [10:10:24] prtksxna: heh, yeah! [10:35:33] (03PS1) 10Yuvipanda: Turn off AutoSuggest/AutoCorrect for Captcha [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110529 [13:45:25] (03PS2) 10Yuvipanda: Fix minor jshint warning [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110436 [13:45:27] (03PS2) 10Yuvipanda: Turn off AutoSuggest/AutoCorrect for Captcha [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110529 [17:51:04] hey brion [17:51:11] rawr [17:52:06] YuviPanda: https://gerrit.wikimedia.org/r/#/c/110504/ <- y'all think this is ugly so .... why ? :D [17:52:40] brion: you'll note that there are no dependencies on it :P I wanted to have it up, and I think even merging it is ok [17:52:53] also isn't there a way to specify global styles that would set the fonts? [17:53:02] brion: absolutely none, sadly [17:53:03] instead of customizing every view... [17:53:06] d'oh [17:53:27] brion: I'd rather have this code in now, since we can always tweak the font we want to use later (one line code change) [17:53:42] (03CR) 10Brion VIBBER: [C: 032 V: 032] Fix minor jshint warning [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110436 (owner: 10Yuvipanda) [17:53:47] YuviPanda: ah true [17:53:51] (03PS3) 10Yuvipanda: Make the UI use OpenSans too [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110504 [17:54:13] (03CR) 10Brion VIBBER: [C: 032 V: 032] Turn off AutoSuggest/AutoCorrect for Captcha [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110529 (owner: 10Yuvipanda) [17:56:59] (03CR) 10Brion VIBBER: [C: 032 V: 032] "Most importantly this gives us a single place we can now change the font for things, so that's good. :D" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110504 (owner: 10Yuvipanda) [17:58:42] (03CR) 10Jdlrobson: [C: 032] "Probably should put bug number in comment but I guess git blame can be used." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110498 (owner: 10JGonera) [17:59:22] (03Merged) 10jenkins-bot: Fix VE error with latest VE master [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110498 (owner: 10JGonera) [18:00:12] (03PS2) 10Jdlrobson: QA: Tests for story 1628 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109807 [18:00:14] (03PS6) 10Jdlrobson: QA: Define expected behaviour for VisualEditor overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 [18:00:16] (03PS8) 10Jdlrobson: QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 [18:00:18] (03PS3) 10Jdlrobson: Story 1628: Put toolbar in header of VE overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110064 [18:00:57] (03CR) 10jenkins-bot: [V: 04-1] QA: Tests for story 1628 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109807 (owner: 10Jdlrobson) [18:01:53] (03CR) 10Brion VIBBER: "testing this shortly..." [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/110471 (owner: 10Mhurd) [18:02:02] first i need more coffee [18:02:31] (03CR) 10jenkins-bot: [V: 04-1] Story 1628: Put toolbar in header of VE overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110064 (owner: 10Jdlrobson) [18:06:26] oh so that's why my computer keeps going to sleep [18:06:31] cat knocked out the power supply again [18:06:38] * brion cat == home terrorist [18:07:02] if his demands of kibbles in his bowl are not met, laptop power will be depleted again! [18:07:51] brion: hehe :P [18:14:51] jdlrobson: on https://gerrit.wikimedia.org/r/#/c/109011/8/tests/browser/features/step_definitions/keep_going_steps.rb change "keep_going_element.when_present" to "keep_going_element.when_present(10)"... [18:15:30] jdlrobson: I'm getting occasional timeouts, e.g.: timed out after 5 seconds, waiting for {:class=>"overlay-bottom", :tag_name=>"div"} to become present (Watir::Wait::TimeoutError) [18:15:30] features/keepgoing.feature:12:in `Then I see the KeepGoing drawer prompting me to continue editing' [18:16:35] mm okay sure [18:17:01] chrismcmahon: doing now [18:17:04] kenanwang_, we had around 1.5% of Blackberries in June [18:17:11] kenanwang_: i know you mentioned you wont be in the office today, is there still a way folks like jdlrobson can walk you through ready for signoff items? [18:18:00] ok jgonera thanks [18:18:20] awjr i will email our some directions to the team [18:18:24] out* [18:18:28] awesome thanks kenanwang_ [18:18:31] kenanwang_: I have 2 stories waiting for sign-off right now, so let me know how you want to review those [18:18:47] kenanwang_, also, we have to talk briefly about cards 1652 and 1653 for the next iteration (we didn't get to them during the estimation meeting) [18:19:15] ok kaldari [18:19:18] JeffHall: chrismcmahon i know you guys are both super busy, but we have a a bunch of browser tests for MF stuck in code review - any chance you guys can take a look at them soon? [18:19:30] jgonera: yep [18:19:41] awjr, I'll try to review some of them today [18:19:47] cool jgonera [18:20:02] kenanwang_, do you want to do a hangout later today? [18:20:14] i should have some time late this afternoon to lend a hand with code review [18:20:17] can we talk around 1? [18:20:25] awjr: Sure, I can take a look right after lunch [18:20:29] kenanwang_, can we make it 2? [18:20:30] (03PS9) 10Jdlrobson: QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 [18:20:32] ^ chrismcmahon [18:20:36] awesome JeffHall thank you :D [18:20:42] awjr: yes, actually, guess I'll paste my comment above into gerrit jdlrobson [18:20:51] chrismcmahon: :D [18:20:55] chrismcmahon: no need already fixed [18:21:08] (03CR) 10Brion VIBBER: "note: this is being updated, will review when next patchset is in :D" [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/110471 (owner: 10Mhurd) [18:21:25] (03CR) 10Cmcmahon: "hange "keep_going_element.when_present" to "keep_going_element.when_present(10)"..." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 (owner: 10Jdlrobson) [18:21:36] chrismcmahon: i've already done that [18:21:59] jdlrobson: awesome, I think this is really close, let me bring down the lates [18:22:00] t [18:30:49] jdlrobson: wondering why this times out even after 10 seconds, looking: Then I see the KeepGoing drawer prompting me to continue editing # features/step_definitions/keep_going_steps.rb:1 [18:30:49] timed out after 10 seconds, waiting for {:class=>"overlay-bottom", :tag_name=>"div"} to become present (Watir::Wait::TimeoutError) [18:30:49] features/keepgoing.feature:12:in `Then I see the KeepGoing drawer prompting me to continue editing' [18:32:04] chrismcmahon: what is happening in your browser? [18:32:10] does the registration happen correctly? [18:32:19] jdlrobson: that's what I'm trying to figure out [18:33:06] jdlrobson: ah, I see. the new user is not always created properly, I just watched that fail, figuring out why [18:33:09] chrismcmahon: are you using chrome or firefox? [18:33:14] firefox [18:33:19] mmm beta labs issues maybe? [18:33:26] jdlrobson: no, timing or something [18:36:25] "error saving settings" upon clicking beta thingie [18:36:43] ? [18:36:50] screenshot? [18:37:59] (03CR) 10Jdlrobson: Make edit button permissions work on alpha lazy loaded pages (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 (owner: 10Jdlrobson) [18:40:50] jdlrobson: should I ping someone else about https://gerrit.wikimedia.org/r/#/c/109878/, or should I just go ahead and merge it? [18:41:52] YuviPanda: it's on my radar. I will look at it if it still needs review when i get the time, maybe later today. I have no idea if you should merge it or not as I have not looked at the code. [18:42:18] jdlrobson: ok [18:50:55] (03PS10) 10Cmcmahon: QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 (owner: 10Jdlrobson) [18:53:18] (03PS11) 10Cmcmahon: QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 (owner: 10Jdlrobson) [19:00:52] brion: legoktm didn't get the iOS build [19:00:57] brion: any idea what might have happened? [19:01:16] hmm [19:02:50] ah i had to add him in a second place [19:02:52] lemme fix that [19:02:57] brion: also StevenW? [19:03:23] YuviPanda: steven walling? he installed it already [19:03:27] brion: ah, ok! [19:03:28] or another steven w :D [19:03:31] just wanted to double check [19:03:33] ok :D [19:03:42] THE WORLD IS NOT BIG ENOUGH FOR TWO STEVENWS! [19:05:06] (03PS4) 10Jdlrobson: QA: Add browser tests for ajax pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109453 [19:05:08] (03PS5) 10Jdlrobson: Make edit button permissions work on alpha lazy loaded pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 [19:05:10] jdlrobson: JeffHall I added some hygiene to https://gerrit.wikimedia.org/r/#/c/109011/11/tests/browser/features/step_definitions/common_steps.rb . It's not bulletproof but it's better. I'd like to merge the whole thing and see what happens. [19:06:02] (03CR) 10Jdlrobson: QA: Add browser tests for ajax pages (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109453 (owner: 10Jdlrobson) [19:06:52] (03CR) 10jenkins-bot: [V: 04-1] Make edit button permissions work on alpha lazy loaded pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 (owner: 10Jdlrobson) [19:07:02] (03CR) 10Jdlrobson: "Hey Jared. any luck?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104324 (owner: 10Jcf2000) [19:07:55] (03CR) 10Jdlrobson: "Hey Theo let me know if anything is not clear :)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/108539 (owner: 10Theopolisme) [19:11:28] (03CR) 10Cmcmahon: [C: 032] QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 (owner: 10Jdlrobson) [19:12:04] (03Merged) 10jenkins-bot: QA: KeepGoing browser tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109011 (owner: 10Jdlrobson) [19:17:32] (03PS6) 10Jdlrobson: Make edit button permissions work on alpha lazy loaded pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 [19:17:38] (03CR) 10jenkins-bot: [V: 04-1] Make edit button permissions work on alpha lazy loaded pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 (owner: 10Jdlrobson) [19:17:42] chrismcmahon: WOOP :D [19:17:49] chrismcmahon: what was the issue? [19:18:19] jdlrobson: some sort of race conditions doing beta settings. I [19:19:02] jdlrobson: I'm still not sure my changes are 100% bulletproof, let's see what happens in the builds. if we get bogus failures because of settings problems, we can dig deeper. [19:27:37] (03PS5) 10Jdlrobson: QA: Add browser tests for ajax pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109453 [19:27:39] (03PS7) 10Jdlrobson: Make edit button permissions work on alpha lazy loaded pages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109455 [19:29:05] (03PS7) 10Jdlrobson: QA: Define expected behaviour for VisualEditor overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 [19:29:33] (03PS8) 10Jdlrobson: QA: Define expected behaviour for VisualEditor overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 [19:30:16] (03PS7) 10Jdlrobson: Make KeepGoing/toast confirmation run when editing using VE [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/108990 [19:34:41] kenanwang_: I'm wondering if you could do the acceptance meeting with brion instead. [19:34:49] sure [19:35:08] (03PS9) 10Jdlrobson: QA: Define expected behaviour for VisualEditor overlay for bug 59932 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 [19:35:13] (03PS8) 10Jdlrobson: Make KeepGoing/toast confirmation run when editing using VE [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/108990 [19:35:23] (03PS2) 10Jdlrobson: Story 1625: Drop the preview screen [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109779 [19:35:53] ok what time was that again? 12 or 12:30? [19:36:02] brion: 12:00 [19:36:08] ok [19:36:11] works for me :D [19:36:52] brion: kenanwang_ the AF story - we hit all the acceptance criteria, and the rest of the design will be finished up in the visual styling sprint (top nav, icons) [19:37:12] I have two largeish forms to fill before tomorrow :( (Visa + Zurich) [19:40:03] brion: kenanwang_ thanks! [19:43:47] aaaaah i got disoriented swiping on what i thought was the app but was actually a screenshot of the app [19:44:30] brion: hahaha! [19:44:36] brion: https://www.dropbox.com/s/nr1evpummbnlbhq/2014-01-31%2019.32.26.png is the login screen now [19:44:45] brion: the Open Sans seems to be throwing me off a little bit [19:44:52] :) [19:45:01] that kerning just feels wrong doesn't it :D [19:45:48] brion: yeah, it does [19:46:06] we're gonna have to do some kind of font shoot-out aren't we :D [19:46:15] brion: so a hacky way to fix kerning is to insert a space after every character, and set textScaleX on that character to adjust it positively or negatively [19:46:19] brion: FUCKING INSANE, that is [19:46:29] lol [19:46:39] don't do that :D [19:46:45] brion: and that breaks line wrapping, so the workaround is to use ZWNJs instead :P [19:46:53] since you can scale them to be non zero width :P [19:46:59] so they are NZWNJs :P [19:47:08] oh i'm gonna choke on my coffee :D [19:47:19] brion: :D [19:47:23] let's just pick a font we like better in the end ;) [19:47:40] brion: yeah, after eating popcorn at other people's font discussions, I am going to totally stay out of this one [19:47:46] brion: I'll just go with whatever May picks :P [19:47:48] hehe [19:50:41] ehm, I'm fulfilling my promise to fall asleep [19:52:44] brion: I updated the invite to include you [19:52:46] brion: thanks! [19:55:01] (03PS3) 10Kaldari: Story 1621: Fixing VE content area typography/design for mobile [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110122 [19:56:25] rmoen, in VE, on desktop where can I find the code that is triggered when clicking an [edit beta] section link? [19:56:59] jgonera: in ve.init.mw.ViewPageTarget [20:01:34] man i wish i had more SSD space on this MBP [20:01:51] if/when i get a new one i'm maxing out the storage [20:01:56] brion: shit shit shit https://android.googlesource.com/platform%2Fbuild/+/08d410f53249c18d752f56a881ed2335403080d4 [20:01:57] :D [20:01:58] \o/ [20:01:59] :O [20:02:01] O_o [20:02:03] O_O [20:02:05] o_O [20:02:08] :D [20:02:21] hopefully it won't break everything \o/ [20:02:45] brion: yeah [20:02:52] brion: also did you sync up with kenanwang_? [20:02:58] i'm in the hangout [20:03:02] kenanwang_: poke poke [20:03:20] Hey brion and YuviPanda just got out of meeting [20:03:24] ok [20:03:38] brion: ok, I'll try to finish up login. tx [20:03:44] whee [20:04:01] ok brion so I gotta setup my android phone [20:04:11] ok lemme know if i can help [20:04:19] yep [20:04:27] will check back in in 10ish [20:05:05] ok [20:06:45] damn you can get em configured up to 1TB of SSD now [20:08:50] i think i'll wait until they officially support 60Hz UHD displays [20:08:55] and.... i have money for one.... [20:08:56] details [20:11:39] ah the new 15"er has two thunderbolt 2 ports, but the 13"er still has single thunderbolt (non-2) port? [20:11:45] that might get me to stick with the 15" model [20:12:13] ah well another day [20:13:30] jdlrobson: Is this Keepgoing test ready for review, or is it still WIP for now? https://gerrit.wikimedia.org/r/#/c/109436/ [20:15:17] maybe i'll get an external SSD for my VMs, they're the biggest thing eating disk space [20:16:31] or i could get more space with an HDD but ..... I'm so spoiled now by SSD speeds [20:16:44] compiling things on my old HDD mac is just painful [20:19:00] JeffHall: so i'm a little confused about WIP [20:19:05] I'm dong test driven development [20:19:09] so the test is meant to fail [20:19:13] i clean it up in the test that follows it [20:19:20] should i not be using wip for that purpose? [20:21:31] jdlrobson: The @wip is fine with me, just wasn't sure if that was a placeholder meaning that the patch itself was still WIP. So it sounds like you are ready for review on this one, so I can do that now! [20:21:32] rmoen, but that link is not really followed, so I assume there's some jQuery(...).on('click',...) somewhere, right? I can see onViewTabClick but not onEditTabClick, is there something like that somewhere? [20:21:51] thanks JeffHall :) [20:21:55] check it passes in the follow up ;-) [20:25:28] brb [20:29:30] (03CR) 10Jhall: [C: 032] "Jdlrobson clarified that the Scenarios tagged @wip are expected to fail for the time being (since he is using a TDD process), so based on " [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 (owner: 10Jdlrobson) [20:30:06] (03Merged) 10jenkins-bot: QA: Define expected behaviour for VisualEditor overlay for bug 59932 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109436 (owner: 10Jdlrobson) [20:30:15] (03PS1) 10Yuvipanda: Add Login Activity, link from Left Nav [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/110622 [20:32:11] kenanwang_: still poking at the device? should i stand by or go grab a sandwich :D [20:32:57] I just got it up [20:33:10] man the mingle design change is weirding me out [20:33:14] Is it the link that yuvi sent? [20:34:31] Or the one from the email yesterday? [20:34:37] kenanwang_: the one from my email yesterday [20:34:48] 'This sprint's Wikipedia app builds (iOS & Android)' [20:35:30] K [20:35:39] rmoen, it's puzzling for me how VE figures out where to scroll to after clicking an edit section link... I replaced this.section = currentUri.query.vesection || null; with this.section = 2 and it didn't change a thing... [20:37:57] brion: off to sleep. I apparently have to write my own persistant cookie handler, which is fucked up. will figure stuff out over the weekend. [20:38:01] brion: night! [20:38:09] oh fun :D [20:38:12] nighty night [20:39:24] Got it installed brion [20:39:44] woohoo [20:39:59] I'll take a look at them and let you know if i have questions [20:40:05] great :D [20:40:53] Haha I think I need help [20:41:09] heh ok [20:41:15] What's the best way to trigger the captchad and abuse filters [20:42:14] kenanwang_: captcha: add a URL [20:42:33] kenanwang_: abusefilter: ..... hmm easiest on testwiki is to type 'poop' but that's probably not a real filter on real wiki :D [20:42:56] Haha ok [20:43:19] i can show you a demo over hangout, i've got a simulator set up with a testwiki page in history that i can use to trigger [20:44:24] Is test wiki an option on this app? [20:44:40] If you want you can invite me to a hangout [20:44:43] right now [20:46:06] (03PS1) 10Jdlrobson: Load Tablet styles if portrait or landscape meets threshold [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110625 [20:47:07] kenanwang_: can we sign some stuff off? [20:47:53] kenanwang_: card: https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1630 result: http://mobile-reportcard.wmflabs.org/graphs/thanks-daily [20:48:29] jdlrobson: on with Brion doing a sign off [20:49:01] But Ill ping you when I'm off with brion [20:54:47] kaldari: so event logging is broken on enwiki thanks [20:55:28] i just thanked you but it's not showing up :( [21:00:56] (03CR) 10Awjrichards: [C: 04-1] "The browser tests for this fail for me:" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110079 (owner: 10Jdlrobson) [21:03:19] awjr: they pass for me.. [21:03:23] whee [21:03:36] jdlrobson: weird, it looks like it's a messaging issue [21:03:50] maybe that patch just needs a rebase [21:03:56] are you running latest version of ConfirmEdit ? [21:04:03] oh maybe not [21:04:10] I think we ran into a similar problem before [21:04:16] lemme try again [21:05:08] is confirmedit where that message comes from? [21:05:38] jdlrobson: i signed off on that card [21:05:47] thanks kenanwang_ [21:05:52] you're still looking into why there are no thanks? [21:05:56] yeah jdlrobson you were right, after updating ConfirmEdit seems to pass, sorry for the red herring [21:06:05] yeh apparently there is still an issue there - some data validation issues in the Echo extension [21:06:11] ori is looking into it [21:06:21] ori / nuria [21:06:31] (03CR) 10Awjrichards: [C: 032] "I take it back - I was using an old version of ConfrimEdit to test against. After updating ConfirmEdit, tests pass fine for me. Code looks" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110079 (owner: 10Jdlrobson) [21:07:05] jdlrobson: i moved #1609 to ready for signoff [21:07:23] awjr: it's parent still needs merging.. [21:07:26] https://gerrit.wikimedia.org/r/#/c/110078/1 [21:07:33] doh [21:07:35] *its [21:09:28] code looks good, lemme just test [21:10:12] does @wip have any effect on the tests? [21:10:27] jdlrobson: and i presume standing alone (just testing that patchset) the tests should fail, yes? [21:10:51] correct [21:11:03] (03CR) 10Awjrichards: [C: 032] QA: Add tests for Story 1609 captcha refresh [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110078 (owner: 10Jdlrobson) [21:11:04] JeffHall: chrismcmalunch ^ i think @wip tests are ignored if they fail [21:11:39] (03Merged) 10jenkins-bot: QA: Add tests for Story 1609 captcha refresh [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110078 (owner: 10Jdlrobson) [21:11:43] do i need to configure something locally for that to happen? it doesn't seem to happen automagically [21:12:08] (03Merged) 10jenkins-bot: Story 1609: Push captcha refresh to stable [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110079 (owner: 10Jdlrobson) [21:13:19] awjr: include -w [21:13:36] i think.. [21:13:56] or --tags ~@wip [21:14:18] awjr: chrismcmalunch JeffHall can hopefully confirm [21:14:30] jdlrobson: you are correct, that worked [21:14:45] should probably put something in README about that [21:15:11] and on https://www.mediawiki.org/wiki/Quality_Assurance/Browser_testing/Running_tests [21:19:56] (03CR) 10Awjrichards: [C: 032] QA: Add browser tests for story 1610 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110117 (owner: 10Jdlrobson) [21:20:05] (03CR) 10Awjrichards: [C: 032] Story 1610: Promote already logged in message to stable [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110118 (owner: 10Jdlrobson) [21:20:18] jdlrobson: i also moved #1610 to ready for signoff [21:21:24] (03Merged) 10jenkins-bot: QA: Add browser tests for story 1610 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110117 (owner: 10Jdlrobson) [21:21:38] (03Merged) 10jenkins-bot: Story 1610: Promote already logged in message to stable [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110118 (owner: 10Jdlrobson) [21:25:18] awjr jdlsrobson: Sorry, was away for a moment, but sounds like you resolved the question. Additonal documentation on Cucumber tag usage is here: https://github.com/cucumber/cucumber/wiki/Tags [21:25:35] thanks JeffHall [21:27:23] i added a note about @wip to the Running_tests page on mw.o [21:35:08] mhurd: kenanwang_ brion : great job clearing the sign off column [21:35:29] \o [21:35:32] / [21:35:38] almost lost an arm there [21:35:41] hahaha [21:35:50] brion: i see the translate wiki card is still in doing. are we blocked ? [21:36:51] tfinc: temporarily blocked, i'll see if i can get it moving with nike or siebrand [21:37:17] brion: how are you other cards going? i dont see you with anything else on the board [21:37:23] is the board not up to date ? [21:38:06] i'll grab the login menu if that doesn't get done first, otherwise i'll concentrate on some fixups and polish [21:38:16] we noticed some bugs in the stuff we signed off [21:38:25] (e.g. it works but there's some oddities to still clean up) [21:38:30] i think ken an filed a bug :D [21:38:35] mhurd already has two stories on his plate to finish today [21:38:38] ok [21:38:48] 1 point wont make a huge difference [21:38:53] tfinc: almost done on the one i had to pull out of review [21:39:04] kaldari you around? wanna walk through 1620? [21:39:12] then focusing brions time on review makes sense [21:39:20] ok [21:41:17] mhurd: i'll grab a sandwich then review that :D [21:41:23] brion: k [21:41:38] should have it submitted in a couple minutes [21:41:40] :) [21:41:55] jdlrobson: jgonera kaldari do i need to do anything special to set up VE locally to work with MF? [21:42:01] awjr: many things [21:42:07] documented anywhere, jdlrobson? [21:42:09] installed VisualEditor extension [21:42:17] run git submodule init [21:42:21] and git submodule update there [21:42:27] check out Parsoid extension [21:42:29] run npm start [21:42:41] obviously install VisualEditor in LocalSettings.php [21:42:45] then it should just work in alpha [21:42:48] it's documented all over the place [21:42:53] but that's the quick summary [21:43:13] kk [21:43:15] thanks [21:47:47] (03PS1) 10Cmcmahon: Do not create bogus pages like Selenium_Edit_Test in prod [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110636 [21:48:04] kenanwang_: yes [21:49:22] jdlrobson: jgonera please merge before the day is out: https://gerrit.wikimedia.org/r/110636 . [21:49:45] (03CR) 10Jdlrobson: [C: 032] Do not create bogus pages like Selenium_Edit_Test in prod [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110636 (owner: 10Cmcmahon) [21:49:50] (03PS2) 10Jdlrobson: QA: Do not create bogus pages like Selenium_Edit_Test in prod [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110636 (owner: 10Cmcmahon) [21:49:54] just adding qa prefix ;-) [21:50:00] (03CR) 10Jdlrobson: [C: 032] QA: Do not create bogus pages like Selenium_Edit_Test in prod [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110636 (owner: 10Cmcmahon) [21:50:11] kenanwang_: You should be able to test it on beta labs. Keep in mind though that you won't see the CTA unless you are in the test group, so you may have to create 2 accounts. [21:50:29] kaldari: kaity how should CTAs appear in the new tablet design - look really weird as it stands [21:50:55] Right I'll try now [21:50:55] (03Merged) 10jenkins-bot: QA: Do not create bogus pages like Selenium_Edit_Test in prod [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110636 (owner: 10Cmcmahon) [21:51:24] jdlrobson: Good question. I think we may need to redesign them for tablet [21:51:32] jdlrobson: jgonera we can't be creating bogus articles on enwiki [21:52:05] chrismcmahon: :) [21:52:13] i think that was a mistake on my part sorry bout that :) [21:53:06] kenanwang_: If it doesn't work, I may need to turn on a config var on Beta Labs [21:53:12] checking... [21:53:16] jdlrobson: Yeah, I got the marked-for-deletion email. I think I finally got to the point where we are running no browser tests on prod. almost every test is invasive in some way. [21:54:32] kenanwang_: yeah, looks like a config var is needed... [21:55:14] kenanwang_: gimme a couple minutes... [21:55:15] Hmm worked for me on beta I think [21:55:31] kenanwang_: it may be turned on there already, checking... [21:55:34] jdlrobson: i think im close but im getting an error when it ry to run my local wiki with VE enabled: [21:55:34] Original exception: [6223253b] /testing/core/index.php/Main_Page Exception from line 245 of /mnt/hgfs/testing/core/includes/resourceloader/ResourceLoader.php: ResourceLoader duplicate registration error. Another module has already been registered as oojs [21:55:39] know anythinga bout that? [21:56:05] awjr: Do you have an old version of the code? [21:56:22] James_F: VE, MF, parsoid, and core are all at head of respective masters [21:56:24] awjr: I think that got fixed weeks ago. (Duplicate registration of oojs dependency.) [21:56:33] awjr: Hmm. Regression, then. :-( [21:56:55] :( [21:57:15] kenanwang_: Yeah, looks like it is turned on for the en.wiki version of beta labs :) [21:57:33] according to the configs [21:57:35] awjr: It's trivial to fix (just wrap the dependency registration in an if-not-already-available call), but… [21:57:50] awjr: update core [21:58:06] jdlrobson: core is already up to date [21:58:09] (03PS2) 10Mhurd: Displays abuse filter warnings. Warning links open externally. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/110471 [21:58:13] mmm that's strange [21:58:16] brion: ^ [21:58:23] i don't see why it would be registered twice then.. [21:58:25] awjr: I'll let jdlrobson help instead. :-) [21:58:36] kenanwang_: Did it work in stable mode? [21:58:47] heh [21:59:15] oh unless the error has been cached since i updated [21:59:21] awjr: it's possible [21:59:29] did you ever try the magic RL caching thingymagic [21:59:33] Cool I'll sign off Kaldari [21:59:35] Do we need a task to make sure we turn on in production? [21:59:40] o_O [21:59:41] try an incognito window.. [21:59:52] Also did it hit deployment train this past Thursday? [21:59:55] i am already in incognito :) [22:00:11] kenanwang_: No, it's already turned on, just not loaded in stable. [22:00:29] jdlrobson: working for you with everything at head of master? [22:00:49] kenanwang_: if the patchset i just submitted to brion passes review, the abuse filter card will be ready for sign-off soon. if you want a peek at how it looks on ios i just attached 3 screenshots to https://trello.com/c/NVw38u0P/8-assets [22:02:29] works for me yeh awjr [22:02:46] have you got any other extensions that may be interfereing? [22:03:08] awjr: grep for oojs and see what's registering it [22:03:47] kaldari what do you mean by loaded? [22:03:52] jgonera: you around/ [22:03:53] ? [22:05:00] kenanwang_: I mean the KeepGoing javascript module doesn't get loaded on en.wiki unless you are in stable (currently). As soon as the new code is deployed there, however, it will be loaded regardless (without needed any extra config change). [22:05:06] cool mhurd [22:05:19] kenanwang_: oops, I meant beta, not stable :) [22:05:42] kaity: hey! is may w.f.h. today? [22:05:42] ok and which deployment train was it on? [22:05:50] kaldari^ [22:05:53] kenanwang_: Let me check [22:06:34] kenanwang_, just a sec [22:08:04] kenanwang_: 1.23wmf12 which was deployed to mediawiki.org yesterday, and en.wiki next thursday [22:08:16] sorry jdlrobson got pulled away by dude installing my kitchen sink [22:08:18] thanks kaldari [22:08:23] i found the problem and got it working [22:08:45] i've run into this before - i forgot to link my checkout of VE with my test environment >_< [22:08:50] thanks for the hlep [22:08:54] jdlrobson: James_F ^ [22:10:07] Aha. That'd do it. [22:12:24] awjr do you have a few minutes now? [22:12:33] sure jgonera what's up? [22:12:54] mhurd: no shes in! [22:13:01] kaity: ah! thanks! [22:13:03] awjr, can you join a hangout with Kenan and me? I've just sent you an invite [22:13:08] yeah sec [22:13:20] awjr, we're talking about the upcoming iteration [22:13:26] k [22:14:16] jdlrobson: what pages were you lookin at for CTAs? [22:14:51] kaity: any page in beta on my desktop computer when anon [22:14:53] click the edit button [22:15:00] feels like the cta should shrink with everything else [22:16:28] jdlrobson: yes totally [22:22:34] we can take a look at design ideas monday? [22:28:53] kaity, is 1462 (Releasing the Overlay UI into Stable) ok? [22:29:52] (03CR) 10Brion VIBBER: [C: 032 V: 032] "The 'review edit' bar & icon could use some visual feedback on touch; we can do that later though." [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/110471 (owner: 10Mhurd) [22:30:01] mhurd: coupla notes but it's coming along nicely :D [22:30:37] mhurd: also i find i have a hard time hitting that 'save' button; it may be too small a touch target [22:30:57] jgonera: what about the issue with the type size and zooming in? [22:31:04] did that get resolved? [22:31:24] kaity, yes, it should be fine [22:45:21] kaity: Did you see jdlrobson's question earlier about mobile CTAs on tablet (the blue ones that are typically docked to a page action and stretch the full width of the screen)? They look pretty weird on tablet (since they are extremely wide and short) and we may need a new design for them on tablet. [22:45:51] kenanwang_: ^ [22:46:28] kaldari: kaity: jdlrobson: yes totally [22:46:29] [2:18pm] nuria left the chat room. (Quit: Computer has gone to sleep.) [22:46:29] [2:22pm] kaity: we can take a look at design ideas monday? [22:46:41] probably something more similar to the equivalent CTAs on desktop [22:47:30] kaity, jdlrobson: ah, nevermind, I missed that [22:54:47] kaldari: it looks like the typography update to mobile is a little off [22:55:03] uh oh [22:55:13] kaldari: the line-height seems to be 1.4 em, I think we said 1.6 em, right? [22:55:25] yes, it should be 1.6 [22:55:33] (03PS1) 10Jdlrobson: Make pointer arrow responsive [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110648 [22:55:38] (03PS1) 10Jdlrobson: Avoid flash of styles when loading tablets [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110649 [22:56:05] kenanwang_: i'm just grabbing some coffee but will you be around to go over 2 stories when i get back? [22:56:36] kaity: It looks like that's because https://gerrit.wikimedia.org/r/#/c/108155/ still hasn't been merged :( [22:56:48] jdlrobson, jgonera: ^ [22:56:49] kaldari: ohh ok cool@ [22:57:30] oh yeh i was meaning to ask you about this kaldari [22:57:40] what release was being aimed for? [22:57:45] i know i was too trigger happy last time [22:57:52] kaldari, how is this related to how mobile look like? [22:57:57] it's VectorBeta [22:58:05] looks* [22:58:14] also i still completely don't get why the font was changed again - the commit message doesn't really explain the shift in direction at all [22:58:27] oops, nevermind then [22:58:31] anyway brb but yeh jgonera is right - should have no effect on mobile [22:58:40] I was thinking that was mobile [22:58:59] kaity: OK, I'm not sure why the line-height isn't right in mobile then. Lemme look... [22:59:21] (03PS2) 10Jdlrobson: Load Tablet styles if portrait or landscape meets threshold [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110625 [23:03:07] jdlrobson: the font changes were the result of a discussion during the meeting. I'm not sure what else I can say about them other than what is in the commit msg. Everyone agreed that the preferred font (i.e. the one that most closely matches what design wants to see) should be first, and it should fall back to something as close as possible on machines that don't have that font (typically Linux). The discussion involved Steven, Spage, Jared, t [23:03:07] designers, and myself and Juliusz. I can revert them if you want to merge without those changes. [23:05:12] jdlrobson, we checked what various fonts match to on Ubuntu using fc-match and manual testing in Chrome and Firefox [23:05:25] jdlrobson: Part of the impetus for changing it is that apparently there was feedback from Linux users that the free font for the header was ugly and they would prefer to control the fallback themselves via the Linux font mapping. [23:06:27] Thus 'Times' -> something that looks like Times. Unfortunately Georgia doesn't typically have a fallback mapping on Linux [23:07:54] jdlrobson: are you subscribed to the QA mail list? [23:07:57] So basically we were degrading the experience for Linux users by paying lip-service to free fonts, rather than following the normal way that Linux typically deals with font stacks on the web [23:12:14] kaity: where are you seeing the bad line-heights on mobile? It looks good to me on beta labs. [23:12:35] kaity: are you in Beta mode? [23:12:42] kaldari: yea sorry I saw them on en wikipedia beta and I thought that was the latest [23:12:49] beta labs looks good! [23:12:54] yay! [23:13:14] kaity: sorry the updates have been slow to merge [23:13:17] (03PS1) 10Cmcmahon: update for pwd [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 [23:13:21] kaldari: there's weird stuff happening with infoboxes - they seem to get wider on tablets [23:14:05] kaldari: jdlrobson: said that was a separate bug, or should it be part of this story? https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1565 [23:15:14] kaity: I believe we have a different max-width for infoboxes on tablet than on small mobile. [23:15:25] 400px if I remember [23:16:26] kaity: let me look at the code though... [23:18:01] (03CR) 10CSteipp: "I think you used the wrong variable, but is the only use of the account to login there in the test? If so, it would be much, much better t" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 (owner: 10Cmcmahon) [23:19:24] kaldari: maybe we should look at having a consistent max width [23:20:13] kenanwang_: the abuse filter patch was merged. were those screenshots good enough for signoff? [23:20:58] (03PS2) 10Cmcmahon: update for pwd [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 [23:22:00] kaity: actually it looks like the way we handle infobox width on small mobile is somewhat complicated.... [23:24:23] (03PS3) 10Cmcmahon: update for pwd [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 [23:25:12] kaity: We actually set the width to 100%, but we render the infobox as a block element instead of a table, so that any overflow causes the infobox itself to be horizontally scrollable rather than causing the entire page to be horizontally scrollable. [23:25:13] (03CR) 10Cmcmahon: [C: 032] "merging after talking to Chris Steipp" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 (owner: 10Cmcmahon) [23:25:49] (03Merged) 10jenkins-bot: update for pwd [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110653 (owner: 10Cmcmahon) [23:25:52] kaity: on tablet, we just render them as regular tables, but set a max-width of 400px. [23:26:26] kaldari: oh my goodness [23:28:00] kaldari: what would happen if we changed that to 300px [23:28:06] for example [23:28:15] kaity: so on small mobile the width of the infobox is actually the same width as whatever window you are viewing it in (minus the margins). This is the grey area you see. Within the infobox, the cell elements are rendered naturally. [23:29:29] jdlrobson: im testing out https://gerrit.wikimedia.org/r/#/c/109779/2 and getting a test failure (not getting a toast confirmation) [23:29:34] is it passing for you [23:29:36] ? [23:30:55] kaity: if we just change it to 300px and allow it to render as a regular table, that would probably be fine. The only disadvantage is that if someone were viewing it on a screen smaller than 300px (+ margins) wide, the entire page would become wider than the viewport. [23:31:13] actually i dont seem to get a toast message when i edit/save manually, jdlrobson [23:31:57] kaity: the way it is implemented now is basically a hack to make sure that the page itself is never wider than the viewport, so you never have to scroll horizontally to read. [23:32:03] kaldari: maybe we should look more on monday [23:32:09] sure [23:32:31] fails in master too; i see a @wip at the top of the feature file, so is that expected? [23:33:23] awjr: looking.. [23:33:25] kaity, jdlrobson, jgonera: Do we have any minimum screen size that we support. In other words is there a certain size below which we don't care how it renders? [23:33:36] awjr: yeh that's expected [23:33:41] kk [23:33:46] the test that is added should pass [23:33:47] (03CR) 10Awjrichards: [C: 032] Story 1625: Drop the preview screen [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109779 (owner: 10Jdlrobson) [23:33:52] (03PS3) 10Jdlrobson: Story 1625: Drop the preview screen [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109779 [23:33:53] it does :) [23:34:10] jdlrobson: i just merged and moved #1625 to ready for signoff [23:34:22] awjr: https://gerrit.wikimedia.org/r/#/c/108990/ will fix that one [23:35:00] awjr: is kenanwang_ around to sign off? I asked him earlier but didn't get a response [23:35:09] chrismcmahon: yeh i'm on the qa list [23:35:11] not sure if he's still around [23:35:21] i talked to him maybe an hour ago [23:35:36] jdlrobson: according to his calendar he is traveling [23:35:41] so he's probably off for the day [23:35:43] jdlrobson: OK. there was a problem creating new users, I just talked to Chris Steipp about it [23:35:55] i just saw - the test password change? [23:36:21] jdlrobson: yes. there were 3 accounts on test2wiki, I've fixed all their passwords [23:36:46] thanks chrismcmahon [23:37:01] sorry for the trouble [23:38:24] jdlrobson: right now I just merged a change so that test won't run in the build, and if it runs locally it will use MEDIAWIKI_PASSWORD env var [23:38:48] jdlrobson: we should talk to Steipp about the best way to do what you want to do. [23:39:50] jdlrobson: Any verdict on the fonts? I'll need to circle back with the designers if you don't want to change them. [23:40:39] jdlrobson: im seeing the toast message with that patchset (manually testing) but the browser test still fails - maybe a timing issue? [23:40:46] is the test passing for you? [23:42:05] kaldari, are you working on https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1636 ? [23:43:12] I was just about to start on it. Hopefully it shouldn't be too hard to fix. [23:43:16] kaldari: i don't have any 'verdict' i just want us to be clear why this is happening so we don't get shot by the people who argued strongly for free fonts [23:43:32] awjr: link reminder would help so i know what you are talking about - juggling many things right now [23:43:43] jdlrobson: https://gerrit.wikimedia.org/r/#/c/108990/8 [23:44:14] so it doesn't pass for you awjr ? [23:44:16] i'll take a look [23:44:22] correct jdlrobson [23:44:25] i suspect its a timing issue tho [23:44:34] submitting the edit seems to take a while on my local instance [23:44:36] which one awjr - keep going or editor-ve [23:44:40] editor-ve [23:44:55] so no toast confirmation about editing in VE [23:44:56] ? [23:45:01] *after editing [23:45:35] jdlrobson: correct [23:45:50] errrr [23:45:57] jdlrobson: also, the keep going test is failing for me - both scenarios, on 'I click edit' [23:45:58] does anything happen awjr [23:46:09] jdlrobson: when i test manually i see the toast [23:46:26] when i watch selenium work, i see the spinner after it clicks submit, then it times out and fails [23:47:01] mm maybe VE changed and it broke [23:47:09] jdlrobson: it works when i test manually tho [23:47:15] that's why i think it might be a timing issue [23:47:22] timed out after 5 seconds, waiting for {:class=>"ve-ce-documentNode ve-ce-branchNode", :tag_name=>"div"} to become present (Watir::Wait::TimeoutError) [23:47:25] yeah [23:47:36] is that was fails for you too? [23:47:39] yes [23:47:48] oh [23:47:49] wait [23:47:49] no [23:47:53] i should've read the whole error [23:48:01] jdlrobson: imed out after 5 seconds, waiting for {:class=>"toast position-fixed visible", :tag_name=>"div"} to become present (Watir::Wait::TimeoutError) [23:49:49] awjr: mm let me have a play around with it [23:50:55] (03PS9) 10Jdlrobson: Make KeepGoing/toast confirmation run when editing using VE [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/108990 [23:50:58] ^ try this one awjr it works for me [23:51:20] trying [23:52:11] hmm still timeout but lemme try again after clearing caches etc [23:52:32] awjr: it seems to take a while for it to pass for some reason for me - but it does eventually pass [23:53:06] jdlrobson: :( im still getting the same error [23:54:21] awjr: i can replicate now trying to work out what is going on [23:54:27] it used to work.. [23:54:28] groovy [23:55:23] (03CR) 10Awjrichards: [C: 032] Story 1625: Drop the preview screen [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109779 (owner: 10Jdlrobson) [23:55:57] (03Merged) 10jenkins-bot: Story 1625: Drop the preview screen [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109779 (owner: 10Jdlrobson) [23:57:22] (03CR) 10Awjrichards: "Needs manual rebase for merge conflicts" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/109807 (owner: 10Jdlrobson) [23:57:44] jdlrobson: awjr really old bug about VE and Flow that might be relevant: https://bugzilla.wikimedia.org/show_bug.cgi?id=55766 [23:57:57] kaldari, do you need hash of the commit that caused this bug? [23:58:12] sure, if you have it [23:59:04] chrismcmahon: interesting, but that bug suggests the problem is only in chrome - i am seeing this in firefox with the automated tests [23:59:27] chrismcmahon: seems unrelated to that