[11:30:03] (03CR) 10JGonera: ""for instance it doesn't tell me what to do with the last modified bar"" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103618 (owner: 10Jdlrobson)
[12:10:35] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100919 (owner: 10Jdlrobson)
[13:47:14] (03CR) 10JGonera: [C: 04-1] "Just a minor concern." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103948 (owner: 10Jdlrobson)
[13:51:00] (03CR) 10JGonera: [C: 031] Replace usage of SpecialPage::getTitle with getPageTitle [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103953 (owner: 10Legoktm)
[13:54:37] (03CR) 10JGonera: [C: 04-1] "What about EditorApi?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103887 (owner: 10Kaldari)
[14:02:27] (03CR) 10JGonera: [C: 04-1] "Two minor things, other than that looks OK." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103570 (owner: 10Jdlrobson)
[14:07:00] (03CR) 10JGonera: [C: 031] "Fewer files, less confusion." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102853 (owner: 10Jdlrobson)
[14:09:44] (03CR) 10JGonera: [C: 04-1] Remove tap highlight on section headings (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103767 (owner: 10Jdlrobson)
[14:10:05] (03PS7) 10JGonera: Remove tap highlight on section headings [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103767 (owner: 10Jdlrobson)
[15:30:18] (03CR) 10Siebrand: "i18n reviewed. Left inline comments." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102043 (owner: 10Jdlrobson)
[16:01:50] (03CR) 10Brion VIBBER: [C: 032 V: 032] Make it possible for results to be read multiple times [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/103891 (owner: 10Yuvipanda)
[16:03:32] (03CR) 10Parent5446: Tool: Blow Kaldari's code review mind (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103884 (owner: 10Jdlrobson)
[16:14:23] (03CR) 10Brion VIBBER: "Ah that explains it :D" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/103626 (owner: 10Yuvipanda)
[16:19:46] jgonera, what's the difference between modules/editor and modules/editorNew?
[16:20:30] MaxSem, editor is the old overlay design + common files such as api abstraction, editorNew is what we have only in beta/alpha, i.e. new overlay design and visual editor
[16:21:05] in near future I hope they'll both get merged (when we push new overlays to stable)
[16:21:22] aka new will become the only
[16:23:47] yes, plus the common stuff from old (api abstraction)
[16:42:39] (03PS1) 10MaxSem: Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052
[16:43:28] (03CR) 10jenkins-bot: [V: 04-1] Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052 (owner: 10MaxSem)
[16:43:38] dafuuq?
[16:44:40] MaxSem, onevar
[16:44:54] I'm not a fan of it, but that's what we use
[16:45:01] stupid conventions of a stupid language
[16:45:06] ;)
[16:45:20] it's arguably less stupid than PHP
[16:45:31] (03PS2) 10MaxSem: Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052
[16:46:00] even if it is, the margin is really small;)
[16:46:27] (03CR) 10jenkins-bot: [V: 04-1] Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052 (owner: 10MaxSem)
[16:47:27] (03PS3) 10MaxSem: Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052
[17:32:01] (03CR) 10JGonera: [C: 04-1] "Happy to merge when small issues from OverlayNew.less are resolved." (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/101428 (owner: 10Jdlrobson)
[17:37:57] (03CR) 10Jdlrobson: [C: 04-1] "-1ing till I hear back from @Hooman to push down the queue (also this patch needs a rebase)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102004 (owner: 10Hoo man)
[17:39:33] (03CR) 10JGonera: [C: 04-1] "Very small nitpicks. Also (just suggestions, not required for this to be merged):" (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103854 (owner: 10Jdlrobson)
[17:40:20] (03PS2) 10Jdlrobson: Replace usage of SpecialPage::getTitle with getPageTitle [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103953 (owner: 10Legoktm)
[17:41:27] (03CR) 10Jdlrobson: [C: 032] Hygiene: Replace usage of SpecialPage::getTitle with getPageTitle [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103953 (owner: 10Legoktm)
[17:42:01] (03Merged) 10jenkins-bot: Hygiene: Replace usage of SpecialPage::getTitle with getPageTitle [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103953 (owner: 10Legoktm)
[17:43:31] (03CR) 10MaxSem: [C: 032] Replace usage of SpecialPage::getTitle with getPageTitle [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/104007 (owner: 10Legoktm)
[17:43:49] (03Merged) 10jenkins-bot: Replace usage of SpecialPage::getTitle with getPageTitle [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/104007 (owner: 10Legoktm)
[17:44:38] (03CR) 10MaxSem: Tablet detection support (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/92036 (owner: 10MaxSem)
[17:46:34] (03PS3) 10MaxSem: Tablet detection support [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/92036
[17:47:41] (03CR) 10Jdlrobson: [C: 032] Use main page mode when appropriate for previews (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052 (owner: 10MaxSem)
[17:48:16] (03Merged) 10jenkins-bot: Use main page mode when appropriate for previews [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104052 (owner: 10MaxSem)
[17:53:14] (03CR) 10Jdlrobson: "Thanks for the code reviews - I just knocked this up with not much thought about code quality but I'll make sure it gets polished up as su" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103884 (owner: 10Jdlrobson)
[17:59:16] (03CR) 10JGonera: [C: 04-1] "Thanks for working on this, looks interesting. Comments inline." (034 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[18:16:01] hey violetto are you around?
[18:17:10] jdlrobson, kaldari, MaxSem do you need anything from me today?
[18:17:29] jgonera, I wanted to discuss templates
[18:17:29] nope
[18:17:40] although any +2s you have left would be appreciated ;-)
[18:19:15] jgonera: are you working on the header update?
[18:19:21] jdlrobson, jgonera, kaldari - any comments on templates? any questions why I chose Twig?:)
[18:19:25] kaity, jdlrobson is working on it
[18:19:46] cool thanks
[18:19:48] jdlrobson, there's a few patches close to a +2, but they require small fixes
[18:19:52] MaxSem: did you see the email about Swig?
[18:20:00] yep
[18:20:09] weapons race escalates
[18:20:09] MaxSem, are the results somewhere?
[18:20:39] jgonera, I emailed the code and performance results
[18:21:01] jdlrobson, I'll give your gerrit script a try on Monday ;)
[18:21:18] MaxSem, where to?
[18:21:23] jdlrobson: moiz and I will update that header spec today to include a search icon
[18:21:28] thanks kaity
[18:21:34] jgonera: you'll love it :D
[18:21:44] MaxSem: I only saw the results for Twig. Have you also profiled any PHP implementations of Mustache?
[18:21:56] jdlrobson: we're also proposing to replace just the number for notifications with a bell icon + number
[18:22:09] also, kaity, I guess the hamburger icon changed slightly too, can you provide an SVG with that icon too?
[18:22:21] jgonera: sure
[18:22:46] MaxSem, can't see it on mobile-tech or inbox
[18:23:23] jgonera, "Preliminary research results: Twig"
[18:25:43] MaxSem, ok, I see it
[18:25:57] We should probably profile one of the PHP implementations of Mustache as well for comparison.
[18:26:01] so, MaxSem, jdlrobson, kaldari should we do a similar POC with mustache?
[18:26:04] right
[18:28:51] MaxSem: any thoughts on that?
[18:28:51] kaity: a change to echo was not a part of the original spec… you know how i hate surprises ;-)
[18:29:04] jgonera: MaxSem yeh a similar POC with mustache would be great
[18:29:17] that's why I asked "any questions why I chose Twig?"
[18:29:19] ;)
[18:29:28] would give a great comparison
[18:29:35] (03PS28) 10Dr0ptp4kt: Experiment: Special:MobileWebApp and manifests for offline FF & Chrome, FFOS + CWS. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100919 (owner: 10Jdlrobson)
[18:30:15] jdlrobson: true! you can say no :)
[18:30:30] Mustache looks more limited, and even though in theory simple is good this doesn't work in practice for a whaladon like MW
[18:31:44] jdlrobson: but changing search is also kind of a big change
[18:31:49] In what ways is it more limited? It would be good to list these in the RfC
[18:32:10] moizsyed: you should have an invite to mobile-tech now
[18:32:11] jdlrobson: should we push changing search and echo to another iteration?
[18:32:30] jdlrobson: thanks!
[18:33:37] Tyler mention mustache being limited as well but didn't offer any specifics. Obviously any templating language is going to be limited and we don't necesarily want the one that is least limited :)
[18:34:47] kaldari, actually it's in the first line of http://mustache.github.io/ :P
[18:34:48] MaxSem, yes, it is more limited, but there's no Twig for JS, or is there?
[18:35:03] Swig, apparently
[18:35:06] jgonera, https://github.com/justjohn/twig.js/
[18:35:25] 72KB, minified...
[18:35:34] that's over 7x more than what we have right now
[18:36:33] MaxSem: well, mustache isn't technically "logic-less". I just want to know what are the actual differences.
[18:36:50] and which of those differences are actually needed
[18:36:57] SWIG is 35KB, not as horrible but still more
[18:37:35] kaldari, imagine you have to split a template in 5 parts because it doesn't have {{ if }} and then glue them together in PHP
[18:37:56] true, I agree we need if/else + foreach + escpaing. What else?
[18:38:01] yeah, unless there's something in Twig that somebody really needs, I'd rather follow the "less is more" mantra
[18:38:14] MaxSem, mustache has if, a primitive one but it does
[18:39:29] kaldari, extensibility - without having an integration with MW i18n it's gonna look awful
[18:39:59] that's a good point
[18:40:22] any other ideas for requirements?
[18:40:29] MaxSem, that's true, but why would Mustache not be extensible with i18n? it has lambdas
[18:40:54] I'm just adding to requirements
[18:41:05] I think we should list all the requirements and "could haves"
[18:41:16] and benchmark mustache too
[18:41:25] add JS version size as one of the arguments
[18:41:33] do we have a wiki page for it already?
[18:44:59] greg-g: https://office.wikimedia.org/wiki/User:Hfung/Product_Manager_Biweekly_Meeting
[18:46:09] MaxSem, jgonera, jdlrobson: I've started listing the requirements and nice-to-haves here: https://www.mediawiki.org/wiki/Requests_for_comment/HTML_templating_library#Templating_library_options_and_considerations
[18:51:46] (03CR) 10Siebrand: "It's completely unknown to me what the change is that has been made between version 27 and 28. I see it rebases and changes in here all th" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100919 (owner: 10Jdlrobson)
[18:52:15] (03PS1) 10Jdlrobson: Hygiene: Gerrit tool [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/104122
[18:53:27] (03CR) 10Jdlrobson: "Siebrand this patch is highly experimental and will continue to change until we have some sort of stability. It might be worth removing yo" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/100919 (owner: 10Jdlrobson)
[18:53:31] MaxSem, jgonera, jdlrobson: http://bits.shutterstock.com/2013/03/07/mustache-vs-swig-templating-shootout/
[18:54:11] kaldari, thanks, will read up on it later
[18:54:15] i agree that less is more is good
[18:54:19] greg-g: sorry about that curve ball :)
[18:54:30] templates shouldn't need to be as powerful as jinja
[18:54:40] you just end up writing code in a template language which is far worse
[18:54:45] greg-g: i am serious that we need to understand who the customer/s of this are. past ops i've never really heard a compelling case
[18:55:13] OK, moving on. I have some questions about bug 58979 (goecoordinates)...
[18:56:50] MaxSem, jdlrobson: Since it looks like the GeoData extension that Max wrote actually stores geodata in the database and makes the data available via an API, why would Nearby be dependant on a wiki template's HTML output?
[18:57:19] Sorry I don't know the history here
[19:00:25] kaldari, we invoke geonotahack upon seeing coordinates in the upper right corner, not actual GeoData
[19:00:28] tfinc: right channel? i don't see any greg-g / tfinc conversation here?
[19:00:45] i can tab complete and see greg-g :)
[19:01:11] there's a difference between these: sometimes people don't want title coordinates
[19:01:13] MaxSem: ah, because otherwise we would have to make an API request on every page load to see if it had geodata or not
[19:01:39] (03CR) 10Jdlrobson: "Follow up in https://gerrit.wikimedia.org/r/104122" (036 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103884 (owner: 10Jdlrobson)
[19:01:46] jdlrobson: backchannel from a hangout meeting ;)
[19:02:05] kaldari, in principle nothing stops us from sticking primary coordinate ffrom GeoData into page HTML as a data value
[19:02:18] MaxSem: Yeah, that's what I was thinking
[19:02:50] or even just a simple global variable flag
[19:02:51] tfinc: yeah, my working definition of the customer(s) is/are: myself, erik, other decision makers. The more we share what we're doing, the more we find out that we need to make changes to what we're doing based on what other's in those roles think.
[19:02:59] * greg-g is tearing down the walls
[19:03:01] ;)
[19:04:47] awjr, MaxSem, jdlrobson: so card 1583 sounds like it is actually a story rather than a bug. Would anyone object to me creating a story for it?
[19:04:57] https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1583
[19:05:41] nope
[19:05:50] kaldari: sure thing - create a story and whack it in the backlog
[19:06:02] i wouldn't say it's a high priority right now
[19:11:11] (03PS8) 10Jdlrobson: Update page previews and geonotahack to use new overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/101428
[19:13:57] (03PS9) 10Jdlrobson: Update page previews and geonotahack to use new overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/101428
[19:14:29] (03CR) 10Jdlrobson: "Remember this is all in beta and none of the associated styles need to be finalised or even promoted to stable." (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/101428 (owner: 10Jdlrobson)
[19:15:28] new story created: https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1586
[19:20:01] (03PS2) 10Jdlrobson: In alpha mode, save and restore the set of open sections in the history [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:20:29] (03CR) 10Jdlrobson: "Thanks for the patch but yeh a few issues. I've added the relevant bug number to this!" (033 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:20:56] jdlrobson: technically mustache also supports filters through it's lambda support
[19:22:21] (03CR) 10Jdlrobson: In alpha mode, save and restore the set of open sections in the history (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:22:29] (03CR) 10Jdlrobson: [C: 031] "See comments on PS1" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:22:43] (03CR) 10Jdlrobson: [C: 04-1] "Whoops that should have been a -1 :)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:22:57] kaldari: it doessss?! oh nos
[19:23:21] jdlrobson: although I don't think we've ever used it. jgonera might know
[19:24:59] (03CR) 10Jdlrobson: [C: 032] Remove tap highlight on section headings [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103767 (owner: 10Jdlrobson)
[19:25:56] so MaxSem when do you want to talk SkinMobile / SkinMinerva >?
[19:26:21] yeah. kaldari, have you seen it?
[19:26:34] (03PS3) 10Jdlrobson: Update Talk overlay to use new overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103854
[19:28:33] MaxSem: The SkinMinerva changes? Yes, but haven't looked at it extensively.
[19:29:19] I still need to review it
[19:29:41] my problem is that the mess just changes its shape but doesn't go away - now instead of overcomplicated inheritance there's if ( #isMobile ) everywhere
[19:29:42] kaldari: would help to get your view on whether we should take that approach or not
[19:30:22] MaxSem: What if there was one single mobile function that only runs if isMobile ?
[19:30:39] I would looove to see the number of differences between mobile reduced instead:)
[19:31:10] Even though it is still a bit messy, it seems more in line with the general architecture of how skins are supposed to work and be used.
[19:31:21] MaxSem: I'd love that too, but I think some are inevitable for example will section collapsing ever be in core?
[19:31:54] well, minerva itself isn't going in core either:)
[19:32:19] but MaxSem it's a skin that could be supported by core
[19:32:26] and I think we all agree that we want to move in the direction of the mobile interface being part of a skin implementation, right?
[19:32:34] also MaxSem I feel like it's acceptable to have a different skin for a mobile vs desktop site
[19:32:36] yep
[19:32:38] or more so, I mean
[19:32:45] yep was to kaldari
[19:32:52] the fact we have one file might encourage us to kill isMobileMode stuff more
[19:33:00] for instance I have no clue why footer needs to be different
[19:33:46] kaity: hey!
[19:33:57] hey monte
[19:34:11] kaity: are you remote today?
[19:34:23] mhurd: yes Im in Ohio!
[19:34:28] kaity: whoa!
[19:35:28] kaity: time to do a hangout later for table of contents stuff?
[19:35:31] (03PS4) 10Jdlrobson: Update Talk overlay to use new overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103854
[19:35:46] (03CR) 10Jdlrobson: Update Talk overlay to use new overlays (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103854 (owner: 10Jdlrobson)
[19:36:18] yea gimme 10 minues mhurd
[19:36:25] kaity: awesome!
[19:37:53] kaldari: grr i now have your "You are about to submit multiple commits" bug
[19:38:15] git checkout master / git checkout -b foo / git review / "You are about to submit multiple commits" < there should be nothing to review!!
[19:38:46] (03CR) 10jenkins-bot: [V: 04-1] In alpha mode, save and restore the set of open sections in the history [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103845 (owner: 10Pcc)
[19:39:05] jdlrobson: yeah, I've tried everything to get rid of that, but haven't figured out the problem. Seems to be a bug with git review.
[19:39:23] i'm gonna ask in wikimedia-dev
[19:39:37] (03Merged) 10jenkins-bot: Remove tap highlight on section headings [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103767 (owner: 10Jdlrobson)
[19:41:53] mhurd: I have 20 mins before a meeting, is that enough you think?
[19:43:20] MaxSem: kaldari Sooo.. SkinMobile / SkinMinerva what's the way forward?
[19:43:54] JeffHall: chrismcmahon what's the state of https://gerrit.wikimedia.org/r/#/c/97497/ ?
[19:45:33] kaity: would you have a bit of time after>
[19:45:34] ?
[19:46:15] mhurd: yes Im free at 12:45 or so
[19:46:25] kaity: that works! :)
[19:47:01] jdlrobson: I think this is probably in the right direction. I'll review it further today.
[19:47:08] mhurd: do you have a current mockup
[19:47:50] jdlrobson: I know we're working on the FIXME aspects of that, but we probably won't do much right away with Zeljko and Antoine both on holiday
[19:49:14] kaity: i have my orig mocks, and may's iteration, but today i sketched some revisions which would be fun to chat about
[19:49:35] mh
[19:49:48] whos mhurd?
[19:49:52] mhurd: cool! can you send me may's iteration? I haven't seen
[19:50:08] thats Monte!
[19:50:11] monteeeeeeeee
[19:50:20] sorry, im still a noob
[19:50:36] hehe ;)
[19:51:41] kaity: ya i'll take snapshot of my stuff and get may's for you by 12:45
[19:51:48] thanks
[19:53:27] jdlrobson, I don't like it but defer to others
[19:54:07] MaxSem: okay thanks :)
[19:54:25] or, rather, I see no improvement
[19:54:52] (03PS2) 10Jdlrobson: Update wgPageName on refresh [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103948
[19:55:27] kaldari: run git fetch --all
[19:55:35] that fixes the review issue
[19:59:25] Hi, I'm a GCI student, and I was wondering which part of MobileFrontend handles non-existentent files (e.g. https://en.m.wikipedia.org/wiki/File:Foozajashsaash.png)
[20:03:05] (03PS9) 10Jdlrobson: Hygiene: LESS file reorg (killing enwp.less) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103570
[20:03:18] (03PS10) 10Jdlrobson: Hygiene: LESS file reorg (killing enwp.less) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103570
[20:06:05] (03CR) 10Jdlrobson: i18n: consistency tweaks (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103643 (owner: 10Shirayuki)
[20:09:11] jdlrobson: https://gerrit.wikimedia.org/r/#/c/104128/ is the core patch to get language variants
[20:13:34] jcf2000: well the nonexistent page message is coming from includes/skins/SkinMobileBeta.php
[20:13:37] handleNewPages()
[20:15:09] hey jcf2000 - have you tried finding out yourself?:)
[20:29:08] (03CR) 10Jdlrobson: [C: 04-1] "Thanks for this - looking a bit closer to what can work." (036 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/74922 (owner: 10Pcc)
[20:35:20] (03CR) 10Jdlrobson: [C: 032] Tablet detection support [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/92036 (owner: 10MaxSem)
[20:35:56] (03Merged) 10jenkins-bot: Tablet detection support [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/92036 (owner: 10MaxSem)
[20:36:29] (03CR) 10Jdlrobson: [C: 04-1] "chrismcmahon: jdlrobson: I know we're working on the FIXME aspects of that, but we probably won't do much right away with Zeljko and Antoi" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/97497 (owner: 10Hashar)
[20:36:48] dr0ptp4kt: yurik care to merge https://gerrit.wikimedia.org/r/#/c/103658/ ?
[20:40:02] (03CR) 10Jdlrobson: [C: 04-1] "grep getToken . -R" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103887 (owner: 10Kaldari)
[20:41:08] (03CR) 10Jdlrobson: "Waiting on Kaldari's thoughts..." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/102853 (owner: 10Jdlrobson)
[20:42:11] jdlrobson, hang on, will test
[20:42:45] * jdlrobson grabbing lunch
[20:51:49] MaxSem: Yes, I did try to find it myself
[20:52:36] theopolisme: Thanks
[20:56:43] (03CR) 10Dr0ptp4kt: "See SkinMobile.php question." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/103658 (owner: 10MaxSem)
[20:57:42] lol.
[20:59:31] hiya, I need to see a beta-labs log. chrismcmalunch said they're available on deployment-bastion:/data/project/logs , but that host is unknown (and I probably don't have rights). I'm in the deployment-prep project but not an admin of it
[21:04:38] ^ignore that, sorry
[21:10:20] (03CR) 10Brion VIBBER: "A good start! A few quick notes on things that need tweaking:" [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/103858 (owner: 10Mhurd)
[21:10:38] (03CR) 10Brion VIBBER: "Love the googly eyes on the debug dump menu item ;)" [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/103858 (owner: 10Mhurd)
[21:19:09] MaxSem: are you following https://gerrit.wikimedia.org/r/#/c/104128/1 ?
[21:19:28] now i do
[21:22:01] jdlrobson: thanks, having someone other than the patch author comment is helpful
[21:22:20] brion: woohoo!
[21:22:40] kaity: chatted with jared! good for now :)
[21:22:53] mhurd: ok cool
[21:23:09] could you still send what you were working on?
[21:23:23] it sounds interesting!
[21:23:32] theo|away: i'm hoping MaxSem will have more useful input
[21:25:08] kaity: i'll condense the sketches down. should be a fun goal for a first pass at TOC
[21:25:52] i'll be home this evening and back in the office next week
[21:26:06] see ya guys! will review a little more over the weekend and start poking more android as well :D
[21:26:18]