[00:27:19] YuviPanda, you're on the mobile apps team responsible for the iOS Wikipedia app right? [00:28:16] Krenair: yup, although I do just the android bits [00:28:37] do you know if there are any plans to support iOS 7? [00:28:49] Krenair: the 'current' app is dead. No work is done on it at all. [00:28:59] Krenair: new app is being built now. apps/ios/wikipedia [00:29:03] Krenair: and it supports only 6 and 7 :) [00:29:06] presumably the next one will support ios7? [00:29:08] and the styling is all 7ish [00:29:09] okay. [00:30:52] YuviPanda, any idea when the new one should be available? [00:31:07] Krenair: January end or february start, I'd believe [00:31:23] Krenair: although we make beta builds every other week and email mobile-l :) [00:31:57] YuviPanda, this is for someone else, I don't personally have any iOS device. Don't think I should be advertising beta builds to them [00:32:05] oh completely agreed, Krenair [01:35:57] Does the App Store have that issue where someone releases a buggy minor version and the reviews get flooded with negative comments? [01:36:29] still have that * [01:37:07] Gloria: yup, still does. [01:37:11] Android, at least [01:37:13] iOS, idk [01:37:42] Fun. [01:37:53] :) [01:37:56] * Gloria NP: "Defying Gravity" by Idina Menzel & Kristen Chenoweth from "Wicked (Original Broadway Cast Recording)" [05:44:51] (03PS1) 10Legoktm: Only show geonotahack beta feature if GeoData is installed [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100350 [17:15:24] (03PS1) 10Cmcmahon: hygiene and appearance [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100397 [17:33:28] (03PS1) 10Yuvipanda: Handle out of order network responses to search query [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/100404 [17:38:38] (03CR) 10Cmcmahon: [C: 032] "maintenance" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100397 (owner: 10Cmcmahon) [17:41:26] (03Merged) 10jenkins-bot: hygiene and appearance [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100397 (owner: 10Cmcmahon) [17:42:00] (03CR) 10Robmoen: "I agree also. thanks for feedback. I'll be working on this today." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99773 (owner: 10Robmoen) [17:42:32] (03CR) 10Robmoen: "I agree also. thanks for the feedback." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99776 (owner: 10Robmoen) [18:13:46] hey brion [18:14:01] hey hey [18:15:29] brion: still trying to join the hangout, it just seems to sit there and wiggle disconcertingly at me [18:15:58] jdlrobson, remind me, finally you're doing the interview, not me, right? [18:16:48] "Sorry, you have been logged out [probably in another window]. Please log in again." sonofa [18:17:09] kenanwang, jdlrobson now that im reading https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1484 more closely, i think this is a wontfix [18:17:16] brion: won't let me join either [18:17:21] ok i think i'm on now [18:17:23] keeps 'loading' forever [18:17:24] but i see nobody else [18:17:27] I'm in but now one is here [18:17:35] ohh [18:17:37] brion [18:17:39] kenanwang: I'm having issues connecting, not sure why [18:17:41] ok i see you kenanwang [18:18:06] brion: it keeps crashing my chrome :| [18:18:10] awjr: OK That's fine [18:18:24] kenanwang, jdlrobson afaict the functionality the guy is talking about on desktop is something project-specific; probably some magical css or a gadget/extension they're using to generate that; they would probably need to make whatever tool they're using support mobile - it's not something for us to deal with [18:18:26] cool [18:18:30] and... firefox just keeps showing me a 'loading' [18:18:42] brion: kenanwang I'm just going to mail my status in, Hangout is being a PITA today [18:19:02] ok [18:22:26] (03CR) 10Jdlrobson: [C: 04-1] "Max what are you thoughts on my comment about $wgMFNearby VS class_exists( 'GeoData' ) ?" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100350 (owner: 10Legoktm) [18:22:42] ^ MaxSem [18:24:52] jdlrobson, remind me, finally you're doing the interview, not me, right? [18:27:35] (03CR) 10MaxSem: "If people want to create GeoData alternatives they should pretty please avoid using the same API so avoid committing a mortal sin of eithe" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100350 (owner: 10Legoktm) [18:28:02] jgonera, jdlrobson, MaxSem, kaldari, kaity, kenanwang: note that our sprint kickoff is rescheduled for an hour earlier than usual and will be in R35 due to a scheduling conflict for kenanwang. the calendar invite should be updated, and i'll mail out an announcement as well [18:28:22] dr0ptp4kt, hi, any thoughts on my patch? [18:28:28] (03PS7) 10Cmcmahon: new test for uploading an image [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99690 [18:28:32] Thanks awjr and all! [18:28:34] (03CR) 10Cmcmahon: [C: 032] new test for uploading an image [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99690 (owner: 10Cmcmahon) [18:29:48] (03Merged) 10jenkins-bot: new test for uploading an image [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99690 (owner: 10Cmcmahon) [18:29:51] awjr: is 1pm the earlier time or the usual time? [18:30:01] kaldari: 1pm is the earlier time, usually it's at 2 [18:30:05] brion: anything from the standup I should knmow about? [18:30:08] mhurd: kenanwang ^ [18:30:12] awjr: thanks, I couldn't remember :) [18:30:15] :D [18:30:38] moment [18:33:33] yuvipanda: ok we're going to try closing out a few more little bits of the last sprint today [18:33:44] brion: okay. which bits? [18:33:55] yuvipanda: think we can do a stop-gap implementation of the history thumbnails doing a fetch on demand like we do in search? [18:34:12] brion: nope. repeated work and will actually take more time. [18:34:16] and having it expire records older than 30 days should pretty much do it [18:34:19] brion: MaxSem is on the mobileview implementation. [18:34:35] ? [18:34:37] brion: hmm, I'll think of a nice way to do that, yeah. [18:34:41] ok [18:34:59] MaxSem: pageimages in mobileview? awjr said it has been picked up this sprint :) [18:35:28] kickoff is in 2.5 hours [18:35:30] yuvipanda: sprint starts later today, but it's there: https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards?favorite_id=10341&view=Next+iteration+story+wall [18:35:32] brion: we can have a 'scheduled runner' thingy, and have it check in onCreate for certain circumstances and run [18:35:39] also, what's that card priority?:P [18:35:46] awjr: sweet! [18:36:11] MaxSem: idk how web team does it, but we've at least 2 featuresets blocked on it :) <- awjr [18:36:54] we can work around it but it offends yuvipanda's sense of good programming to do it the workaround way :) [18:37:09] very much so :P [18:37:19] and it will be nice to do it the proper way, saves us a request which is good on mobile networks [18:37:35] it makes the code a bit ugly, and involves multiple requests, and also some synchronizing since history is an infinite scroller [18:37:38] poor yuvi gets to do all the "bad network testing" for us ;) [18:37:44] hehe :P [18:37:57] brion: I've a 'bad networking' related patch in gerrit, btw [18:38:07] whee [18:38:34] bah, with the new UI style in Firefox nightly i'm having trouble telling Firefox and Chrome windows apart [18:38:39] the tabs look more similar [18:38:42] brion: haha, same for me :D [18:38:52] brion: I keep gmail and stuff open in chrome and firefox for everything else [18:38:52] and they have the same damn hamburger menu icon [18:39:14] i basically use chrome for Hangouts only :P [18:39:20] :P [18:39:22] cause hangouts still gets confused by multiple accounts [18:39:47] you'd think this would be like a core google apps feature they'd want to sell you on [18:40:37] kaldari, I shot you down:P [18:41:06] brion: heh [18:41:16] brion: so I found out that if I open hangouts from *google calendar*, it does not get confused [18:41:19] brion: if i open from gmail, it does [18:41:23] madness [18:41:41] MaxSem: :*( [18:42:20] *( [18:42:32] ( [18:44:01] MaxSem: Ah, looks like Tim has an idea in Bug 53945 [18:44:27] 😱 [18:45:38] brion: mm, smileys! [18:45:59] brion: https://en.wikipedia.org/wiki/Adam_tablet - I've one of these now [18:46:14] brion: a friend was going to throw them away, so I rescued 'em and put CM9 on it. [18:46:37] nice [18:46:59] MaxSem: Change abandoned. Poor little change, it never even had a chance. [18:47:10] brion: and I spend weekend working on github.com/yuvipanda/AndroLua - lets you type lua code on a desktop webbrowser, and *instantly* syncs it to your tablet so you can execute it [18:47:13] :*( [18:47:19] brion: much easier to try random coding in than typing on the tablet or using adb [18:47:44] brion: after building it out, I realized I can make it run on Glass too :D So type code on laptop, have it run on Glass. :D [18:47:45] niiice [18:47:48] haha [18:47:50] awesome [18:48:02] (03CR) 10Brion VIBBER: [C: 032 V: 032] Handle out of order network responses to search query [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/100404 (owner: 10Yuvipanda) [18:48:24] brion: it is using ShareJS to do the syncing, so tons of people can type in the code at the same time :) [18:48:28] s/the// [18:49:39] all right let's try updating to intellij 13... [18:49:54] "what's the worst that could happen?" [18:50:24] brion: "we could be writing Java. oh wait..." [18:52:16] woo it still builds and runs [18:53:04] (03CR) 10Brion VIBBER: [C: 032 V: 032] Quick fix for last schema update. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/99804 (owner: 10Mhurd) [18:54:44] MaxSem: I salvaged the convenience function though, since it will be useful for fixing some other disambiguation-related bugs: https://gerrit.wikimedia.org/r/#/c/100413/ [18:55:48] kaldari, profiling?:) [18:55:56] oh yeah! [18:56:01] one sec... [18:58:41] see if that's better [19:06:41] (03PS2) 10Robmoen: Added handling for captcha support on saveError [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99773 [19:07:05] awjr: Ack, my meeting with Emily and Stephen is also a 1pm [19:08:31] What is kenan's scheduling conflict? [19:16:21] yuvipanda: please mail out your status [19:17:08] kaldari: can you help me get error logging up and running? [19:17:22] i swear it's not working for me properly [19:18:44] tfinc: done [19:18:59] jdlrobson: stick these above your php_info call and see what happens... [19:19:01] error_reporting(E_ALL); [19:19:01] ini_set("display_errors", 1); [19:21:00] $wgShowExceptionDetails = true; [19:21:01] $wgShowSQLErrors = true; [19:21:02] $wgDebugDumpSql = true; [19:21:54] jdlrobson: $wgDebugLogFile = "/tmp/mediawiki.log"; [19:26:06] MaxSem: kaldari https://gerrit.wikimedia.org/r/#/c/99628/1 any ideas how to replicate this issue? [19:26:12] i can't get any notices [19:26:18] brion: do you have your Nexus 10 handy? [19:27:46] jdlrobson, $wgDevelopmentWarnings = true; [19:30:11] ahh MaxSem thanks :) [19:31:03] (03PS2) 10Jdlrobson: Fix logout link code to use Userlogout instead of alias [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99628 (owner: 10Aude) [19:31:09] brion: if/when you do, could you do me a favor and take a screenshot in landscape mode, fullscreen to the extent possible, of [[Encyclopedia]] in a private browsing tab while logged out? I'm trying to get a good high resolution screenshot for a print brochure. [19:31:29] (03CR) 10Jdlrobson: [C: 032] Fix logout link code to use Userlogout instead of alias [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99628 (owner: 10Aude) [19:31:44] ragesoss: wouldn't a retina screen thing also be as good? brion has one of those too :) [19:31:54] awjr: ^ [19:32:10] in a mtg kaldari [19:32:10] yes, a retina would work too. [19:32:21] kenan: ping [19:32:25] (03Merged) 10jenkins-bot: Fix logout link code to use Userlogout instead of alias [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99628 (owner: 10Aude) [19:32:38] where's kenan? [19:33:09] kaldari: terribly hard to catch him on IRC, sadly. [19:33:28] so it seems [19:34:00] kaldari: btw, your keyboard - is it cherry brown switches or cherry blue? [19:34:28] ? [19:34:43] what the heck is cherry blue? :) [19:35:06] cherries are red [19:35:09] :) [19:35:18] kaldari: http://www.daskeyboard.com/blog/mechanical-keyboard-guide/#keyswitches [19:35:41] wow [19:36:52] kaldari: blues are loud, browns aren't as loud while having a similar feel [19:37:20] must be blue [19:37:46] kaldari: you should try browns at some time. I used a blue one for a long time before I switched to browns, feels better. [19:57:11] lolz http://imgs.xkcd.com/store_news/store_gd_g1_QG5Z.png [19:57:13] (03PS1) 10Jdlrobson: Kill some php notices [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100423 [19:58:49] (03PS2) 10Robmoen: Add editing and preview messages to header area [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99776 [20:00:49] jdlrobson: lolz [20:01:30] (03PS1) 10Yuvipanda: Implement deleteAll as a variant of delete [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/100424 [20:08:09] kaldari: sorry about the scheduling conflict - i responded to your email [20:29:55] dr0ptp4kt, any updates? I need to take off earlier today and will be back later at night [20:30:16] yurik, no update yet, sorry. email, for the most part. about to dive in. [20:30:32] oki [20:43:41] (03PS1) 10Yuvipanda: Add a recurring task execution mechanism [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/100429 [20:58:54] yuvipanda: mhurd any blockers to finishing up sprint 19 ? [20:59:20] tfinc: mhurd sent an email to mobile-tech about Eventlogging. [20:59:47] Logging offline activity acceptance criteria... ? [20:59:49] tfinc: other than that just waiting for a feature from the mobile web team (it is in their current iteration - link in my standup status email) [21:00:15] tfinc: yeah, that. [21:00:29] tfinc: on track - just taking a bit longer. another hour i think [21:00:45] yuvipanda: at the standup we agreed that blocking on the web team for the feature would delay us longer then necessary and that we should just use the the two request approach to move forward [21:00:58] yuvipanda: also, kenan decided that event logging could wait till later [21:01:17] tfinc: it's a day's work, and will have to be scrapped in two weeks time. I would rather just keep it at where it is at right now (black squares) until we get mobile web feature done [21:01:29] tfinc: well, in that case there's one patch from me waiting for review from brion, and that should be it. [21:02:18] yuvipanda: do you know when the feature will be deployed by the mobile web team ? [21:02:54] tfinc: it's on this iteration for mobile web, so I expect at worst it should be available for us in our next iteration. [21:03:14] tfinc: MaxSem should be able to give a more accurate estimate if necessary. [21:05:24] yuvipanda: looking at their story wall https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards?favorite_id=10341&view=Next+iteration+story+wall i'm seeing it second to last in priorities [21:06:15] tfinc: the hack is error prone, slow, and will need to be duplicated wherever we want to display thumbnails. I'd rather just bump it until MF has that feature done. [21:06:40] implementing it after MF has that done is 1/10th the effort of doing it with the hack, and I'm not sure what doing the hack right now gives us. [21:06:51] thats a call for kenan to make then. waiting for him to join post sprint kick off [21:07:07] plus were already using it on ioS [21:07:59] tfinc: I should be able to convince kenan, I think. [21:08:32] tfinc: *shrug*. iOS is also architected differently, so perhaps it wasn't as error prone as Android's hack would be. [21:19:18] https://www.dropbox.com/s/qm8z6ijdarzzae5/header-02.png [21:20:32] YuviPanda: hey yuvi! quick question - how is the request for search result thumbs either a hack or error prone? [21:20:50] mhurd: search results don't scroll infinitely [21:20:53] mhurd: history page does [21:21:09] mhurd: so i'll have to wait for a scroll, request another batch, and also handle flings properly [21:21:26] rmoen: you about ? [21:21:42] http://etherpad.wikimedia.org/p/MobileWeb-IterationName [21:21:49] mhurd: and I'd also need to handle cancellations when you've scrolled completely past something that had already had a pageimages request fired off for [21:22:05] YuviPanda: but if you cache thumbs in the first place they'll be there for history [21:22:38] mhurd: fire off a request at save time? [21:23:40] mhurd: I'll also have to handle updating the thumb when the image is changed, and also that means an extra request for every page saved - not *that* bad. [21:24:44] mhurd: but true, it's not as terrible as doing the request when you view it. [21:25:36] YuviPanda: we'll have the image changed issue regardless i think. can solve that later? [21:27:31] mhurd: hmm, it's certainly a lot more elegant than the one I had in mind. [21:29:09] mhurd: okay, I think I can do it in a way that'll make minimal updates necessary later on. Thanks a lot for pointing out this way to me :) I was so focused on the other way that I had not thought of it :) [21:29:35] YuviPanda: you're welcome! :) [21:29:40] mhurd: :) [21:39:47] yuvipanda: mhurd : nice job sorting it out :) [21:39:56] tfinc: :) I'm working on it now. [21:44:41] jdlrobson: you missing anything else for the interview tomorrow ? [21:46:33] (03CR) 10Legoktm: "> Also, IMO, we should explicitly set $wgAllowUserJs and $wgAllowUserCss to false in MobileFrontend.php in the config variables section (s" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/96175 (owner: 10Jdlrobson) [21:50:58] kaldari: I think we sent everyone a notification for your birthday, except you :) [21:51:39] YuviPanda: Ah, that makes sense ;) [21:51:48] :D [22:10:21] (03PS5) 10Kaldari: WIP: New KeepGoing workflow [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98745 [22:11:16] (03CR) 10jenkins-bot: [V: 04-1] WIP: New KeepGoing workflow [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98745 (owner: 10Kaldari) [22:11:54] (03PS6) 10Kaldari: WIP: New KeepGoing workflow [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98745 [22:12:12] (03CR) 10JGonera: [C: 04-1] "Yep, that's much better ;) Unfortunately doesn't seem to work. I enabled captcha and when trying to save I've got:" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99773 (owner: 10Robmoen) [22:12:34] (03CR) 10jenkins-bot: [V: 04-1] WIP: New KeepGoing workflow [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98745 (owner: 10Kaldari) [22:14:00] jdlrobson, why do we have mobile.editor.styles? why can't those styles be in mobile.editor.common? [22:14:46] jgonera: not sure off top of my head - check git blame [22:15:44] jdlrobson, commit message doesn't mention anything, it's from "VE editing on tablets in alpha" [22:16:12] mmm might be to do with alpha/beta stuff? not really sure [22:16:59] (03PS7) 10Kaldari: WIP: New KeepGoing workflow [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98745 [22:21:20] (03CR) 10JGonera: [C: 04-1] Add editing and preview messages to header area (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99776 (owner: 10Robmoen) [22:22:27] jdlrobson, how do you get notices from https://gerrit.wikimedia.org/r/#/c/100423/1 to show up? [22:22:30] (03CR) 10Jdlrobson: "Little confused.." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99773 (owner: 10Robmoen) [22:22:54] (03PS2) 10Jdlrobson: Kill some php notices [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100423 [22:23:00] updated commit message jgonera :) [22:23:06] i was having that issue this morning :) [22:23:20] thanks! [22:25:45] (03CR) 10JGonera: [C: 032] Kill some php notices when $wgDevelopmentWarnings = true; [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100423 (owner: 10Jdlrobson) [22:26:22] (03Merged) 10jenkins-bot: Kill some php notices when $wgDevelopmentWarnings = true; [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100423 (owner: 10Jdlrobson) [22:27:06] jdlrobson, this also seems to give a notice: includes/specials/SpecialMobileHistory.php:10: protected $specialPageName = 'History'; [22:29:03] jgonera: on what url? [22:29:09] are you on latest core? [22:29:13] jdlrobson, every article page [22:29:15] let me check [22:29:16] mm [22:29:44] yep, newest core [22:30:34] or perhaps it's includes/MobileContext.php:292: $redirectUrl = SpecialPage::getTitleFor( 'History', $this->getTitle() )-> [22:33:44] (03CR) 10JGonera: [C: 04-1] Hygiene: Introduce isAlphaGroupMember (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99800 (owner: 10Jdlrobson) [22:45:11] (03CR) 10JGonera: [C: 04-1] "Some inline comments. It would be good to replace all the requests for edit tokens. I guess we get them for free now as I see this in page" (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99799 (owner: 10Jdlrobson) [22:49:09] (03CR) 10JGonera: [C: 04-1] "Is adding all this special casing really worth it just for preserving the position of the last edited bar? Not that many people are allowe" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/98034 (owner: 10Jdlrobson) [22:57:48] awjr: where are you guys at with the discussion on showcases ? [22:58:04] tfinc holding pattern until quarterly planning [22:58:07] k [22:59:33] tfinc: showcases and a handful of other process-related things have had some discussion on the list, in irc, in person, etc but no concrete decisions made. i figured best to let folks focus on the work at hand and discuss it all when we're all focussed on big picture stuff next week [23:07:42] jgonera: for captcha are you using confirmEdit ? [23:08:27] (03CR) 10Dr0ptp4kt: [C: 04-1] "Just one thing left to do." (031 comment) [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/100204 (owner: 10Yurik) [23:17:27] rmoen, to enable it for editing you need $wgCaptchaTriggers['edit'] = true; [23:17:43] jgonera: ahh thanks ;) [23:22:33] mmm yuvipanda, do you need a page image on main page? [23:22:51] MaxSem: would be nice to have [23:23:05] but not absolutely required :) [23:23:39] yuvipanda, I was thinking that MP is different and a PI might _not_ represent it adequately [23:24:00] MaxSem: it'll probably throw up the featured image? [23:24:00] there's no technical problem with adding it:) [23:24:15] (other than stale image:P) [23:24:35] (03CR) 10Robmoen: "Ouch, yeah this surfaces some problems with the mw base target class." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/99773 (owner: 10Robmoen) [23:24:42] MaxSem: ah, that [23:24:49] MaxSem: how cached are pageimages? [23:25:07] PI might not work very well on pages glued together from a bunch of templates because it depends on LinksUpdate [23:25:23] ah [23:25:38] and links don't get updated when a page simply gets re-rendered [23:25:51] MaxSem: meh, keep it on, for consistency's sake? We can conditionally turn it off later [23:26:05] yuvipanda, what about other users? [23:26:19] MaxSem: what other users? :P [23:26:23] *kidding, kidding* [23:26:40] those mystical ppl who 7use it but will never admit it:P [23:26:41] MaxSem: we can fix it in the future to have specific behavior for main page [23:26:50] :P [23:27:28] MaxSem: or just hide it client-side. [23:27:55] MaxSem: we only make a promise of a 'representative image', don't we? if we make it more representative in the future, nothin wrong with it! [23:28:29] * MaxSem imagines an xmas pic on new year [23:29:00] MaxSem: mf-page-image? [23:29:02] :D [23:29:13] * MaxSem bites yuvipanda [23:30:18] * yuvipanda applies band aid liberally [23:30:32] look at me, I'm just a bunch of band aids on top of other band aids! [23:30:40] lolololol [23:31:20] awjr: :D [23:31:27] MaxSem: what else do you propose? [23:31:58] awjr: I'm being chastised in other channels for not capitalizing the 'W' [23:31:59] heh [23:32:05] hahaha [23:32:26] YuviPanda, I'm seriously considering not returning the image for MP [23:32:40] when we fix it to be actually useful, we could add it [23:32:50] MaxSem: that's cool too, MaxSem. [23:34:19] YuviPanda, also - what about PI do you need? [23:34:33] a thumb with some width? [23:34:52] MaxSem: ideally, just the image name. So it can be re-used in different contexts. [23:35:02] I've code that calculates thumb URLs from names locally [23:36:06] omg herecy. what if we change the schema as part of thumbnails overhaul? [23:36:31] MaxSem: I bet there'll be redirects :P) [23:36:45] MaxSem: I don't want to make 2 HTTP requests every time I want to display an image. [23:37:08] lazy panda [23:37:21] okay [23:37:26] MaxSem: mobile networks! latency! ! [23:37:34] we can add another property later [23:37:42] MaxSem: two weeks ago in a bus my latency was about 15s for search :P [23:38:03] MaxSem: yeah. also is it always going to be on Commons? [23:38:34] no [23:38:41] images can be local [23:38:57] right [23:39:16] you know it only when resolving to a file URL [23:40:19] MaxSem: so, I'm going to suggest something :D [23:40:27] MaxSem: return the full URL? [23:40:30] I can munge it to get thumbs [23:40:32] err [23:40:41] * YuviPanda pukes a little into a bucket [23:40:46] okay, let's not do that. [23:41:01] * MaxSem feels pain from that self-inflicted facepalm [23:41:23] yeah, that was terrible even by Mediawiki standards [23:41:31] okay, how 'bout 2 props: name and thumb with a given size? [23:41:54] MaxSem: hmm, name, thumb with size & isLocal? [23:42:13] why isLocal? [23:42:23] then I'll construct URL to $wiki or commons [23:42:57] I'll just give you the thumb [23:43:00] URL [23:43:16] MaxSem: but I'm going to be storing these clientside for future use, at different sizes [23:43:40] s/one size/another/?:P [23:44:04] what if the title had '240px' in it? :P [23:44:13] still, better than fully construct them in the app... [23:44:26] it's almost the same as the one I puked for [23:44:27] replace one last occurence or something... [23:44:51] MaxSem: well, replace first occurance in last part of URL [23:44:57] that's actually mostly accurate [23:46:19] MaxSem: I could do that, but still a bit pukey don't you think? [23:46:56] MaxSem: why not 'oroginSite' or something of that sort? [23:48:20] ask Brion what he thinks?:) [23:48:40] MaxSem: he won't be around for a few hours [23:48:47] meh [23:48:51] MaxSem: how hard is it? other than the API design stuff [23:49:27] not hard at all. other than by doing this we increase the amount of evil in the world [23:49:42] MaxSem: well, that's a given simply because it'll require nonzero lines of PHP code [23:50:57] MaxSem: think we can do this in time for thursday cutoff? [23:51:05] we can [23:51:19] so I'd rather wait for Brion's thoughts [23:51:19] sweet to hear. [23:51:25] MaxSem: concur. [23:51:32] sehr gut [23:51:49] MaxSem: also, how much of a perf hit will this be? [23:52:01] unnoticeable [23:52:07] MaxSem: ah, nice :) [23:52:30] MaxSem: the onlyrequestedsections patch will hit enwiki this thursday, right? [23:52:39] or have I missed a week? [23:52:46] prolly [23:52:57] it will if it's already on mw.org [23:53:06] ...which it should [23:53:16] right. Just wanted to make sure that we didn't merge it two weeks ago. [23:57:18] hey MaxSem [23:58:33] yo kaity [23:58:33] is it possible to see a screenshot of your diff work? [23:58:33] (03PS1) 10MaxSem: Add PageImages to action=mobileview [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100507 [23:58:50] (03CR) 10MaxSem: [C: 04-2] "Waiting for finalized requirements." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100507 (owner: 10MaxSem) [23:59:18] MaxSem: sweet! Thanks :) [23:59:23] MaxSem: added a bunch of people as reviewers :)