[02:00:54] hello [04:02:07] Hello niedzielski-afk :-) Would you mind adding your usual timezone to https://phabricator.wikimedia.org/T115101? Looking forward to working with you! [04:02:53] sorry https://phabricator.wikimedia.org/T114358 [15:20:21] niedzielski: you around? [15:20:31] bgerstle: yeppers [15:20:44] what's up? [15:20:45] niedzielski: so, i'd like to verify that DISPLAYTITLE patch as well [15:21:05] bgerstle: oh right. maybe we can do a batcave or something and chat a bit? [15:21:08] yeah [15:21:10] that'd be great [15:21:16] you free now? [15:21:17] is now an OK time, or do you want to do it later? [15:21:19] yeah, sure [15:21:24] ok cool. omw [15:43:24] dbrant: do you know, when the fix for https://phabricator.wikimedia.org/T109983 goes out to prod? [15:44:31] FlorianSW: we're hoping to release a new Beta this week; so hopefully will make it to production a week after that. [15:44:54] FlorianSW: we're watching this one closely; it's the top source of negative reviews currently. [15:45:03] ah, ok, I'm sure you will sent a notice to mobile-l and mention the bug? [15:45:08] oops, ok :/ [15:45:13] yep [15:45:17] Hopefully this will solve the problem :) [15:45:43] I want to write the reporter of the bug an e-mail and ask, if the new version solves the problem :) (and this requires, that the fix is included in a prod release :P) [15:45:52] great, thanks dbrant! [15:47:34] niedzielski: any unexpected trouble with DISPLAYTITLE? [15:48:36] dbrant: none that i saw but my testing was very light [15:50:28] niedzielski: I tested it by hard-coding a Page displaytitle to contain the HTML entities that could be returned by the htmlspecialchars() function, and it looks like Android's fromHtml() handles them correctly. [15:51:16] dbrant: nice! if you want, share the test string and i'll dump it into my local vagrant [15:51:51] dbrant: probably not necessary but happy to try it out if you like [15:52:29] niedzielski: i'm pretty sure it's not necessary... [15:52:41] (famous last words) [15:52:46] :) [15:59:35] niedzielski: made some progress w/ vagrant, but QQ: where are the submodules cloned? [16:00:10] bgerstle: i think it's just in the root vagrant install: git submodule update --recursive --init [16:00:27] "root vagrant install" you mean the mediawiki-vagrant repo? [16:00:38] or do i need to ssh into the box? [16:00:52] bgerstle: yes, wherever you cloned the root repo, https://gerrit.wikimedia.org/r/mediawiki/vagrant [16:00:57] ok [16:00:59] thought i tried that [16:01:08] it's downloading the trusty vbox now, so might be a while [16:01:33] bgerstle: after you do a vagrant up, you can do ./setup.sh and "vagrant git-update", i think [16:01:56] just make sure you check out the right patch in the end [16:02:02] that's all stuff just to get the box up to date [16:02:07] i already did ./setup.sh [16:24:22] dbrant, hi, what do you think of the ideas for Nearby in https://phabricator.wikimedia.org/T113827 [16:26:14] yurik: love 'em. We'll get around to it eventually. I'll also loop in Kaity and Nirzar on that task, to give them inspiration for mockups. [16:27:39] dbrant, what is the timeline to get map nearby to production version? [16:27:52] (seems that the servers can support them just fine) [16:31:14] yurik: it needs just a bit more UX polish to be production-worthy. I'm guessing it will be two production releases from now. (with the next one being end of next week) [17:41:14] dbrant: what's the url for the android goals (phab/mw/other)? [17:41:47] dr0ptp4kt: you mean this guy? https://phabricator.wikimedia.org/tag/mobile-app-goals/ [17:42:13] niedzielski: is this the thing dbrant said he updated the other day? [17:42:58] dr0ptp4kt: hm, maybe a better question for dbrant after all :) i know he made this board quite recently, so i would _guess_ yes [18:03:27] dr0ptp4kt: hey, sorry; Yes, that Phab board is the one to look at for the Epics we're working on. There's also this spreadsheet where the goals had been drafted: https://docs.google.com/a/wikimedia.org/spreadsheets/d/13aLsGMMXirC5oYmc_N_6s-UW1ZjcidhcIKdgL05ZuMk/edit?usp=sharing [18:13:38] etonkovidova: when you are filing bugs can you put them on the product backlog board instead of the development board? https://phabricator.wikimedia.org/tag/wikipedia-ios-app-product-backlog/ [18:13:59] coreyfloyd: sure [18:14:28] etonkovidova: thanks! (it may have been different the last time you did QA -we made some process changes) [18:15:23] coreyfloyd: np! I was just trying to make "Regression" bugs more visible by putting them on Dev board ... [18:16:11] etonkovidova: ok - Josh probably doesn't look in the dev column to triage bugs - so if you put them there they may get ignored [18:16:38] coreyfloyd: I see :) [18:17:24] etonkovidova: if you think something is really important, I would ping Josh after you file it to make sure he addresses it quickly. [18:18:33] coreyfloyd: ok - understood. Usually, yes, if something is important enough, I talk to someone directly [18:36:00] dbrant, could you check the status of https://phabricator.wikimedia.org/T113829 [18:36:11] can we do something to help? [18:38:51] yurik: so basically, we would need to include the unique app install ID along with each tile request? [18:39:29] dbrant, i think something should be included in the useragent if i uderstood it correctly [18:39:35] looking... [18:41:34] dbrant, yes, i think we should force mapbox to pass wiki app UUID together with the mapbox's sdk version [18:41:52] i don't think its a good idea to have a different identifying method specifically for the app [18:42:06] because that would mean we would have to hack varnsh, and ops don't like that ) [18:42:51] yurik: so you mean put the UUID in the useragent, correct? [18:43:00] dbrant, yep [18:43:15] dbrant, ideally we should have both - mapbox info and uuid [18:43:20] I think that's probably doable. [18:44:02] dbrant, uuid should probably be the "main" UA, with the mapbox sdk version being included somewhere inside [18:44:18] just like you have it for all other requests [18:46:16] yurik: not sure what you mean there; we currently don't put the uuid in the UA... [18:46:39] dbrant, what do you have in the UA? [18:47:20] yurik: Right now the UA is of the form "WikipediaApp/[version] (Android [version])" [18:49:10] dbrant, i don't want to add any user-identifying info beyond what is being passed by other webrequests. So simply add MapboxSDK [version] right after Android [18:49:35] yes, we won't be able to track unique users, but that's preferable to sacrificing privacy [18:49:48] yurik: right; I think we can do that [18:50:26] dbrant, excellente, thanks! do you want to create a separate task for it? [18:50:44] yurik: yep, will do [18:50:54] thx! [19:09:11] dbrant, sec, i'm a bit confused - so you ARE sending UUID with all requests? [19:18:17] dbrant: can you say me, how you've created the new WP logo image in the app? https://github.com/wikimedia/apps-android-wikipedia/blob/master/app/src/main/res/drawable-xxhdpi/w_nav_mark.png :) [19:28:49] yurik: we are sending the UUID with all MW API requests. But we would need to add it to the MapBox requests separately, though. [19:30:06] FlorianSW: there was a .SVG file for it.... but now I can't seem to find it. [19:31:41] dbrant: damn :( It looks great and I would use it as a base for another project (without the W and another logo in it) :/ But my graphic experience isn't good enough to figure out, how to change the png in a nice way, without loosing the awesomness of the base. [19:32:29] FlorianSW: oh wait, it's this: https://github.com/wikimedia/apps-android-wikipedia/blob/master/icon-svgs/128/w_nav_mark.noflip.svg [19:33:55] dbrant: ahh, great! You're my star today :P [19:37:25] ;) [19:55:01] bearND: i'm ready to merge the sdk-23 patch, if you don't object [20:14:37] dbrant: I just merged it [20:14:47] o/ [20:34:56] mdholloway: is T116006 a regression of https://gerrit.wikimedia.org/r/#/c/247100/ or present in master? [20:37:01] niedzielski: unfortunately, i'm not sure, since i haven't figured out exactly how to repro it to check on master. [20:37:34] mdholloway: ok cool. i'll see if i have any luck [21:10:52] mdholloway bearND dbrant|brb: i'm noticing some WebView flickering on my (super slow) API 19 device. is this just me or are other folks seeing it? [21:11:16] only on long articles like obama [21:11:37] i think it's the hardware acceleration change :| [21:12:15] niedzielski: hmm, i haven't, but i have noticed some *severe* slowdown on my nexus 9 (which bernd isn't seeing on his) [21:16:38] mdholloway: can you try something for me on your nexus 9? remove android:layerType="hardware" from the webview in fragment_page.xml [21:16:53] mdholloway: it seems to have cleared up my flicker issue :( [21:18:58] mdholloway bearND dbrant|brb: something really weird is going on with the search box too. sometimes when i enter a single letter, several pour in at once [21:19:20] niedzielski: of course it's working fine now; i'll try next time it seems bogged down [21:19:43] mdholloway bearND dbrant|brb: and the backspace key seems to repeat all caharacters? [21:19:52] this is bananas [21:20:51] niedzielski: i definitely haven't seen anything amiss with the search box. [21:21:06] like i type "w", then "a", and it jumps to "wwa". i hit back space and get "ww". then backspace again and get "www" [21:21:25] mdholloway:guess i'll try rebooting... :) [21:26:32] ok, switched to master and did a reboot. both flickering and the keyboard issue are present :( [21:26:48] niedzielski: yep, i'm seeing the keyboard issue. likely a result of the text-filter patch [21:29:38] dbrant mdholloway: ok and just reconfirmed flickering resolves when i turn off hardware accel :| [21:33:43] i'm not, however, seeing flickering on my devices (api 18 and 19). [21:36:07] i haven't noticed any slowdown on my api 19 device, either. [21:37:49] my api 19 device is exceptionally slow. slow as in, i check my notifications while i wait for apps to launch [21:40:35] niedzielski: yeah, that sounds like how my nexus 9 has (usually) been lately. like, can count on an ANR popup just trying to expand the quick facts [22:07:11] mdholloway|afk dbrant|afk: hm, no flickering on my galaxy note ii (api 19) [22:15:45] dbrant: niedzielski: so the issue with keyboard is a known one? I filed it as https://phabricator.wikimedia.org/T116091 [Regression 2.0.131-alpha-2015-10-20] Attempting to enter search criteria produces 'The server could not complete the request' [22:26:01] etonkovidova: different keyboard issue i think [22:27:08] niedzielski: sigh - ok. Looks very dramatic... [22:27:44] etonkovidova: i have seen this issue you describe on the API 23 emulator i think after booting up. looks like it's not emulator only :( [22:28:15] niedzielski: sigh again :(