[01:58:48] (03CR) 10Dbrant: [C: 032] Disambiguation and page issues dialog on bottom [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171512 (owner: 10BearND) [02:01:30] (03CR) 10Dbrant: [C: 032] Slide up/down animations for BottomDialogs [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171725 (owner: 10BearND) [02:04:41] (03CR) 10Dbrant: [C: 032] Left/right animation inside page info dialog [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171875 (owner: 10BearND) [02:18:28] (03CR) 10Dbrant: [C: 032] Disambig items have thumbnail, title, and Wikidata description [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171876 (owner: 10BearND) [02:20:50] I'm looking to augment a series of field guides to flora and fauna. Each page in the field guides describes some species, and textually describes the range where the specie may be found. For this to be more useful, I would like for the users to be able to select a region of the world and get a list of pages (or better, a page which transcludes the relevant pages) for that region. I was thinking the geodata extension migh [02:20:50] tag each page to cover regions. Some regions are small, some are large, and none are circular. Most pages would need multiple coordinate tags. For ease of content creation, perhaps each page could transclude a template naming a region, and that template could add the tags? Or maybe there is a better way? [02:23:11] Also, does GeoData sound like the right tool for this job? [02:24:05] (03CR) 10Dbrant: [C: 032] Page info dialog header click feedback + clickable [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172479 (owner: 10BearND) [02:24:45] (03PS7) 10Dbrant: Drag some Wikipedia domain stuff into the Site class [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/164876 (owner: 10Alex Monk) [02:53:32] (03CR) 10Dbrant: [C: 04-1] Drag some Wikipedia domain stuff into the Site class (031 comment) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/164876 (owner: 10Alex Monk) [02:59:09] (03CR) 10Dbrant: [C: 032] Faster up/down animations for bottom dialogs [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172496 (owner: 10BearND) [03:17:24] (03CR) 10Dbrant: [C: 04-1] "A couple notes:" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172454 (owner: 10Deskana) [03:23:26] (03PS9) 10Dbrant: Face detection in lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 [03:23:45] (03PS14) 10Dbrant: Implement syntax highlighting in Edit window. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/137684 [03:24:21] (03PS3) 10Dbrant: Remove pinch-zoom in the WebView. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171850 [03:25:16] (03PS14) 10Dbrant: Make infoboxes (and other tables) collapsible. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 [06:37:33] (03CR) 10BearND: [C: 04-1] "The white background seems a bit jarring in dark mode." (033 comments) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 (owner: 10Dbrant) [06:39:44] (03CR) 10BearND: [C: 032] Remove pinch-zoom in the WebView. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171850 (owner: 10Dbrant) [06:41:27] (03Merged) 10jenkins-bot: Remove pinch-zoom in the WebView. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/171850 (owner: 10Dbrant) [06:43:12] (03CR) 10BearND: [C: 04-1] "In dark mode there is hardly a difference between templates and other text." [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/137684 (owner: 10Dbrant) [07:25:58] 3Wikipedia App / 3General/Unknown: Disable translations to wuu and lzh? - 10https://bugzilla.wikimedia.org/73295 (10Nemo) 3NEW p:3Unprio s:3normal a:3None A translator asks if translations to Literary Chinese and wuu can be disabled, since the locales are not available on phones (according to the use... [07:46:39] (03PS8) 10Alex Monk: Drag some Wikipedia domain stuff into the Site class [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/164876 [09:27:53] yo [09:30:23] phuedx: hi! [09:30:28] how did it go? [09:47:21] fine [09:47:28] nothing doing, unfortunately [09:47:30] ^ joakino [09:47:36] sorry, was making breakfast [09:47:42] i don't usually eat breakfast, but when i do [09:47:50] phuedx: pity [09:47:59] yeah :( [09:48:02] bon appetite [09:48:36] eggs and fish [09:48:40] gotta love dem eggs and fish [09:48:53] brb [09:49:21] back [09:52:30] * phuedx is watching the last quarter of the phabricator thingemy [09:55:20] nom nom nom nom nom [09:56:20] joakino: what're you working on? [09:57:25] right now i'm with emails and administrative stuff [09:57:35] phuedx: nothing in particular [09:57:44] okie poke [09:57:49] was just looking at the backlog [10:02:53] phuedx: i ended up yesterday fighting with the ios bug [10:03:05] phuedx: i'm going to try a bit more, but i'm about to give up... [10:03:30] the scrolling thing? [10:03:31] then i'll go code review and find something else to work on [10:03:34] yes [10:10:10] i think all of the code review is either blocked on external reviewers or awaiting work [10:10:20] people should take fewer holidays [10:13:00] i am full of eggs and fish [10:15:20] phuedx: fish for breakfast [10:15:22] haha [10:15:28] awesome [10:15:43] smoked mackerel and scrambled eggs [10:15:51] a little salt and pepper [10:15:52] boom [10:15:57] protein bomb [10:16:07] 👍 [10:16:18] i'm going bouldering at lunch, so the protein may help [10:16:25] although i should probably eat it /afterwards/ [10:17:54] well this good breakfast its going to keep hunger away for a good while, just eat some carbs later for extra energy [10:18:05] very cool [10:18:31] i think i'm going to bike a bit later, it is sunny today [10:18:54] if you ever visit london, i'll take you to the bouldering centre 10 minutes away from my house [10:21:02] phuedx: 👍 awesome plan [11:17:55] 3MobileFrontend / 3Feature requests: Indicators are not shown in mobile view - 10https://bugzilla.wikimedia.org/73299#c1 (10Bartosz Dziewoński) Yeah, MobileFrontend's Minerva skin needs to have support for them added (assuming that it is wanted, I'm not sure if it currently displays FA icons and the like). I... [11:20:25] 3MobileFrontend / 3Feature requests: Indicators are not shown in mobile view - 10https://bugzilla.wikimedia.org/73299#c2 (10Bingle) Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/gb3kNPJJ [11:40:50] phuedx: phuedx/fiveMinuteAbs lol wtf [11:41:08] if your branch names don't contain jokes [11:41:11] you're doing something wrong [11:42:35] phuedx: damn it [11:42:46] haha [11:43:03] phuedx: i've been reviewing taht [11:43:18] cool design, and very clean code, too big of a commit in my opinion [11:44:01] but with small commits rebasing is a nightmare... so, bleh [11:44:50] phuedx: i'm running vagrant git-update to get the mediawiki cookie thingy [11:45:46] joakino: maybe wikigrokuser.js could've been broken out [11:45:48] phuedx: add a card into the developer backlog to migrate all $.cookie, it is better your way [11:46:13] you should take a look at the review history of mediawiki.cookie ;) [11:46:20] it took a while [11:46:23] but it's better for it [11:46:35] link? [11:48:19] phuedx: it is just that I like reviewing smaller commits, so you see with the commit messages and the small code changes the intentions of the author and it is easier to follow (when there is a big wall of code it is harder to keep focus) [11:48:27] phuedx: maybe i'm just getting old [11:49:01] joakino: smaller things are easier to reason about [11:49:14] that's how i say it [11:49:24] commits, teams, systems, classes, methods [11:49:28] books [11:49:30] paragraphs [11:49:33] premises [11:49:37] pick your poison [11:49:43] yep [11:49:52] maybe i could've broken out wikigrokuser [11:50:00] that patch is large because i like unit tests ;) [11:50:05] sure [11:50:34] what i'm saying is: i agree with you, i'll break it up next time around [11:50:49] *i'll try harder to break it up next time around* i meant [11:51:34] that's why I prefer branches with lots of small commits, review and code with as many small commits as you want, and later merge rebasing into master a big commit with a meaningful message extracted out of all the small commits [11:52:09] phuedx: the problem is that gerrit makes it pretty hard to do that flow, and it bothers you less when you work with big commits [11:52:18] that's why I don't like gerrit xd [11:52:26] sure, it makes it harder to do the last part of that flow [11:53:14] anyway rant over [11:53:30] phuedx: your change is good, I'm not sure also of how I would have split it [11:53:41] 👍 [11:53:51] there's a follow-up change that /should/ be done [11:54:10] which is related to my comment about there being two things responsible for choosing which version of wikigrok to load [11:54:25] that can be broken out later [11:55:15] meaning what, the abtest and the user? [11:57:42] (03CR) 10Jhernandez: WikiGrok A/B test dry run (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [11:58:22] there's something that sits between wikigrok.js and wikigrokuser/abtest [11:58:53] phuedx: do you want me to wait for somebody else? i think it is ready for +2 [11:59:14] i'd like jdlrobson (at least) to get back to me about mw.cookie [11:59:23] phuedx: yes we should leave wikigrok.js as empty of logic as possible, and add tests to what we refactor [11:59:23] just so everyone's on the same page [11:59:26] oki, then +2 [11:59:30] sorry, +1 [11:59:33] :) [11:59:35] cool.io [12:00:30] (03CR) 10Jhernandez: [C: 031] WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [12:15:43] gonna go for a boulder [12:15:58] enjoy!1 [12:24:00] (03CR) 10Jhernandez: [C: 04-1] Open up the WikiGrok A/B to alpha (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172551 (owner: 10Phuedx) [13:32:51] Florianschmidtwelzow you here? [14:03:48] (03PS15) 10Dbrant: Implement syntax highlighting in Edit window. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/137684 [14:30:04] (03PS2) 10Phuedx: Open WikiGrok A/B test to all users on alpha/beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172551 [14:34:35] i'm out for lunch [14:49:40] (03PS1) 10Dbrant: [WIP] Final styling tweaks for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 [14:49:42] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Final styling tweaks for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 (owner: 10Dbrant) [14:56:26] (03PS2) 10Dbrant: [WIP] Final styling tweaks for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 [14:57:21] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Final styling tweaks for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 (owner: 10Dbrant) [15:05:17] (03PS1) 10Phuedx: Hygiene: Use spinner template instead of Icon [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172743 (https://bugzilla.wikimedia.org/73251) [15:05:19] (03PS1) 10Phuedx: Hygiene: Use the spinner class everywhere [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172744 [15:06:10] 3Wikipedia App / 3General/Unknown: iOS & Android: MobileFrontend styles unconditionally adding borders to tables - 10https://bugzilla.wikimedia.org/68579#c7 (10Andre Klapper) 5PATC>3NEW Resetting bug report status from PATCH_TO_REVIEW to NEW as there are no patches left to review. [15:06:51] coffee time! [15:33:43] (03PS15) 10Dbrant: Make infoboxes (and other tables) collapsible. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 [15:37:03] (03PS16) 10Dbrant: Make infoboxes (and other tables) collapsible. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 [16:03:58] dbrant: are you around ? [16:04:11] matanya: yep [16:04:39] hello, i would to to report that with the new alpha, the page mostly times out after a saved edit [16:05:03] i.e i edit a page, get the message the page was saved, and never get back to the article [16:05:40] dbrant: furthermore i would like to thank you for the cover picture for the article, it is very nice. [16:05:53] but on the other hand argue for it's need [16:06:57] it draws more bandwith with very little info added [16:06:57] so i would prefer it to be an option for the user to choose wheather he wants it on or off [16:08:16] dbrant: if you wish me to open a bug report for this, i can [16:08:23] matanya: thanks! True, the "need" of the cover picture may be more aesthetic than practical, but we're simply trying to bring the app into 2014, instead of 2004. [16:09:23] matanya: hmm, i'll surely look into the editing issue! I've reproduced it just now. [16:10:01] dbrant: ok thanks, if you need anything from me, do poke me [16:34:27] 3Wikipedia App / 3MobileApp extension: 12.11.14 alpha times out after saving an edit - 10https://bugzilla.wikimedia.org/73314 (10matanya) 3NEW p:3Unprio s:3normal a:3None how to reproduce: 1. use the alpha from the subject 2. edit a page and save. 3. you get a message your edit was saved, but you... [16:39:12] 3Wikipedia App / 3Android App: allow user to configure if he wants cover images in the app - 10https://bugzilla.wikimedia.org/73315 (10matanya) 3NEW p:3Unprio s:3enhanc a:3None The new alpha recently add a cover image to articles, which is a very nice feature in terms ofveye kandy. However, this lea... [16:39:40] 3Wikipedia App / 3Android App: 12.11.14 alpha times out after saving an edit - 10https://bugzilla.wikimedia.org/73314 (10Dmitry Brant) [16:49:54] (03CR) 10BearND: [C: 032] Make infoboxes (and other tables) collapsible. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 (owner: 10Dbrant) [16:50:39] bearND: thx! note that there's also a patch in MobileApp that corresponds to this. [16:50:55] dbrant: ah, thanks for the heads-up [16:51:39] (03Merged) 10jenkins-bot: Make infoboxes (and other tables) collapsible. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/165605 (owner: 10Dbrant) [17:41:41] hey joakino phuedx! [17:41:51] yo [17:42:22] heads up, i'm thinking a good focus for kitty grooming that we should start with is getting our tools in order e.g. the grunt file, jsbeautify, publishing stuff. Any objections to focusing on tasks around that to start with? [17:43:59] publishing stuff like docs? [17:44:04] or publishing stuff like code metrics? [17:44:07] or both? (!!!) [17:44:19] both :) [17:44:23] \o/ [17:44:27] absopositively [17:44:36] phuedx: obviously depending on how 'big' we think things are [17:44:40] and what's achievable [17:44:44] obvs [17:44:56] so i was thinking we could focus discussion around that tomorrow and get some units of work identified and estimated [17:44:59] (03PS3) 10Dbrant: Final styling tweaks/fixes for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 [17:45:06] i love that you say units of work [17:45:10] suresies [17:45:11] :P [17:45:29] So yeh if you think of any more wild crazy ideas around tools that we should add please add 'em to the trello board [17:45:53] (03CR) 10jenkins-bot: [V: 04-1] Final styling tweaks/fixes for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 (owner: 10Dbrant) [17:45:54] do we have code coverage tools for php? [17:46:02] phuedx: i believe there are some in phpunit [17:46:03] i've written a card, but i'm curious what's already there [17:46:05] in core [17:46:09] ok [17:46:10] i don't know where they go though [17:46:14] i'll take a look and flesh the card out [17:46:29] don't want to us to commit to something that's essentially undefined [17:47:11] spike! [17:47:13] spike! [17:47:14] spike! [17:47:37] phuedx: https://integration.wikimedia.org/cover/mediawiki-core/master/php/index.html [17:47:39] this exists [17:47:47] (also lolz) [17:47:56] awesome [17:47:57] and :'( [17:48:28] bmansurov: did you take yesterday off? [17:48:38] or did you work with these crazy cats? [17:49:00] jdlrobson: i chilled yesterday [17:49:15] bmansurov: cool :) [17:49:17] jdlrobson: icon sizes: everything in ems, right? [17:49:47] (this is always where i get things in a muddle) [17:50:02] i think that, as a team, we should commit to less chilling [17:50:04] more work [17:50:04] jdlrobson: phuedx wow those metrics are sad [17:50:14] phuedx: correct. although in the future there will be two icon sizes - indicators (16px) and icons (24px) so we shouldn't have to worry about icon sizes [17:50:21] at least that's where TrevorP|Away is trying to get us [17:50:54] phuedx: let's get to the point we don't have to even think about how icons exist :) [17:50:57] we just add them to things [17:51:10] that'd be nice, wouldn't it? [17:51:43] phuedx: on https://gerrit.wikimedia.org/r/#/c/172743/ should we just replace the spinner with an icon? [17:51:47] after all it is an icon... [17:52:25] jdlrobson: sure after we use the spinner consistently, and manipulate it consistently [17:52:31] ;) [17:52:32] (03PS4) 10Dbrant: Final styling tweaks/fixes for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 [17:52:36] phuedx: that would be nice. :) [17:52:59] joakino: I was also thinking about maybe creating some kind of AsyncView which handles View's which have to make api requests to get data to render. [17:53:01] i'll push a follow-on patch to replace the spinner with an icon [17:53:06] i'm not good at naming stuff though [17:53:36] jdlrobson: meaning one that would automatically show and hide the spinner? [17:54:26] (03CR) 10Jdlrobson: [C: 04-1] "Please also update the category button to be neutral as well. They look strange together in alpha." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172408 (owner: 10Bmansurov) [17:54:48] (03PS3) 10Jdlrobson: Proposal: Merge CtaDrawer with EditorOverlay Anonwarning [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:00:24] (03PS2) 10Bmansurov: Conform to the MobileWebWikiGrok schema [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172443 [18:00:37] (03CR) 10Jdlrobson: [C: 04-1] Proposal: Merge CtaDrawer with EditorOverlay Anonwarning (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:10:56] 3Wikipedia App / 3Android App: Android app: Changing to night mode while searching cancels the search - 10https://bugzilla.wikimedia.org/72604#c1 (10Bernd Sitzmann) 5NEW>3RESO/FIX The menu to change themes is not available anymore from search. So one cannot get into this situation anymore. [18:17:00] (03CR) 10Jdlrobson: Proposal: Merge CtaDrawer with EditorOverlay Anonwarning (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:17:09] (03PS4) 10Jdlrobson: Merge CtaDrawer with EditorOverlay Anonwarning [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:17:20] (03PS5) 10Jdlrobson: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:18:02] * phuedx goes back to eating his sammich [18:18:52] om nom [18:20:37] chicken, bacon, lettuce, tomato, mayo, tobasco sauce [18:20:40] i am so happy [18:24:23] (03PS3) 10Jdlrobson: Hygiene: Clearly identify when/how things get initialised. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172457 [18:24:32] (03CR) 10Bmansurov: "Isn't category button also updated already?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172408 (owner: 10Bmansurov) [18:24:34] (03PS3) 10Jdlrobson: Hygiene: Put TableOfContents in own file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172458 [18:24:37] DarTar: Where do you have the WikGrok A/B test stuff written up? [18:24:53] (03CR) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:25:02] (03CR) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [18:25:06] (03CR) 10jenkins-bot: [V: 04-1] Hygiene: Clearly identify when/how things get initialised. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172457 (owner: 10Jdlrobson) [18:25:14] (03CR) 10jenkins-bot: [V: 04-1] Hygiene: Put TableOfContents in own file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172458 (owner: 10Jdlrobson) [18:26:05] jdlrobson: can you check this again? I think the category button is already updated: https://gerrit.wikimedia.org/r/#/c/172408/ [18:27:17] (03CR) 10Florianschmidtwelzow: "@bamnsurov: i think so, but haven't tested." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172408 (owner: 10Bmansurov) [18:27:40] 3MobileFrontend / 3beta: Can't scroll in edit mode - 10https://bugzilla.wikimedia.org/73255#c5 (10Jhernandez) Can't reproduce with Android 4 and neither Chrome nor Firefox. Please can you tell us which android version has your phone? Also which browser are you using? If you don't know how to get that info... [18:28:56] jdlrobson: let me know when we can chat about limn. thanks [18:29:14] bmansurov: sure [18:32:15] limn chats! [18:32:18] (03CR) 10Jdlrobson: [C: 04-1] Add tests to WikiGrokDialogB (038 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172569 (owner: 10Jhernandez) [18:32:29] bmansurov, jdlrobson: i'd really appreciate any notes you guys make being published somewhere [18:32:41] no swearing, obvs ;) [18:32:42] what is limn [18:32:51] joakino: don't open pandora's box ;-_ [18:32:53] phuedx: ok, i'll take some notes and share [18:33:05] mobile-reportcard.wmflabs.org is the output [18:33:10] ^ joakino [18:33:30] bmansurov: awesome [18:33:33] okay bmansurov let me get you up and running. Do you want to me to grab a room and hangout? [18:33:39] :D [18:33:41] jdlrobson: sure [18:33:56] let me see if i can find one [18:33:58] maryana bmansurov jdlrobson phuedx joakino maxsem rmoen kaldari [18:34:03] Bug triage? [18:34:09] joakino: This page has some useful documentation (although it is specifically about the EE dashboard, not the mobile dashboard): https://wikitech.wikimedia.org/wiki/EE_Dashboard [18:34:10] (03CR) 10BearND: [C: 032] Final styling tweaks/fixes for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 (owner: 10Dbrant) [18:34:14] oh yeh bmansurov let's do bug triage first [18:34:15] yo yo [18:34:18] suggestion: highlight words in yer irc client [18:34:18] https://trello.com/c/MSED64NS/46-bug-73243-talk-add-discussion-button-look-like-other-overlay-buttons [18:34:24] mobilewebteam? [18:34:26] webteam? [18:34:26] thx kaldari|2 jdlrobson [18:34:41] so kristenlans doesn't have to type all of that repeatedly ;) [18:34:48] phuedx: good suggestion :-) [18:35:10] I like webteam [18:35:12] are we starting from the top? [18:35:43] joakino: Here is our reportcard (with limn graphs): http://mobile-reportcard.wmflabs.org/ [18:35:50] Maryana it depends, do we want to get in to those stories or just triage bugs? [18:36:05] (03Merged) 10jenkins-bot: Final styling tweaks/fixes for lead images / wikidata description. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172735 (owner: 10Dbrant) [18:36:09] kristenlans: a good description of the last.fm webteam's practices: http://paulrhayes.com/2013-06/last-fm-webteam/ [18:36:12] (details irc comms) [18:36:43] kristenlans: i think "add event_isWP0" was fairly uncontroversial. let's move that into ready for dev [18:36:54] wikigrok error state can go to next sprint [18:37:12] Maryana: before moving it can we at least sanity check those story points [18:37:14] and i don't know what "document our limn stuff" is :) [18:37:19] oh yeah [18:37:23] since no one has looked at that card other than kaldari :) [18:37:27] do it [18:37:59] So all we are basically doing is adding an isZero field [18:38:14] how do we know if we are in zero or not dr0ptp4kt [18:38:30] if you split the card into two: log the data, whisper the limn incantation at the appropriate shrine, you'd two small-ish stories [18:38:38] phuedx: +1 to that [18:38:46] jdlrobson: on the scrum of scrums meeting. let's discuss afterward [18:38:47] * jdlrobson does that [18:39:15] Maryana: can we just add a graph of zero edits? [18:40:15] or do they not need a graph? [18:40:29] * jdlrobson creates https://trello.com/c/dEwKAcki/55-update-existing-limn-graphs-to-show-zero-edits [18:40:55] joakino / phuedx / rmoen any thoughts about https://trello.com/c/d1Xa7bvt/24-5-add-event-iswikipediazero-to-mobile-web-edit-schema ? [18:40:58] bmansurov: ^ [18:41:41] awaiting an answer to 6:38:14 PM how do we know if we are in zero or not dr0ptp4kt [18:41:42] change null to false as MaxSem said? [18:41:46] jdlrobson: What's the status of core templates right now? [18:41:53] still, teeny tiny [18:42:14] jdlrobson: what is the 'new field' [18:42:17] jdlrobson: estimates? [18:42:18] phuedx: i'll talk with you and jdlrobson after i get off this meeting [18:42:31] kaldari|2: ask TrevorP|Away/brion :) mobile web is just waiting on mustache in core [18:42:58] http://hatjitsu.wmflabs.org/17699 < joakino rmoen etc [18:43:01] ta [18:43:06] was about to ask [18:43:23] jdlrobson: Is there a patchset for that yet? [18:43:34] kaldari|2: not that i know of, but we are not doing anything relating to that [18:43:41] we got our stuff merged and using core [18:43:41] cool, thanks [18:43:49] idk if i can properly estimate this but i'm giving it a go [18:43:55] so looks like 1-3 [18:44:06] any strong reasons for the 3? [18:44:33] i said 3 but thats me, knowing about logging, and the schemas etc it is probably a 1 [18:44:43] or 2 [18:44:45] i said 2 just because of the open question to dr0ptp4kt :) [18:44:51] i'm fine with 2 [18:45:02] i voted one, but danger 2 is fine [18:45:03] 👍 [18:45:06] (y) [18:45:18] jdlrobson: ctrl+command+space [18:45:20] okay 2 it is :) [18:45:25] 🙀 [18:45:29] https://trello.com/c/dEwKAcki/55-update-existing-limn-graphs-to-use-new-editing-schema < might as well do this one next [18:45:30] 👻 [18:46:00] 🚀 [18:46:07] i don't think anyone except for jdlrobson and kaldari|2 know about limn graphs [18:46:14] bmansurov: you'll know soon [18:46:16] no idea [18:46:19] but basically this is just updating an sql query [18:46:22] yeah, but i can't estimate now [18:46:26] ok [18:46:42] jdlrobson: kaldari|2 we need at least a good description on the card [18:46:49] like, a technical description xd [18:47:20] limn graphs are so lovely... [18:47:43] done [18:47:48] irc needs out of band sarcasm metadata [18:47:49] i'd suggest a 1. If anyone wants to hatjitsu though we can :) [18:48:06] looks like we are hatjitsuing [18:48:11] waiting on one person [18:48:17] lol [18:48:17] heheh [18:48:19] who was the 5? [18:48:23] danger 5 from me [18:48:30] i did jon*2 = 2 [18:48:40] only one thing better than writing limn graphs is hacking on limn itself. [18:48:43] i did 3 (which is my goto answer when I don't know how to properly estimate) [18:48:47] ;/ [18:48:56] i'm sorta ok with a three [18:48:57] there is a ? too ;) [18:49:08] i'm trying to find the code to convince you it's a 1 [18:49:13] if someone points me in the right direction i can get all up in your sql [18:49:20] jdlrobson: not the time ;) [18:49:29] i gotta go get george to bed in a sec [18:49:34] ok ;-) [18:49:45] can i pull you down to a 2 at least? It really is just a one liner [18:49:49] a very long one liner [18:49:49] 👶💤 [18:49:53] but a one liner sql nevertheless [18:49:53] ^ [18:50:03] ... [18:50:07] ... [18:50:08] jdlrobson: we trust you, let's go with a 1 [18:50:11] zzzzz [18:50:16] danger 2 [18:50:17] go [18:50:18] danger 2 okay [18:50:22] ✌️ [18:50:32] fear-based estiamtion™ [18:50:39] https://trello.com/c/MSED64NS/46-bug-73243-talk-add-discussion-button-look-like-other-overlay-buttons [18:50:52] i'm kinda hoping FlorianSW will want to work on this? :) [18:51:31] basically the add button is huge. But since it's in beta and we're still not sure how talk/flow fit in i'm assuming it's a low priority for Maryana [18:51:36] and a bug so doesn't need estimation [18:51:44] ping him then, and will see [18:51:53] do we need input from design there? [18:51:59] joakino: possibly. ok i'll leave it there and we'll come back to it [18:52:08] i'll let Maryana decide [18:52:11] ok [18:52:13] i gotta go [18:52:16] backlog :) [18:52:17] https://trello.com/c/gb3kNPJJ/54-bug-73299-indicators-are-not-shown-in-mobile-view < lasty a feature request [18:52:20] * jdlrobson waves at phuedx [18:52:21] george is standing at the top of the stairs [18:52:28] "come on daddy!" [18:52:30] "bed time!" [18:52:31] "read!" [18:52:42] (i'm assuming the limn graph docs will come out of bmansurov :) [18:52:55] phuedx: bye! 👋 [18:52:57] sure [18:53:18] bye phued [18:53:23] …x [18:53:55] i'm not sure what the status of indicators are but i'd say not a high priority Maryana [18:54:49] backlogged (maryana in real life said so) [18:54:51] i have no idea of what those are [18:54:57] 👍 [18:55:03] kaldari|2: what did you say about https://trello.com/c/Elmti0S9/53-blocked-set-a-flag-config-var-to-start-the-test ? [18:55:12] we were going to kill it? [18:55:26] jdlrobson: Yeah, I think we can kill that. [18:56:05] omg [18:56:10] lol joakino [18:56:11] what is even happenign on that card [18:56:12] oh jesus [18:56:14] i'm terrified [18:56:18] LOLOLOLOL [18:56:19] the gif went to cover lol [18:56:23] make it go away [18:56:26] Can phabricator do this? [18:56:37] i don't think so [18:56:43] we should all pay pro trello accounts [18:56:52] lolz [18:56:53] and start working on phabello [18:57:02] * jdlrobson will build phabricator gold [18:57:51] ok end of triage [18:57:57] i'm gonna go help bmansurov now. [18:58:04] cool [18:58:05] everyone got enough work to do for rest of day/tomorrow joakino ? [18:58:08] awesome [18:58:17] jdlrobson: i'll grab something tomorrow [18:58:23] i'm leaving now, 8pm [18:58:30] See ya joakino [18:58:37] bye, joakino [18:58:39] joakino: cool have a good evening :D [18:58:47] thanks for all the developer backlog cards :) [18:58:48] i'll go tomorrow through the dev board, trying to fill in details and add more stuff jdlrobson [18:58:56] yeh joakino go nuts [18:59:00] should be a fun conversation [18:59:20] yeah 😬 [19:00:51] 🌘 bb! [19:02:36] bmansurov: grabbing a room [19:02:40] jdlrobson: hi :) Yes, i will work on this bug (i tested it in a free minute, but only "add" is at the top of the button (not like any other overlay continue button, e.g. in EditorOverlay). I'm wondering if we can just transform it into a button and handle a click in TalkOverlay.js? [19:03:18] jdlrobson: work on this bug -> work on this bug after i finished setting up my new computer (takes a while :/) [19:03:54] FlorianSW: not sure i follow [19:04:04] you tried changing the label to 'add' ? [19:04:15] jdlrobson: yes [19:04:59] FlorianSW: and no good? [19:05:37] no, it's still at the top [19:05:47] (like now, but without "discussion") [19:05:51] jdlrobson: ^ [19:05:54] FlorianSW: but that's consistent with editor no? [19:05:58] top right, blue, text label [19:06:27] jdlrobson: no, editor is using button, TalkOverlay changed to link (a) [19:06:43] jdlrobson: no :D I mean the text in the button :) [19:07:08] bmansurov: 1) /analytics/limn-mobile-data [19:07:17] 2) wikimedia/limn [19:08:08] https://git.wikimedia.org/tree/wikimedia%2Flimn.git and https://git.wikimedia.org/tree/analytics%2Flimn-mobile-data.git [19:09:08] https://github.com/wikimedia/limn [19:09:39] 1) https://github.com/wikimedia/limn.git [19:09:49] https://git.wikimedia.org/git/analytics/limn-mobile-data.git [19:10:23] cd limn-mobile-data and cat README.txt [19:14:51] YuviPanda: can you give bmansurov access to the EventLogging tables on stat1? [19:14:57] or stat1003 [19:15:02] whatever we call it these days :) [19:15:16] jdlrobson: it's a 3 day process involving manager approval, I'm afraid. I can file the paperwork, though. [19:15:20] FlorianSW: ah i see, yeh the button and link should be styled the same - they should use the same class [19:15:28] 3 'business days' [19:15:35] YuviPanda: erggg [19:15:48] YuviPanda: i'm trying to get baha up and running so he can build limn graphs [19:16:04] jdlrobson: I think it's stupid, let me also paralelly start a thread to make that faster. [19:16:04] jdlrobson: ok :) Will do it so. [19:16:16] FlorianSW: you're a star :) yey for more generic code [19:16:21] YuviPanda: it's epic having you in ops :) [19:16:27] jdlrobson: hehe :) [19:16:28] i hope you are enjoying it so far [19:16:36] make all the things better!: ) [19:16:45] jdlrobson: I totally am :) Mostly improving betacluster atm. [19:16:56] and making alerts so people know it has problems before it goes down [19:17:04] jdlrobson and phuedx: to identify whether the page is theoretically eligible for zero-rating, check for window.zeroGlobalConfig. that object will contain boolean key called 'enabled'. true means that the page is in fact zero-rated, whereas false means it is in fact not zero-rated. if you're extending event logging schemas, it would be nice to capture whether the page was theoretically zero-rated as well as whether it was in fact zero-rat [19:17:05] or was not zero-rated. that will help in analysis to determine how much zero-rated traffic could have been achieved if only, for example, fr.m.wikipedia.org was zero-rated on this particular configuration...and so on. we should probably get on a video call to go over this a bit more. jdlrobson is getting back from lunch, so maybe a bit later? [19:17:27] bmansurov: you're bmansurov@wikimedia.org, right? [19:17:30] phuedx: i realize maybe it's getting late. i can hop on video now with you if you like [19:17:37] YuviPanda: yes [19:17:38] dr0ptp4kt: i'm actually at other side of room helping baha [19:17:56] jdlrobson: also, suggest that anyone else who doesn't have access ask for it now. I got dbran.t and bearN.D access as soon as they joined. [19:17:57] we can talk afterwards and let phuedx sleep :) [19:18:07] jdlrobson: I think it should be part of onboarding [19:18:09] YuviPanda: sure. [19:18:14] phuedx: i stand corrected. phuedx, you available? if you want jdlrobson to handle the discussion that's cool. [19:18:17] jdlrobson: anyone else who needs it? [19:18:42] jdlrobson: I'll also request stat1002 access, which makes it possible to run hive queries (pageviews / useragent stats) [19:18:48] YuviPanda: i'm not sure i'm finding out [19:18:52] YuviPanda: you still on mobile-tech? [19:18:58] jdlrobson: I am, have it filtered. [19:20:10] bmansurov: did you get an email from RT just now/ [19:20:11] ? [19:20:27] jdlrobson: cc me as well, if you're writing to mobile-tech? [19:21:24] YuviPanda: will do sorry talking to bmansurov [19:21:36] jdlrobson: yeah, 'tis ok. IRC is asynchronous, etc :) [19:23:14] YuviPanda: no I haven't [19:23:45] bmansurov: hmm, ok. Can you upload your public ssh key to officewiki under your userpage? [19:23:54] YuviPanda: ok [19:25:04] YuviPanda: https://office.wikimedia.org/wiki/User_talk:Bmansurov_(WMF) [19:25:38] bmansurov: cool! you're all good now, except have to wait till Monday. [19:25:47] YuviPanda: ok thank you [19:25:55] bmansurov: yw! I'll notify you when it's done. [19:26:35] cool [19:27:10] 3MobileFrontend / 3beta: Can't scroll in edit mode - 10https://bugzilla.wikimedia.org/73255#c6 (10pamdavies7) Thanks for the two links. The first tells me Android 4.1.2 (Jelly Bean) The second says: Android Browser 4 on Android (Jelly Bean) I can scroll up and down articles and talk pages, no problem.... [19:28:59] yo [19:29:28] (03CR) 10Kaldari: [C: 04-1] WikiGrok A/B test dry run (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [19:30:45] jdlrobson, dr0ptp4kt: i'm here now [19:30:55] kaldari|2: that's right, othe [19:31:02] … [19:31:28] phuedx: https://en.wikipedia.org/wiki/Othe [19:31:47] that's exactly what i meant, obviously! [19:32:26] Looks like a nice place, but I'm not sure it's relevent to the code ;P [19:32:50] (03PS3) 10Phuedx: Open WikiGrok A/B test to all users on alpha/beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172551 [19:32:52] (03PS8) 10Phuedx: WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 [19:33:13] i like to litter my documentation with places that i'd like to visit [19:33:18] what do you guys do for fun? [19:38:50] dr0ptp4kt: theoretically zero-rated? [19:39:05] mhurd: brion dbrant minor breaking change to the search API, not sure if it is relevant to apps: https://lists.wikimedia.org/pipermail/mediawiki-api-announce/2014-November/000074.html [19:39:20] * brion peeks [19:39:49] i don’t think that’ll break us [19:40:10] phuedx: So it looks like 1 important thing got left out of the acceptance criteria for that... [19:40:19] MaxSem: ^ [19:40:31] YuviPanda: Pretty sure our queries don't use that. [19:40:35] YuviPanda: Thanks for forwarding. :) [19:40:38] :D cool [19:41:21] phuedx: 'theoretically' meaning that the page is being served over a partner network. due to, for example, older networking gear, some operators can only zero-rate on cleartext http. [19:41:27] If a user has already seen one version of wikigrok as an anonymous user, we don't want them to see a different version as a logged in user. [19:42:36] dr0ptp4kt: is there already code to determine that? [19:43:12] Maryana: IT'S HAPPENING [19:43:32] Maryana: http://i.imgur.com/TSoPijc.png [19:43:50] heh. yeah.. [19:44:02] fun times [19:44:07] phuedx: yeah, it's in ZeroBanner. got a few minutes to do google hangout? [19:44:58] woah, is gerrit ok? [19:45:22] kaldari|2: that should be handled by only ever looking at the token to determine the bucket [19:45:33] wait... [19:45:57] phuedx: put up your public key under your office wiki userpage? [19:46:23] was that a question or a command? [19:46:45] COMMAND [19:46:48] seriously though, i can't login to gerrit :( [19:46:50] OPS NO REQUEST, ONLY COMMAND [19:46:54] oh, that's unrelated... [19:47:09] YuviPanda: https://office.wikimedia.org/wiki/User:Phuedx_(WMF) [19:47:18] i did that a while ago it'd seem [19:47:39] ah, maybe you already have access... [19:47:42] * YuviPanda checks [19:47:46] maybe i already do? [19:47:56] i don't remember actually requesting it though :/ [19:47:57] phuedx: Looks like for logged in users it's still looking at alpha/beta state: https://gerrit.wikimedia.org/r/#/c/171630/8/javascripts/modules/wikigrok/wikigrokuser.js [19:47:58] ah [19:47:58] no [19:48:02] you have deployment access [19:48:05] just nothing on stat [19:48:28] phuedx: We should probably get rid of the alpha/beta distinction and have everyone bucketed based on their token [19:48:53] phuedx: paperwork filed! You shall have it on Monday [19:49:00] <3 YuviPanda [19:49:04] :D [19:49:10] kaldari|2: that's mibad [19:49:11] DarTar: Did you see my question from earlier? [19:49:16] dbrant: i updated the etherpad (https://etherpad.wikimedia.org/p/in_article_search_query) for the in-article search query [19:49:25] that function isn't called anywhere other than in wikigrokuser.getversion [19:49:28] which isn't called :/ [19:49:29] kaldari|2: if you have an onboarding document, can you add 'get stat1003 and stat1002 access') to it? [19:49:30] mhurd: cool! [19:49:37] phuedx: ah [19:49:37] i'll remove it and it'll be a whole lot easier to read! [19:49:44] phuedx: good idear [19:49:58] i thought i was going to use it, then didn't [19:49:58] YuviPanda: sure [19:50:01] iirc [19:50:06] kaldari|2: sweet, thanks [19:50:13] hey kaldari|2, link? [19:50:26] dr0ptp4kt: i've got about 10 minutes, is that enough? [19:50:40] DarTar: Just asked where you have the A/B test stuff written up? [19:50:49] phuedx: i think so. i too have 10, i mean 9, minutes [19:51:34] kaldari|2: https://meta.wikimedia.org/wiki/Research:Mobile_microcontributions/WikiGrok [19:51:54] Thanks! [19:52:38] kaldari|2: any status update on the ETA to disable the test on enwiki? [19:53:04] DarTar: ? [19:53:23] We disabled WikiGrok on Friday [19:53:53] I thought that was not the case and we still had it live as of Monday [19:54:02] kaldari|2: nv then :) [19:54:22] (03PS1) 10Cmcmahon: QA: update beta checkbox locater and Gemfile [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172800 [19:55:03] (03CR) 10Cmcmahon: [C: 032] "maintenance" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172800 (owner: 10Cmcmahon) [19:55:40] (03Merged) 10jenkins-bot: QA: update beta checkbox locater and Gemfile [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172800 (owner: 10Cmcmahon) [19:57:24] (03PS4) 10Jdlrobson: Hygiene: Clearly identify when/how things get initialised. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172457 [19:57:46] phuedx: Ah, that's much nicer :) [19:58:12] kaldari|2: ? [19:58:20] https://gerrit.wikimedia.org/r/#/c/171630/8/javascripts/modules/wikigrok/WikiGrokAbTest.js [20:00:28] (03PS4) 10Jdlrobson: Hygiene: Put TableOfContents in own file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172458 [20:02:08] kaldari|2: yeah, splitting the two things up made sense [20:02:16] environment setup and experiment configuration [20:03:02] kaldari|2: i can't log in to gerrit right now, but i'll remove that method first thing tomorrow morning [20:05:27] (03CR) 10Kaldari: [C: 04-1] WikiGrok A/B test dry run (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [20:06:18] phuedx: So you can't push to gerrit at all right now? [20:06:40] kaldari|2: can't login [20:06:46] i could probably push [20:06:55] it's just 8:10 pm ;) [20:07:53] phuedx: No problem. I could clean it up for you in the meantime (if that's OK) [20:08:02] that's totally ok! [20:08:21] DOOO EEET! [20:08:34] okie poke [20:08:37] am orf for the night [20:08:42] g'night folks [20:14:51] kaldari|2: hey! can you silence your wikilove_collector.sh cron on stat1003? [20:15:44] YuviPanda: Sure, if you can tell me how [20:16:16] kaldari|2: ah, sure. edit the crontab to add '> /dev/null 2>&1' at the end [20:16:18] (sans quotes [20:16:19] ) [20:16:23] (from https://unix.stackexchange.com/questions/100722/how-do-i-completely-silence-a-cronjob-to-dev-null) [20:16:25] thanks [20:16:42] redirects stdout to /dev/null, redirects stderr to stdout, which is already going to stderr [20:17:11] YuviPanda: done [20:17:54] thanks! [20:19:38] phuedx: Shouldn't mw.cookie be $.cookie? [20:20:00] I'll fix.... [20:20:11] kaldari|2: see the code review!!! [20:20:20] there's comment discussion about mw.cookie [20:20:27] that's right, i irc in my sleep [20:22:30] phuedx: Ah, I see [20:22:37] hoom [20:22:43] i can't login to gerrit [20:23:10] there also doesn't appear to be a reset password link [20:24:39] phuedx: Is mw.cookie live on en.wiki or is it brand new? [20:24:52] it's been around for a long time [20:25:03] I mean for mobile [20:25:06] tfinc: ty! The 'get stat access' is now in the onboarding document for mobile web team, so this should be smoother next time around :) [20:25:17] YuviPanda: on wiki ? [20:25:19] ah, in that case, brand new ;) [20:25:36] tfinc: kaldari|2 said he's going to figure out how to add it, not sure if he's actually done it yet. [20:25:43] * YuviPanda didn't want to edit that himself, etc. [20:25:50] (03PS1) 10Jdlrobson: Fix editor switcher [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172811 (https://bugzilla.wikimedia.org/73057) [20:25:56] get i on the wiki kaldari|2 for it to be real :) [20:25:56] 3MobileFrontend / 3beta: Talk: Add discussion button doesn't look like other Overlay buttons - 10https://bugzilla.wikimedia.org/73243 (10Florian) a:3Florian [20:26:01] phuedx: I wanna try to push this stuff out tomorrow, so I might switch it $.cookie for now [20:26:08] kaldari|2: sure [20:26:28] i'll write a hygiene card to switch to mw.cookie [20:26:36] make sure you update the tests ;) [20:26:53] tfinc, YuviPanda: It's on the wiki :)_ [20:26:58] ah, cool :) [20:27:04] phuedx: thanks [20:27:14] i really should be sleeping :/ [20:27:14] thanks kaldari|2 [20:27:22] trying to get the gerrit thing sorted before i work tomo [20:27:23] phuedx: go to bed :) [20:28:03] thank kaldari|2 [20:29:29] Maryana: tfinc any idea whether this page serves any purpose these days? https://www.mediawiki.org/w/index.php?title=Mobile_Beta [20:30:16] jdlrobson: doubt it if we don't link to it from our beta opt in, what are the page view stats on it? [20:32:29] jdlrobson: ha, no, that's super outdated [20:38:26] (03PS1) 10Bmansurov: Fix WikiGrokDialogB button spinner [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172817 (https://bugzilla.wikimedia.org/73249) [20:38:35] (03CR) 10BearND: [C: 04-1] "This text based stuff lends itself to some unit testing." (034 comments) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/137684 (owner: 10Dbrant) [20:44:31] (03PS1) 10Dbrant: Fix reloading of page after editing. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172818 (https://bugzilla.wikimedia.org/73314) [20:51:12] right [20:51:15] i'm really going to bed now [20:51:17] ! [20:51:48] https://www.youtube.com/watch?v=V1tmkAFb_Os [20:53:48] (03PS1) 10Dbrant: Fix possible crash in full-text search. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172860 [21:03:15] maryana kaldari be there in a second [21:05:02] jdlrobson: https://gerrit.wikimedia.org/r/#/c/172582/3/i18n/en.json this won't work, if i use the message in a template? [21:06:32] (03PS3) 10MaxSem: Add wikidata descriptions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171767 [21:07:05] kaldari|2, ready to be merged ^^^^ [21:07:15] (03CR) 10jenkins-bot: [V: 04-1] Add wikidata descriptions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171767 (owner: 10MaxSem) [21:08:25] FlorianSW|2: you can do 'mobile-frontend-editor-anoneditwarning' => 'parse' if you use the ParsedMessageModule.. [21:09:02] jdlrobson: i do :) But it will result in html to be submitted as a parameter for the template, right? [21:09:08] And this will not work :( [21:09:23] FlorianSW|2: you can render html in templates [21:09:27] use three curly braces [21:09:31] {{{htmlString}}} [21:09:33] MaxSem: is this okay to merge https://gerrit.wikimedia.org/r/154873 [21:09:52] *facepalm* sorry, thanks jdlrobson :) [21:11:24] (03PS2) 10Jdlrobson: Add QUNIT_MODULE filter for grunt qunit command [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172573 (owner: 10Jhernandez) [21:11:28] (03CR) 10Jdlrobson: [C: 032] Add QUNIT_MODULE filter for grunt qunit command [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172573 (owner: 10Jhernandez) [21:12:09] (03Merged) 10jenkins-bot: Add QUNIT_MODULE filter for grunt qunit command [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172573 (owner: 10Jhernandez) [21:12:14] (03PS1) 10Bmansurov: Update watchstar icons [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172883 (https://bugzilla.wikimedia.org/73053) [21:12:23] (03PS6) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) [21:13:05] (03CR) 10jenkins-bot: [V: 04-1] Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [21:14:15] (03CR) 10BearND: [C: 04-1] "Works well for me. Just need to get rid of the circle." (032 comments) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 (owner: 10Dbrant) [21:16:01] (03PS7) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) [21:16:21] dr0ptp4kt: not sure exactly where we left off; do you still want to have a 10 minute-ish hangout to try to resolve the local stuff, or just forget it? [21:17:29] jhobs: i'll call you in a minute, ok? [21:17:29] (03CR) 10Kaldari: [C: 04-1] Add wikidata descriptions (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171767 (owner: 10MaxSem) [21:17:36] dr0ptp4kt: k [21:18:07] (03PS4) 10MaxSem: Add wikidata descriptions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171767 [21:18:38] (03CR) 10BearND: [C: 032] Fix possible crash in full-text search. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172860 (owner: 10Dbrant) [21:19:08] (03Merged) 10jenkins-bot: Fix possible crash in full-text search. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172860 (owner: 10Dbrant) [21:22:57] (03PS10) 10Dbrant: Face detection in lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 [21:24:40] (03PS1) 10Robmoen: Disable CentralNotice on beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172885 [21:28:33] (03CR) 10Jdlrobson: [C: 04-1] Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors (038 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [21:29:09] kaldari|2: resuming data QA for WIkiGrok based on the data we collected, lmk if there’s anything you want me to look into in particular [21:30:56] dbrant: one issue I've noticed with lead images is that the first item in the ToC doesn't scroll to the title anymore, just below the title since the title is now native [21:31:57] bearND: hmm, that is true. how about a bug? [21:36:08] "Assignments do not return a value. Thus the common mistake of writing if (i=0) when if (i==0) is meant will cause a compile-time error." [21:36:12] Holy fuck. [21:36:17] * Deskana shakes his head at Objective C. [21:36:27] 3Wikipedia App / 3Android App: first ToC entry doesn't scroll all the way up - 10https://bugzilla.wikimedia.org/73327 (10Bernd Sitzmann) 3NEW p:3Unprio s:3normal a:3None Now that we have lead images implemented, the first item in the ToC doesn't scroll to the title anymore, just below the title since... [21:36:32] dbrant: ^ [21:36:42] bearND: thx :) [21:41:48] (03CR) 10Bmansurov: "So, the problem is that the animation isn't working in alpha. In beta I see the animation when both watching and unwatching, but in alpha " [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172453 (https://bugzilla.wikimedia.org/73050) (owner: 10Bmansurov) [21:44:26] dbrant: http://i.imgur.com/VDWqDoQ.png (the circle is from the face detection) LOL [21:44:53] bearND: lol @_@ [21:51:03] (03CR) 10Bmansurov: Hygiene: Use spinner template instead of Icon (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172743 (https://bugzilla.wikimedia.org/73251) (owner: 10Phuedx) [21:54:17] (03CR) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors (038 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [21:55:10] (03PS11) 10BearND: Face detection in lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 (owner: 10Dbrant) [21:55:29] (03CR) 10BearND: [C: 032] Face detection in lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 (owner: 10Dbrant) [21:56:02] (03Merged) 10jenkins-bot: Face detection in lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/168773 (owner: 10Dbrant) [21:56:29] (03CR) 10Bmansurov: [C: 031] Add max height and scroll to reference drawer [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172424 (owner: 10Robmoen) [21:56:42] (03PS8) 10Florianschmidtwelzow: Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) [21:58:00] (03CR) 10Florianschmidtwelzow: [C: 04-1] Merge CtaDrawer with EditorOverlay Anonwarning for anonymous editors (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172582 (https://bugzilla.wikimedia.org/72852) (owner: 10Florianschmidtwelzow) [22:03:55] (03PS2) 10BearND: Fix reloading of page after editing. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172818 (https://bugzilla.wikimedia.org/73314) (owner: 10Dbrant) [22:04:16] (03CR) 10BearND: [C: 032] Fix reloading of page after editing. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172818 (https://bugzilla.wikimedia.org/73314) (owner: 10Dbrant) [22:04:25] (03Merged) 10jenkins-bot: Fix reloading of page after editing. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172818 (https://bugzilla.wikimedia.org/73314) (owner: 10Dbrant) [22:04:36] bearND: I'm posting that to Facebook. :P [22:05:44] Deskana: bearND: it saw the tire as an eye, but not the headlights?? [22:06:41] dbrant, bearND: Reminds me of this: https://encrypted-tbn2.gstatic.com/images?q=tbn:ANd9GcSmOWov-yk2FHQBCXnJG_RdBFxN53EuhkyvD1OYZiU91mxAx7_uC2yhFs8w [22:07:26] 3MobileFrontend / 3stable: Long gallery captions break viewport - 10https://bugzilla.wikimedia.org/73183#c3 (10Bahodir Mansurov) I agree, Florian. [22:07:43] σ_σ [22:08:25] jdlrobson: FYI, Sarah found a better room for the grooming salon [22:08:48] kristenlans: \o/ [22:09:31] (03PS1) 10BearND: Missing qq string [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172896 [22:09:57] (03CR) 10Jdlrobson: [C: 04-1] "This replaces https://gerrit.wikimedia.org/r/#/c/172453/ and fixes the bug there as well no?" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172883 (https://bugzilla.wikimedia.org/73053) (owner: 10Bmansurov) [22:10:55] MaxSem: is this okay to merge https://gerrit.wikimedia.org/r/154873 [22:13:42] (03CR) 10Jdlrobson: [C: 032] Add max height and scroll to reference drawer [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172424 (owner: 10Robmoen) [22:13:45] (03PS2) 10Bmansurov: Update watchstar icons [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172883 (https://bugzilla.wikimedia.org/73053) [22:14:16] (03CR) 10Bmansurov: "jdlrobson, no, this does not fix that bug. That is related to animations." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172883 (https://bugzilla.wikimedia.org/73053) (owner: 10Bmansurov) [22:14:24] (03Merged) 10jenkins-bot: Add max height and scroll to reference drawer [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172424 (owner: 10Robmoen) [22:14:32] bmansurov: it does fix the bug though :) [22:14:39] the animation is off due to the icons being misaligned :) [22:17:02] jdlrobson: when I tried it it didn't fix the bug. if it's working for you then great [22:17:03] (03CR) 10MaxSem: Fix wgMFShowRedLinks (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/154873 (owner: 10Florianschmidtwelzow) [22:17:30] jdlrobson, ^^ I have some questions [22:24:51] (03CR) 10Florianschmidtwelzow: Fix wgMFShowRedLinks (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/154873 (owner: 10Florianschmidtwelzow) [22:25:05] MaxSem: wgMFEnableBeta actually makes it possible to go into beta [22:25:19] (3rd parties don't have beta/alpha by default) [22:25:24] brrr [22:25:42] Woooo [22:25:45] so just your elseif comment :) [22:25:47] Android L images for Nexus devices are out [22:26:03] (03CR) 10Jdlrobson: [C: 04-1] "(Max has an else if comment that needs to be addressed)" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/154873 (owner: 10Florianschmidtwelzow) [22:27:56] (03CR) 10Bmansurov: [C: 04-1] "check out the templates files, some of them still use the "loading" class." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172744 (owner: 10Phuedx) [22:36:16] (03CR) 10Bmansurov: [C: 032] Fix editor switcher [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172811 (https://bugzilla.wikimedia.org/73057) (owner: 10Jdlrobson) [22:36:57] (03Merged) 10jenkins-bot: Fix editor switcher [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172811 (https://bugzilla.wikimedia.org/73057) (owner: 10Jdlrobson) [22:49:25] 3MobileFrontend / 3alpha: Table of contents icon in alpha is small and not aligned correctly - 10https://bugzilla.wikimedia.org/73115#c2 (10Bahodir Mansurov) Are you talking about the left icon? How big should it be? How should it be aligned? [22:50:56] (03PS2) 10Robmoen: ApiResponse: return success status and unit tests [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/172467 [22:51:44] (03PS1) 10Jdlrobson: Hygiene: MobileWebCta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172901 [22:51:46] (03PS1) 10Jdlrobson: Hygiene: Consistent file naming [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172902 [22:51:48] (03PS1) 10Jdlrobson: Hygiene: Separate init code path for events [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172903 [22:53:35] (03CR) 10Jdlrobson: [C: 04-1] "less/centralNotice.less is empty.. also we should probably do this in" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172885 (owner: 10Robmoen) [22:58:42] Deskana: you were right, it says "Similar pages" [22:58:50] (03CR) 10Jdlrobson: "Check with Max about caching implications." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172885 (owner: 10Robmoen) [23:01:14] (03PS1) 10BearND: Change Other meanings string back to Similar pages [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172907 [23:03:29] (03CR) 10Jdlrobson: [C: 04-1] Open WikiGrok A/B test to all users on alpha/beta (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172551 (owner: 10Phuedx) [23:04:00] dbrant|brb: any chance you could CR a couple of simple string changes so I can do a TWN sync? [23:04:57] (03CR) 10Bmansurov: WikiGrok A/B test dry run (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:06:00] (03CR) 10Yuvipanda: [C: 032] Missing qq string [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172896 (owner: 10BearND) [23:06:17] (03CR) 10Yuvipanda: [C: 032] Change Other meanings string back to Similar pages [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172907 (owner: 10BearND) [23:06:18] bearND_: ^ [23:06:25] (03Merged) 10jenkins-bot: Missing qq string [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172896 (owner: 10BearND) [23:06:37] (03Merged) 10jenkins-bot: Change Other meanings string back to Similar pages [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/172907 (owner: 10BearND) [23:06:50] YuviPanda: ty :) [23:06:54] :) yw [23:07:11] fwiw, I've always considered wording changes + doc changes self mergeable [23:07:40] (03PS1) 10BearND: Manual import of translations from TWN [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/172908 [23:07:51] (03PS9) 10Kaldari: WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:09:03] YuviPanda: even though Siebrand reviews string changes? [23:09:22] true, but these are fairly trivial ones. [23:09:33] introducing new ones, perhaps not, but those usually have code in them too [23:15:58] (03CR) 10Bmansurov: Hygiene: Goodbye mobile.stable.common (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171732 (owner: 10Jdlrobson) [23:16:34] (03PS1) 10Amire80: Remove unnecessary quotation mark [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172910 [23:17:19] (03PS10) 10Kaldari: WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:18:18] (03PS2) 10Jdlrobson: i18n: Remove unnecessary quotation mark [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172910 (owner: 10Amire80) [23:18:23] (03CR) 10Jdlrobson: [C: 032] i18n: Remove unnecessary quotation mark [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172910 (owner: 10Amire80) [23:19:29] (03CR) 10Kaldari: WikiGrok A/B test dry run (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:19:55] (03CR) 10Jdlrobson: [C: 04-1] Hygiene: Use spinner template instead of Icon (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172743 (https://bugzilla.wikimedia.org/73251) (owner: 10Phuedx) [23:20:13] (03CR) 10Kaldari: [C: 032] WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:20:46] (03CR) 10Jdlrobson: "Also as mentioned on irc, I'd rather we placed spinner.hogan with an Icon - additionalClassNames could be spinner loading and loadingMessa" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172743 (https://bugzilla.wikimedia.org/73251) (owner: 10Phuedx) [23:21:00] (03Abandoned) 10Kaldari: Open WikiGrok A/B test to all users on alpha/beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172551 (owner: 10Phuedx) [23:21:19] (03CR) 10Bmansurov: [C: 031] "Any reason to add the empty file "javascripts/loggingSchemas/init.js"" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172458 (owner: 10Jdlrobson) [23:22:25] (03Merged) 10jenkins-bot: i18n: Remove unnecessary quotation mark [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172910 (owner: 10Amire80) [23:22:27] (03Merged) 10jenkins-bot: WikiGrok A/B test dry run [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/171630 (owner: 10Phuedx) [23:27:24] (03CR) 10Bmansurov: [C: 031] Hygiene: Clearly identify when/how things get initialised. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172457 (owner: 10Jdlrobson) [23:33:07] (03PS2) 10Robmoen: Disable CentralNotice on beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172885 [23:34:32] ( ! ) Notice: Undefined variable: wmgUseWikibaseRepo in /vagrant/settings.d/puppet-managed/10-WikidataBuildResources.php on line 5 MaxSem [23:34:39] (03CR) 10Bmansurov: "Don't you have to rename those in javascript files (where schemas are required) too?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172902 (owner: 10Jdlrobson) [23:34:55] bmansurov: mm? [23:35:19] jdlrobson: for example, in WikiGrokDialog.js [23:35:21] (03CR) 10Robmoen: "No longer behind feature flag. On by default as Beta is 0.1% of mobile page views." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172885 (owner: 10Robmoen) [23:35:25] schema = M.require( 'loggingSchemas/mobileWebWikiGrok' ), [23:35:25] errorSchema = M.require( 'loggingSchemas/mobileWebWikiGrokError' ), [23:35:52] oh i see what you mean - yeh i guess i should you can -1 it and i'll fix that up [23:35:59] ok [23:36:10] (03CR) 10Kaldari: ApiResponse: return success status and unit tests (031 comment) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/172467 (owner: 10Robmoen) [23:36:12] (03CR) 10Bmansurov: [C: 04-1] Hygiene: Consistent file naming [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172902 (owner: 10Jdlrobson) [23:36:53] rmoen: I disagree with phuedx on https://gerrit.wikimedia.org/r/#/c/172467/1/includes/api/ApiResponse.php . Let me know what you think. [23:37:08] jdlrobson, https://gerrit.wikimedia.org/r/172912 [23:37:35] (03CR) 10Yurik: Re-implemented missing Zero Portal Lua function (036 comments) [extensions/ZeroPortal] - 10https://gerrit.wikimedia.org/r/171472 (owner: 10Yurik) [23:37:56] (03PS2) 10Yurik: Re-implemented missing Zero Portal Lua function [extensions/ZeroPortal] - 10https://gerrit.wikimedia.org/r/171472 [23:38:02] kaldari|2: Since it only returns false when the file is not set, I think it makes sense. Maybe I'm not understanding completely. [23:42:28] (03CR) 10Jdlrobson: [C: 032] Change "Read in another language" button to neutral button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172408 (owner: 10Bmansurov) [23:43:01] (03CR) 10Bmansurov: [C: 032] Hygiene: Separate init code path for events [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172903 (owner: 10Jdlrobson) [23:43:21] (03Merged) 10jenkins-bot: Change "Read in another language" button to neutral button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172408 (owner: 10Bmansurov) [23:45:17] (03CR) 10Bmansurov: [C: 032] Hygiene: MobileWebCta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172901 (owner: 10Jdlrobson) [23:45:35] 3MobileFrontend / 3Hygiene: Hygiene: categoryButton class is camel case not hypenated - 10https://bugzilla.wikimedia.org/73339#c1 (10Bingle) Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/E1vedzda [23:45:35] 3MobileFrontend / 3Hygiene: Hygiene: categoryButton class is camel case not hypenated - 10https://bugzilla.wikimedia.org/73339 (10Jon) 3NEW p:3Unprio s:3normal a:3Jon CSS names should be hyphenated. .categoryButton should be .category-button [23:45:56] (03Merged) 10jenkins-bot: Hygiene: MobileWebCta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/172901 (owner: 10Jdlrobson) [23:47:27] (03CR) 10Robmoen: ApiResponse: return success status and unit tests (032 comments) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/172467 (owner: 10Robmoen) [23:51:32] (03PS3) 10Robmoen: ApiResponse: return success status or die on EventLogging fail. [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/172467 [23:51:51] (03CR) 10jenkins-bot: [V: 04-1] ApiResponse: return success status or die on EventLogging fail. [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/172467 (owner: 10Robmoen) [23:52:10] 3Wikipedia App / 3Android App: first ToC entry doesn't scroll all the way up - 10https://bugzilla.wikimedia.org/73327#c1 (10Dan Garry) (In reply to Bernd Sitzmann from comment #0) > Now that we have lead images implemented, the first item in the ToC doesn't > scroll to the title anymore, just below the title... [23:52:22] mhurd, dbrant|brb, bearND_: You guys may be interested in this workshop at the dev summit: https://www.mediawiki.org/wiki/MediaWiki_Developer_Summit_2015#Using_Elasticsearch [23:52:31] mhurd, dbrant|brb, bearND_: Since we're heavy consumers of their API. :) [23:53:16] (03CR) 10Dr0ptp4kt: "About to review PS2." [extensions/ZeroPortal] - 10https://gerrit.wikimedia.org/r/171472 (owner: 10Yurik) [23:55:47] bearND_: you about? [23:55:54] mhurd: yes [23:56:23] bearND_: quick question, for wikidata query are you using “props=description”? [23:56:37] rather - “props=descriptions” [23:56:57] mhurd: yes, props=descriptions [23:57:12] bearND_: there’s not explicit “shortdescriptions” prop? [23:57:35] mhurd: not that I'm aware of [23:57:47] bearND_: cool thanks! [23:59:38] mhurd: in https://www.wikidata.org/wiki/Special:ApiSandbox (and selected wbgetentities) I only see descriptions