[16:17:19] <brion>	 mooooobile
[16:17:56] <MaxSem>	 weeeeere heeeeerre
[16:18:09] <brion>	 yayyyyyy
[16:20:35] <brion>	 now where'd that iPhone go...
[16:20:42] <brion>	 still reorganizing my home office
[16:45:27] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Hygiene: Start enforcing modules [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/74813 (owner: 10Jdlrobson)
[16:46:10] <grrrit-wm>	 (03Merged) 10jenkins-bot: Hygiene: Start enforcing modules [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/74813 (owner: 10Jdlrobson)
[16:49:54] <brion>	 github's not letting me push to Commons-iOS :(
[16:50:15] <grrrit-wm>	 (03CR) 10JGonera: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 (owner: 10Jdlrobson)
[16:50:53] <wikipedia-github>	 01[13Commons-iOS01] 15brion pushed 1 new commit to 06master: 02http://git.io/DUIirA
[16:50:53] <wikipedia-github>	 13Commons-iOS/06master 1461c9707 15Brion Vibber: Bump version to 1.0.8
[16:50:55] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Bug 52515: Correct reply message [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78346 (owner: 10Jdlrobson)
[16:51:01] <brion>	 yayyyy there it goes
[16:51:44] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Bug 51647: Allow right clicking on nearby [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78348 (owner: 10Jdlrobson)
[16:52:06] <grrrit-wm>	 (03Merged) 10jenkins-bot: Bug 51647: Allow right clicking on nearby [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78348 (owner: 10Jdlrobson)
[16:56:39] <MaxSem>	 hmm, no standup today?
[16:56:52] <jdlrobson>	 MaxSem: i was wondering the same
[16:56:56] <jdlrobson>	 not in calendar so i guess not
[16:57:17] <MaxSem>	 alright
[16:57:30] <MaxSem>	 anything anyone wanted me to hygienize?
[16:57:31] <grrrit-wm>	 (03CR) 10JGonera: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 (owner: 10Jdlrobson)
[16:57:55] <jdlrobson>	 jgonera: https://wmf.ci.cloudbees.com/job/MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox/146/ < did your touch event changes cause that?
[16:58:37] <jgonera>	 jdlrobson, does this test run in alpha? we run tests only in stable, right?
[16:58:50] <jdlrobson>	 jgonera: i'm not sure - i was wondering about that
[16:59:06] <Maryana>	 maxsem, no standups till monday
[16:59:13] <Maryana>	 arthur cancelled all our meetings till then :)
[16:59:28] <jdlrobson>	 jgonera: i asked
[16:59:34] <Maryana>	 does anybody know what's going on with caching?
[16:59:47] <jdlrobson>	 jgonera:  > my friend replaced the foundry square article image with his :) http://en.wikipedia.org/wiki/Foundry_Square
[17:00:18] <jgonera>	 jdlrobson, it looks like something weird is happening there: https://saucelabs.com/jobs/80fd82e5bbc1459281cc1a60eb11bbf3# no CSS whatsoever
[17:01:14] <jgonera>	 nice
[17:02:31] <MaxSem>	 Maryana, what caching?
[17:03:00] <grrrit-wm>	 (03PS2) 10Jdlrobson: Alpha talk: Stop blank replying [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 
[17:03:01] <Maryana>	 hmm, looks like you're not cc'ed on the ops email
[17:03:02] <jdlrobson>	 jgonera: that's odd
[17:03:08] <Maryana>	 jdlrobson can explain, maxsem
[17:03:21] <jdlrobson>	 Maryana: i can try
[17:03:30] <jdlrobson>	 we could just forward to Max :)
[17:03:37] <Maryana>	 yeah.. doing
[17:04:06] <jdlrobson>	 Maryana: done
[17:04:25] <Maryana>	 bah, you beat me!
[17:04:45] <Maryana>	 maxsem, {{you've got mail}}
[17:05:40] <MaxSem>	 meh, I saw that email before
[17:05:44] <brion>	 testing commons 1.0.8 on testflight....
[17:05:58] <MaxSem>	 can't categorise it as a caching problem though:)
[17:08:41] <brion>	 ok it runs on my 3gs, that pleases me
[17:08:45] <brion>	 let's try submitting to ye store
[17:12:25] <brion_>	 aaaand we're submitted
[17:13:23] <chrismcmahon>	 jgonera: jdlrobson beta labs was toast yesterday, it's better today
[17:13:47] <chrismcmahon>	 I'm re-running that build, it should be better again 
[17:31:18] <jgonera>	 chrismcmahon, thanks
[17:31:29] <chrismcmahon>	 jgonera: it's green again now
[17:32:03] <jgonera>	 ok, good, we didn't break anything ;)
[17:32:18] <chrismcmahon>	 jgonera: turns out beta had an intermittent hardware problem so various servers (apaches, caching, etc.) would fall over more or less randomly, and often.  seems to be fixed. 
[17:32:31] <chrismcmahon>	 jgonera:  not just beta actually, all of labs 
[17:32:48] <jgonera>	 I see
[17:34:19] <MaxSem>	 jdlrobson & Maryana - load on mobile caches hasn't changed so it's an analytics issue - not mobile
[17:34:32] <Maryana>	 that's what i figured
[17:34:34] <MaxSem>	 we keep serving the same number of requests:)
[17:35:36] <jdlrobson>	 jgonera: https://gerrit.wikimedia.org/r/#/c/78345/2 < can that be merged now
[17:35:59] <jdlrobson>	 jgonera: also did you see my message about clicking being unreliable in alpha now on desktop browser?
[17:36:22] <jdlrobson>	 MaxSem: did you see all the formatter related bugs?
[17:36:22] <jgonera>	 jdlrobson, I did not, explain "unreliable"
[17:36:45] <jdlrobson>	 jgonera: so load up alpha in your desktop browser and disable emulate touch events
[17:36:48] <jdlrobson>	 click the edit icon and click cancel
[17:36:50] <MaxSem>	 yep
[17:36:51] <jdlrobson>	 click edit icon again and it doesn't work
[17:36:58] <jdlrobson>	 you have to click several times before it does
[17:37:19] <grrrit-wm>	 (03CR) 10JGonera: [C: 04-1] "(2 comments)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 (owner: 10Jdlrobson)
[17:37:20] <MaxSem>	 jdlrobson, want me to work on them before estimation?
[17:37:30] <jdlrobson>	 estimation?
[17:37:54] <jgonera>	 jdlrobson, will check, thanks
[17:38:02] <jdlrobson>	 MaxSem: i think we should move away from using regular expressions to construct the section html
[17:38:09] <jdlrobson>	 it seems too unreliable and inflexible
[17:38:38] <jdlrobson>	 since the parser can work out edit section links surely we should be able to reuse that to identify sections?
[17:39:58] <MaxSem>	 jdlrobson, what you're proposing is to move the regex to parser, not get rid of it. you'll still need the regex for wrapping section content into divs
[17:40:40] <jdlrobson>	 but why MaxSem ? the ParserOutput has a function getSections which identifies toclevel and heading
[17:40:58] <jdlrobson>	 surely if it could work this out it would be trivial to identify what the text of sections is?
[17:41:52] <jdlrobson>	 DOMParse for example could look for a heading with the text / id for the section and then for the following section and thus work out the text of that section
[17:42:23] <jdlrobson>	 we could then move the logic for wrapping the sections into the skin itself rather than doing so in the formatter
[17:43:03] <MaxSem>	 jdlrobson, we don't have the information about sections' positions in output HTML so ol good regexes don't seem avoidable
[17:44:00] <jdlrobson>	 how does ParserOutput do it then?
[17:44:06] <brion>	 \o/ Status Waiting For Review 
[17:44:35] <MaxSem>	 it doesn't do it
[17:44:51] <MaxSem>	 its byteofset is position in source wikitext
[17:45:27] <MaxSem>	 getSections() returns just sections metadata, not actual section content
[17:46:03] <jdlrobson>	 MaxSem: but as you point out in Parser.php  it does know the byte offset
[17:46:13] <jdlrobson>	 could we not use the byteoffset ?
[17:46:15] <MaxSem>	 *source* offset
[17:46:33] <MaxSem>	 completely unrelated to what you want
[17:47:20] <jdlrobson>	 true but surely we should be able to identify sections using a much cleverer method then the existing one?
[17:47:23] <jdlrobson>	 using DOMParse
[17:48:36] <MaxSem>	 would be noticebly slower - find heading, insert a div after it, move all the tags until next heading into the div
[17:49:17] <MaxSem>	 we want to move away from parsing HTML, not add more of it
[17:50:10] <MaxSem>	 hmmm, a little brainstorming could help
[17:50:18] <jdlrobson>	 MaxSem: would there be an easy way to defer the wikification/parsing and wikify individual sections?
[17:50:31] <jdlrobson>	 e.g. getSections returns wikitext of sections and we parse it later
[17:50:44] <jdlrobson>	 (just curious i can imagine that might not be an option)
[17:51:25] <MaxSem>	 that would be a very serious change, too many things are expected to be parsed across sections: refs, HTML elements, etc
[17:52:32] <jdlrobson>	 :/
[17:52:37] <jdlrobson>	 mmm what else could we do
[17:53:14] <MaxSem>	 wait for the rest of the team to return, then get everyone into a hangout, grap a parser dev
[17:53:20] <MaxSem>	 s/grap/grab/
[17:53:21] <jdlrobson>	 MaxSem: if we move to lazy load sections as discussed during hackathon we are only going to need one section at a time
[17:53:47] <MaxSem>	 jdlrobson, still theyy *must* be parsed atltogether
[17:54:19] <jdlrobson>	 MaxSem: i spoke to gabrielle about this and his view was to stop marking up sections differently and use JS to identify sections - but I think this will be more trouble then it's worth - especially considering how we style things currently
[17:54:41] <MaxSem>	 I had messed enought with per-section parsed, could do nothing better than just create mobileview that always parses whole page then caches
[17:55:03] * jdlrobson  is looking at ParserSectionCreate hook

[17:55:32] <jdlrobson>	 "Use this to apply per-section modifications to HTML (like wrapping the section in a DIV)."
[17:55:39] <jdlrobson>	 couldn't we use that?!
[17:56:11] <jdlrobson>	 we could inject the div there..
[17:56:14] <jdlrobson>	 pre parsing
[17:57:22] <jdlrobson>	 ^ MaxSem 
[17:57:31] <MaxSem>	 1) we need to wrap several sections 2) that would result in us either modifying desktop output or fragmenting cache
[17:59:06] <jdlrobson>	 how would that fragment cache?
[17:59:27] <MaxSem>	 separate parses for mobile and desktop
[18:00:30] <jdlrobson>	 so fragmented between mobile and desktop?
[18:00:38] <jdlrobson>	 to be clear that wouldn't fragment cache on mobile
[18:00:47] <jdlrobson>	 (which is the current situation anyway no?)
[18:02:29] <grrrit-wm>	 (03PS4) 10Jdlrobson: Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 
[18:02:30] <jdlrobson>	 ^ jgonera can you merge now?
[18:02:33] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 (owner: 10Jdlrobson)
[18:02:57] <jgonera>	 jenkins says it needs a rebase
[18:04:57] <grrrit-wm>	 (03PS5) 10Jdlrobson: Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 
[18:05:01] <MaxSem>	 jdlrobson, parser cache fragmentation is much, much worse than HTTP proxy fragmentation
[18:12:03] <grrrit-wm>	 (03PS3) 10Jdlrobson: Alpha talk: Stop blank replying [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 
[18:12:56] <jdlrobson>	 jgonera: > this has been irritating me - https://gerrit.wikimedia.org/r/#/c/79079/
[18:13:00] <jdlrobson>	 thank is broken in production
[18:13:24] <jdlrobson>	 MaxSem: i'm not sure what to suggest then.. i just think there is a cleaner way to do this by catching it earlier
[18:13:38] <jgonera>	 jdlrobson, I saw this but have no idea about it, is Kaldari in the office or on vacation?
[18:13:45] <jdlrobson>	 vacation
[18:13:49] <jdlrobson>	 jgonera: you can test it..
[18:13:57] <jdlrobson>	 try clicking the thank button and watch the response
[18:13:59] <jdlrobson>	 it throws an error
[18:14:04] <jdlrobson>	 try with this one and you'll see it doesn't
[18:14:08] <jgonera>	 jdlrobson, you tested it?
[18:14:11] <jdlrobson>	 seems there was an issue in refactoring the code
[18:14:16] <jdlrobson>	 jgonera: yes.. and you can test it as well
[18:14:30] <jgonera>	 yes, but that makes me switch context which is super annoying
[18:14:37] <jdlrobson>	 rev = $user.data( 'user-id' ), < revision = user id?
[18:14:46] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Bug 52856: Fix the thank button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79079 (owner: 10Jdlrobson)
[18:14:46] <jdlrobson>	 it's clearly an error? :)
[18:15:08] <grrrit-wm>	 (03Merged) 10jenkins-bot: Bug 52856: Fix the thank button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79079 (owner: 10Jdlrobson)
[18:24:48] <jdlrobson>	 jgonera: https://gerrit.wikimedia.org/r/#/c/76025/5 < should be ready to go now?
[18:24:54] <jdlrobson>	 jgonera: i want to prove brandon wrong ;-)
[18:25:39] <jgonera>	 jdlrobson, my last comment in patchset 3 remains unanswered
[18:25:51] <jdlrobson>	 which one?
[18:25:59] <jdlrobson>	 i renamed it..
[18:26:11] <jdlrobson>	 ^ jgonera 
[18:26:16] <jdlrobson>	 M.define( 'logging/schemas/MobileWatchlistInteraction', {
[18:26:42] <jgonera>	 well, but the directory name is still the same
[18:26:57] <jgonera>	 I think it should be loggingSchemas/MobileWatchlistInteraction
[18:27:13] <jgonera>	 (both module, and directory)
[18:27:54] <jdlrobson>	 realllyyyy can you just merge and do a follow up commit?
[18:34:37] <jdlrobson>	 so jgonera to be absolutely 100% clear - all you want me to do is rename the directory and you'll +2 ?
[18:34:45] <jdlrobson>	 i don't want to back and forth on this
[18:35:06] <jgonera>	 yes, + consistent module name
[18:36:31] <jdlrobson>	 what does that mean?
[18:36:38] <jdlrobson>	 M.require ('loggingSchemas/MobileWatchlistInteraction' ) ?
[18:37:41] <jdlrobson>	 btw i find this sort of thing a pain in the arse to explain over irc so if we can get in the habit of fixing up these kinds of things ourselves in future that would be cool.
[18:39:31] <jdlrobson>	 should i also rename watchlist.js to MobileWatchlistInteraction.js ?
[18:40:09] <jdlrobson>	 and should it be mobileWatchlistInteraction or MobileWatchlistInteraction ?
[18:40:13] <jdlrobson>	 ^ jgonera 
[18:41:18] <jgonera>	 whatever you prefer, just consistent, alternatively, instead name the module loggingSchemas/watchlist
[18:42:35] <jgonera>	 jdlrobson, it is a pain indeed. I can fix it myself, but I'm afraid that later you'll keep doing things this way, and I'll keep fixing them...
[18:43:22] <jdlrobson>	 jgonera: you are more anal then me when it comes to naming. whether to fix up or i do won't make any difference to whether i do it right nor not :_
[18:43:23] <jdlrobson>	 :)
[18:43:44] <jgonera>	 I know I am ;)
[18:45:20] <grrrit-wm>	 (03PS6) 10Jdlrobson: Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 
[18:45:29] <jdlrobson>	 there ^
[18:45:41] * MaxSem  writes a function called doStuff() to entertain jgonera

[18:46:36] <jgonera>	 someone told me that they've seen a function called similarly somewhere in a real project, I guess it was YuviPanda 
[18:47:03] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 (owner: 10Jdlrobson)
[18:47:44] <grrrit-wm>	 (03Merged) 10jenkins-bot: Story 986: Log watchlist interactions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76025 (owner: 10Jdlrobson)
[18:48:20] <jdlrobson>	 jgonera: https://gerrit.wikimedia.org/r/#/c/79160/ < this would make Maryana very happy
[18:48:59] <Maryana>	 yay!
[18:49:53] <jgonera>	 jdlrobson, not sure if overriding config vars like this is a good practice
[18:50:14] <jdlrobson>	 jgonera: we did it this way before...
[18:50:23] <jdlrobson>	 and jgonera the MobileFrontend.php documentation says this happens
[18:51:03] <jdlrobson>	 "Whether or not to show the upload CTA to logged out users.\n * Note there are a couple of exceptions:\nit will be overriden in beta and alpha modes of the site"
[18:51:13] <jgonera>	 oh, ok
[18:51:44] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Story 920: Force upload CTA on beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79160 (owner: 10Jdlrobson)
[18:52:03] <grrrit-wm>	 (03Merged) 10jenkins-bot: Story 920: Force upload CTA on beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79160 (owner: 10Jdlrobson)
[18:52:05] <jdlrobson>	 thanks jgonera :)
[18:52:21] <jdlrobson>	 jgonera: eventually that config variable should die with fire
[18:52:27] <jdlrobson>	 as soon as we've worked out how to stop the selfies
[18:52:41] <jdlrobson>	 MaxSem: did you take another look at https://gerrit.wikimedia.org/r/#/c/76271/ ?
[18:52:45] <jgonera>	 yep, true
[18:53:03] <jdlrobson>	 also MaxSem volunteer patch that would be good to be merged - > https://gerrit.wikimedia.org/r/#/c/77999/
[18:54:28] <grrrit-wm>	 (03CR) 10MaxSem: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77999 (owner: 10Pypt)
[18:55:33] <jdlrobson>	 jgonera: https://gerrit.wikimedia.org/r/#/c/78345/1 can be merged now?
[18:55:55] <jdlrobson>	 MaxSem: good catch :) If I fix that up on his behalf can we merge?
[18:56:05] <MaxSem>	 yup
[18:57:55] <jgonera>	 jdlrobson, the thing I wanted is $( this ).removeClass( 'error' ); -> $textarea.removeClass( 'error' ); but whatever, it works either way, it's just that $ (jQuery) is not really needed there
[18:58:19] <jdlrobson>	 jgonera: can you fix up?
[18:58:42] <jgonera>	 yes...
[19:00:55] <jgonera>	 jdlrobson, I'm testing it now, and actually removing the error color doesn't work either way
[19:01:27] <grrrit-wm>	 (03PS3) 10Jdlrobson: Render the list of categories as a normal subheading for logged in mobile users. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77999 (owner: 10Pypt)
[19:01:30] <jdlrobson>	 ^ MaxSem 
[19:02:14] <jdlrobson>	 jgonera: o_O are you focusing?
[19:02:25] <jgonera>	 clicking should do it right?
[19:02:38] <grrrit-wm>	 (03CR) 10MaxSem: [C: 032] Render the list of categories as a normal subheading for logged in mobile users. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77999 (owner: 10Pypt)
[19:03:00] <grrrit-wm>	 (03Merged) 10jenkins-bot: Render the list of categories as a normal subheading for logged in mobile users. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77999 (owner: 10Pypt)
[19:04:09] <jgonera>	 I'm puzzled
[19:21:54] <jdlrobson>	 jgonera: yeh… i'm not sure what's going on
[19:21:58] <jdlrobson>	 let me take another look
[19:22:05] <jdlrobson>	 i'm just going to go out and grab some lunch though..
[19:22:07] <jgonera>	 jdlrobson, got it, I'm just tired
[19:22:13] <jdlrobson>	 also haha http://blink.htcsense.com/web/articleweb.aspx?regionid=4&articleid=12044203 :)
[19:23:19] <jgonera>	 wtf
[19:23:23] <grrrit-wm>	 (03PS4) 10JGonera: Alpha talk: Stop blank replying [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 (owner: 10Jdlrobson)
[19:23:46] <grrrit-wm>	 (03CR) 10JGonera: [C: 032] Alpha talk: Stop blank replying [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 (owner: 10Jdlrobson)
[19:24:33] <grrrit-wm>	 (03Merged) 10jenkins-bot: Alpha talk: Stop blank replying [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78345 (owner: 10Jdlrobson)
[19:53:29] <wikipedia-github>	 [13Commons-iOS] 15montehurd opened pull request #118: Fix for potential issue with debug mode in release builds. (06master...06debugMode)  02http://git.io/G-bQxg
[20:17:38] <grrrit-wm>	 (03PS1) 10Jdlrobson: Hygiene: Drop mf- prefix from less files [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79425 
[20:18:44] <grrrit-wm>	 (03PS2) 10Jdlrobson: Bug 52515: Correct reply message [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78346 
[20:18:51] <grrrit-wm>	 (03CR) 10Jdlrobson: [C: 032] Bug 52515: Correct reply message [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78346 (owner: 10Jdlrobson)
[20:19:28] <jdlrobson>	 MaxSem: when is next deployment? tuesday?
[20:19:33] <grrrit-wm>	 (03Merged) 10jenkins-bot: Bug 52515: Correct reply message [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/78346 (owner: 10Jdlrobson)
[20:19:40] <MaxSem>	 apparently:P
[20:27:59] <jdlrobson>	 lolz https://en.wikipedia.org/wiki/Butt_Hole_Road
[20:28:47] <jdlrobson>	 MaxSem: I was going to take a look at this https://bugzilla.wikimedia.org/show_bug.cgi?id=40678
[20:29:25] <MaxSem>	 hmm
[20:29:53] <MaxSem>	 I was looking into it some time ago, but encountered some architectural difficulty
[20:30:02] <MaxSem>	 wonder if it's been resolved by now
[20:30:19] <jdlrobson>	 i was thinking we could render the language section depending on whether action=language is present in the query string
[20:30:26] <MaxSem>	 can't tell for sure cuz don't remember what ti was:P
[20:30:52] <MaxSem>	 ehm, another action|?
[20:31:01] <jdlrobson>	 not good?
[20:31:30] <jdlrobson>	 i was just thinking if you did it in the skin you could do a simple if / else for whether to render the article or just the language section
[20:31:32] <MaxSem>	 there are conflicting views in developer community regarding this - some believe that actions should die in favor of special pages
[20:31:58] <MaxSem>	 I'm undecided myself
[20:32:21] <jdlrobson>	 i think generally query string parameters suck
[20:32:31] <jdlrobson>	 but since there is no way to do sub pages in a programmatic way..
[20:32:39] <jdlrobson>	 (since they are valid wiki titles)
[20:33:23] <MaxSem>	 yep
[20:34:17] <MaxSem>	 and since Special:Languages/VeryLongTitile can be over 255 bytes, it doesn't look like we have a choice:)
[20:34:37] <jdlrobson>	 :/
[20:34:46] <jdlrobson>	 let me just have a quick hack and see what's possible
[20:39:25] <jdlrobson>	 ergg actions are nasty
[20:41:05] <jdlrobson>	 MaxSem: care to chip in on https://bugzilla.wikimedia.org/show_bug.cgi?id=51277 ?
[20:47:29] <MaxSem>	 jdlrobson, "cookies are good, you should use them"?
[20:47:43] <jdlrobson>	 :-) try it..!
[20:47:56] <jdlrobson>	 we should probably stop the redirect if useformat=desktop is in the url though?
[20:49:26] <MaxSem>	 seriously, I can't think of any way to do this without cookies
[20:53:17] <grrrit-wm>	 (03PS1) 10Jdlrobson: Hygiene: Use our own custom reset file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79431 
[20:55:30] <jdlrobson>	 MaxSem: i replied
[21:08:18] <grrrit-wm>	 (03CR) 10Jdlrobson: [C: 032] Regression: slightly wrong styling in uploads nag overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76227 (owner: 10JGonera)
[21:09:05] <grrrit-wm>	 (03Merged) 10jenkins-bot: Regression: slightly wrong styling in uploads nag overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/76227 (owner: 10JGonera)
[21:11:23] <grrrit-wm>	 (03PS2) 10Jdlrobson: Hygiene: Use our own custom reset file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79431 
[21:12:29] <grrrit-wm>	 (03PS3) 10Jdlrobson: HTTPS [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:12:34] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Use HTTPS link rather than HTTP in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:12:36] <grrrit-wm>	 (03CR) 10Jdlrobson: [C: 032] Use HTTPS link rather than HTTP in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:12:39] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Use HTTPS link rather than HTTP in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:12:56] <jdlrobson>	 MaxSem: ^ haha i thought this was a big HTTPS change so i didn't touch - you should have been more specific in your commit message :)
[21:13:03] <jdlrobson>	 can you rebase it?
[21:13:07] <MaxSem>	 on it
[21:17:12] <grrrit-wm>	 (03PS4) 10MaxSem: HTTPS [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 
[21:19:49] <grrrit-wm>	 (03PS3) 10Jdlrobson: Hygiene: Use our own custom reset file [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/79431 
[21:20:36] <grrrit-wm>	 (03PS5) 10Jdlrobson: HTTPS [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:21:00] <grrrit-wm>	 (03PS6) 10Jdlrobson: Use HTTPS links rather than HTTP ones in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:21:06] <grrrit-wm>	 (03CR) 10Jdlrobson: [C: 032] Hygiene: Use HTTPS links rather than HTTP ones in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:21:08] <MaxSem>	 thanks jdlrobson
[21:21:22] <jdlrobson>	 brb i needs coffee
[21:23:07] <grrrit-wm>	 (03Merged) 10jenkins-bot: Hygiene: Use HTTPS links rather than HTTP ones in comments [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/77560 (owner: 10MaxSem)
[21:58:03] <superm401>	 Someone here may want to comment at https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28proposals%29#Short_articles_for_people_with_slow_Internet.3F
[21:58:17] <superm401>	 In particular, I think the mobile skin already works the way they mentioned (i.e. loading sections on demand).
[22:14:46] <brion>	 superm401: as i recall the sections are all there, but they're kept hidden.
[22:14:56] <brion>	 so that means some network cost (but compressed, text isn't that much)
[22:15:11] <brion>	 but it …. may or may not prevent loading of images in collapsed sections
[22:15:12] <superm401>	 Oh, I thought there were separate network calls for them.
[22:15:25] <superm401>	 But maybe those are all at page load, just staggered?
[22:15:26] <brion>	 only in one of th ebeta modes, not sure if we're actively using that still
[23:39:37] <jdlrobson>	 superm401: sorry just saw your conversation around sections but missed the start of the conversation want to elaborate ?
[23:40:39] <superm401>	 See https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28proposals%29#Short_articles_for_people_with_slow_Internet.3F
[23:40:47] <superm401>	 The question is whether the mobile site ever loads sections on demand.
[23:40:52] <jdlrobson>	 ahh ok so yeh we've talked numerous times about doing this
[23:40:59] <jdlrobson>	 i think MaxSem is now onboard
[23:41:08] <jdlrobson>	 we currently experiment with lazy page loading in the mobile alpha version
[23:41:18] <jdlrobson>	 this also only injects sections into the DOM when you click a section
[23:41:59] <superm401>	 jdlrobson, interesting, you may want to reply even though it's in alpha.
[23:42:00] <jdlrobson>	 but ideally we'd like to serve a page with just the lead section and h2s linking to just the content of that section and hijack with javascript those section headings so when clicked they pull the content from the api
[23:42:06] <superm401>	 It's an opportunity to tell people relevant stuff you're working on.
[23:42:11] <jdlrobson>	 superm401: will do
[23:42:35] <superm401>	 How is the second thing ("ideally...") different from "injects sections into the DOM"?
[23:46:14] <jdlrobson>	 superm401: because at the moment we only do this on ajax page loads
[23:46:45] <jdlrobson>	 a page served by http currently doesn't do this as the sections are already served in the page
[23:46:55] <superm401>	 jdlrobson, I see.
[23:46:56] <jdlrobson>	 as there is no fallback that performs well (currently)
[23:47:19] <superm401>	 Right, that makes sense.  Probably would have to do tricky user-agent stuff.