[00:00:46] tfinc: there's no hangout link on the calendar invite [00:03:05] tfinc you around? [00:04:53] (03PS14) 10Dr0ptp4kt: WIP: Add basic Wikipedia Zero alerts for iOS. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/111689 [00:17:16] kaldari, are you working on the ve toolbar story? [00:17:30] it's assigned to me, but I'm actually working on https://bugzilla.wikimedia.org/show_bug.cgi?id=60542 at the moment, which is related to a previous card I worked on. [00:54:07] (03CR) 10JGonera: "I was going to take over this patch and fix the small mistakes, but I discovered more problems with it:" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/110649 (owner: 10Jdlrobson) [01:06:20] (03PS1) 10Kaldari: Fixing thumbnail styles on alpha and beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112616 [01:07:35] jgonera: Can you review this quick fix of a regression: https://gerrit.wikimedia.org/r/#/c/112616 [01:28:32] (03PS1) 10Dr0ptp4kt: Hide edit pencils for now, make mobile langlinks list direct link. [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/112619 [01:30:13] (03PS2) 10Dr0ptp4kt: Hide edit pencils for now, make mobile langlinks list direct link. [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/112619 [01:32:08] (03PS3) 10Dr0ptp4kt: Hide edit pencils for now, make mobile langlinks list direct link. [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/112619 [01:35:40] jgonera: How do I test the languages overlay locally? [01:42:27] kaldari, ?mobileaction=beta [01:54:27] (03PS4) 10JGonera: Hygiene: Cleanup naming convention for special page RL modules [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111146 (owner: 10Jdlrobson) [01:54:59] kaldari, can you check https://gerrit.wikimedia.org/r/#/c/111146 later and merge if it works for you? [02:03:24] (03PS1) 10JGonera: Regression: Fix font size for search input [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112628 [02:03:51] kaldari, also this small regression: https://gerrit.wikimedia.org/r/#/c/112628/ (more urgent) [02:04:08] ok. one sec... [02:06:55] kaldari, where can I test https://gerrit.wikimedia.org/r/#/c/112616/ ? what page? [02:07:58] (03CR) 10JGonera: [C: 032] Fixing thumbnail styles on alpha and beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112616 (owner: 10Kaldari) [02:08:29] (03CR) 10Kaldari: [C: 032] Regression: Fix font size for search input [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112628 (owner: 10JGonera) [02:09:01] (03Merged) 10jenkins-bot: Fixing thumbnail styles on alpha and beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112616 (owner: 10Kaldari) [02:09:31] (03Merged) 10jenkins-bot: Regression: Fix font size for search input [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112628 (owner: 10JGonera) [02:30:47] (03CR) 10Kaldari: Story 1643: Improve typography for overlays (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [02:31:01] (03PS9) 10Kaldari: Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 [02:31:03] (03CR) 10jenkins-bot: [V: 04-1] Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [02:31:47] (03PS10) 10Kaldari: Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 [06:18:52] (03PS2) 10Henry: Add zero-width non-joiner to fix Arabic ligature issue affecting Firefox users [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 [06:39:48] (03CR) 10Henry: "Classic. I realised I'd made this mistake as soon as I shut my eyes to go to sleep last night. Somehow, because this is a non-printing cha" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [06:48:52] (03CR) 10Henry: "Actually, adding one normal space character between the first and the second
  • does the trick just as well. Might it be nicer to " [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [08:51:52] hey MaxSem [08:52:00] MaxSem: did the mobileapp stuff go out yesterday? I don't see https://test.wikipedia.org/w/load.php?debug=true&lang=en&modules=mobile.app&only=styles&skin=vector&* [08:52:53] heh, a few problems were met, I'll email shortly [09:37:51] MaxSem, you are awake? [09:37:58] i thought you don't get online at this time :) [09:37:58] yes [09:40:57] MaxSem, hehe. Oki, want to vchat? [09:41:08] 1 sec [09:41:16] got a toothbrush in my mouth [09:42:51] TMI :-P [14:13:54] (03PS1) 10Zfilipin: Moved Ruby code that multiple repositories use to mediawiki_selenium Ruby gem [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112675 [14:14:48] (03CR) 10jenkins-bot: [V: 04-1] Moved Ruby code that multiple repositories use to mediawiki_selenium Ruby gem [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112675 (owner: 10Zfilipin) [14:17:51] (03PS2) 10Zfilipin: QA: Moved Ruby code that multiple repositories use to mediawiki_selenium Ruby gem [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112675 [14:18:52] (03CR) 10jenkins-bot: [V: 04-1] QA: Moved Ruby code that multiple repositories use to mediawiki_selenium Ruby gem [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112675 (owner: 10Zfilipin) [16:40:54] brion: around? [16:42:17] yo [16:53:34] brb [17:14:14] brion: there's a largish login / logout patch there :) [17:14:21] working on CA now [17:14:22] \o/ [17:14:34] i'll try to peek it over before vanishing into more VPE interviews [17:14:44] brion: oh, today as well? ok! [17:15:02] yep [17:15:04] two glorious days [17:26:35] YuviPanda: heya! [17:26:44] hey mhurd [18:13:13] kaity: have you seen jared around? [18:13:51] kaldari: he's in VPE interviews all day. not sure what that is [18:14:35] ah yeah, same as yesterday :P [18:21:05] (03CR) 10Brion VIBBER: "Couple quick notes inline, still need to test and look over the alert views and such." (036 comments) [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/111689 (owner: 10Dr0ptp4kt) [18:50:53] jgonera and kaldari, can you please take a look at https://gerrit.wikimedia.org/r/#/c/111250/ ? [18:51:14] ...before I start the fun more publicly:) [18:54:35] MaxSem: Personally, I would prefer a more structured approach than just including "blobs". If we define the types of data that we are including, it let's us have RL do more fun things to them, like minification, validation, etc. [18:54:37] MaxSem, sure, will do [18:58:03] kaldari, that's a serious chunk of work, while I was only trying to upstream our template delivery system:) [19:00:13] well, what if we just started with templates then, like we do in mobile? Would that be too provocative? [19:00:15] also, other types of modules can add their own blob handlers with minification etc [19:00:50] we can't start with templates because there's still no agreed upon engine [19:01:08] MaxSem, I have one problem with it ;) for something that will probably become a pretty crucial part of MW, it's scary that it doesn't have any tests [19:01:46] kaldari, also I wanted an explicitly generic delivery method [19:01:56] true, but RL doesn't have to care about what kind of template engine is used [19:02:37] MaxSem: fair enough, I don't feel strongly about it. Just my initial impression. [19:03:27] jgonera, any ideas how to test this stuff which is very much on the edge of PHP and JS? of course I could compare PHP output with a predefined string but that's lame and does not verify that this string actually does what we're asking it to [19:04:33] kenanwang__: sent you the rough deployment time for the new KeepGoing. Let me know if you need a more specific time. Also, I have 2 stories in ready for sign-off, so feel free to poke me if you want to go over them any time soon. [19:06:23] MaxSem: You could just test that "something" gets delivered, i.e. mw.blobs.get( 'whatever' ) isn't null when it's defined. [19:07:39] MaxSem, I think that's better than nothing. Look at current tests of templates in MF or things like testLessFileCompilation in ResourceLoaderTest.php [19:08:40] thanks! [19:08:51] MaxSem, you could also add a test in QUnit, just have a test module with a test template and then see if the template content is OK in QUnit [19:19:51] kenanwang__: I don't think KeepGoing has ever been promoted to stable [19:20:32] Are you sure it should be. I thought I checked on Betalabs. [19:20:45] Kaldari ^ [19:21:00] There was a card for it [19:21:09] hmm, I'll check [19:23:08] kenanwang__: ah, you're right, sorry... [19:23:20] let me look up the deployment date for that.... [19:28:39] kenanwang__: OK, sent you a new email. Sorry I was confused [19:28:59] arg [19:29:03] wrong date [19:30:28] kenanwang__: So it was deployed last Thursday apparently [19:46:29] (03PS3) 10JGonera: Add zero-width non-joiner to fix Arabic ligature issue affecting Firefox users [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [19:47:25] (03CR) 10jenkins-bot: [V: 04-1] Add zero-width non-joiner to fix Arabic ligature issue affecting Firefox users [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [19:48:14] (03CR) 10JGonera: [C: 032] "Let's keep ‌. I only added a comment explaining why it's there. A space seems to add an unnecessary gap between those list elements." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [19:49:04] (03CR) 10jenkins-bot: [V: 04-1] Add zero-width non-joiner to fix Arabic ligature issue affecting Firefox users [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [19:54:28] (03CR) 10JGonera: [C: 032] Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [19:55:19] (03CR) 10jenkins-bot: [V: 04-1] Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [19:56:14] (03CR) 10JGonera: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112595 (owner: 10Henry) [19:57:44] (03CR) 10JGonera: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [20:05:56] Hallo. Is it supposed to be possible to edit the Hebrew Wikipedia from Android using MobileFrontend? [20:06:38] I'm trying to do it and on all articles I get a pencil with a lock, which means that editing is impossible (as far as I understand). [20:06:44] But it works in the English Wikipedia. [20:06:56] WAT. [20:06:58] It works now. [20:07:34] (03CR) 10Kaldari: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [20:07:40] BUT, I don't see anything in the top bar while I'm editing, so there's no save button and I cannot save my edits. [20:07:53] Weird. [20:08:36] I'm using the stock browser on Android 2.3.4. [20:09:34] jgonera: ^ any idea? [20:10:27] (03PS11) 10Kaldari: Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 [20:10:58] (03CR) 10Kaldari: [C: 032] Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [20:11:19] (03CR) 10jenkins-bot: [V: 04-1] Story 1643: Improve typography for overlays [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [20:14:12] chrismcmalunch: having problems with jenkins and qunit: ^ [20:14:33] aharoni: I think you *are* supposed to be edit [20:14:39] aharoni: can you try with desktop chrome? [20:14:52] aharoni: It is possible that android 2.3 has been blacklisted (I do not know this for sure at all) [20:15:24] YuviPanda: trying... [20:15:29] aharoni: ok [20:16:15] YuviPanda: works well. [20:16:32] Maybe some JS fails on my phone's browser. [20:16:36] I wonder how can I debug that. [20:16:48] Is there a JS console on the stock browser or something.. [20:16:49] ? [20:17:07] aharoni: heh, you wish :P [20:17:10] :) [20:17:12] aharoni: it *might* be blacklisted [20:17:15] I don't know for sure [20:17:18] aharoni: try enwiki? [20:17:22] I wonder how you mobile web guys do that. [20:17:23] on your phone, that is [20:17:31] Yes - in enwiki it works. [20:17:49] aharoni: when I did similar things (PhoneGap on Android 2.3) it was a lot of hacks and swearing :) [20:17:50] Can you try it maybe? Do you have some Android 2.3.4 device around, or something similar? [20:18:01] (I'll also try on Android 4.) [20:18:14] aharoni: I do have a couple of android 4.4 devices, around. let me try [20:18:25] I will try to randomly click on things until I see an edit box :D [20:18:55] YuviPanda: here's my favorite article for tests: he.m.wikipedia.org/wiki/ASCII [20:19:04] http://he.m.wikipedia.org/wiki/ASCII [20:19:08] Short Latin title :) [20:20:23] aharoni: I do get an edit button [20:20:27] and I seem to be able to make edits [20:20:30] although I haven't made any [20:20:40] violetto: around? small question [20:20:50] yurik, able to videoconference? [20:21:29] dr0ptp4kt, yep [20:22:03] YuviPanda: all seems to work well on Android 4. I strongly suspect that some JS fails at some point. [20:22:40] aharoni: yeah, sounds likely. [20:22:52] aharoni: I'd suggest filing a bug report, since that automatically gets pulled into 'mingle' [20:23:23] I noticed that on Android 4 the editing button initially has the lock and then it disappears. [20:23:44] I guess that the editing is disabled initially and then some JS enables it, [20:23:55] and on Android 2.3.4 it sometimes fails. [20:24:22] And the top editing toolbar doesn't work at all in Hebrew. [20:25:32] aharoni: yeah, I also noticed the 'lock and disappear' [20:25:43] aharoni: might be a JS failure, yeah [20:30:48] jgonera: I have just noticed that the google calendar event does not have hangout link :) [20:30:52] and I can not add one [20:31:06] aharoni: do file a bug! [20:31:12] OK [20:32:46] zeljkof, on it, just a sec [20:34:38] zeljkof, done, shuold be there now [20:35:30] jgonera: reloading... [20:42:07] very, very curious, YuviPanda : I see odd JS errors when I try to edit from desktop Firefox, too. [20:42:16] aharoni: what js errors? [20:42:28] "Exception thrown by job.error" [20:42:33] "Error: Container must be attached to the DOM" [20:42:33] ... [20:42:35] job?! [20:42:37] oh [20:42:48] hmm [20:42:50] This appears at https://he.m.wikipedia.org/wiki/ASCII#editor/0 . [20:42:56] Works on Chrome. [20:43:18] I'm pretty sure that I could edit from desktop Firefox just a couple of days ago. Was anything new deployed? [20:43:29] aharoni: the entire header code changed, IIRC [20:43:52] let me try this with debug [20:43:58] aharoni: ok [20:44:05] aharoni: are you talking about mobile ff? [20:44:19] No, desktop Firefox. [20:44:34] (I think that it's the first time I try running MF with debug=true. Is it even supposed to work? :) ) [20:45:50] ... no, it doesn't seem to work. [20:45:53] aharoni: ah, right. [20:45:53] aharoni: it is technically supposed to, yeah :) [20:46:07] I guess that I'll report that as a bug as well. [20:49:46] hello . I am interested in developing wiki news app . Please guide me how can i start ? [20:50:04] and which libraries of android ? [20:50:21] yurik, so like this, please: [20:50:43] iif X-CS was present, then tack on X-Zero-Rated: 1 [20:50:47] else do nothing [20:52:03] (03CR) 10Dr0ptp4kt: "We're going to try to do this in Varnish for the time being. Yuri's working on a small patch that does the following:" [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/111399 (owner: 10Dr0ptp4kt) [20:52:09] aharoni: yeah, please do [20:52:33] (03CR) 10Dr0ptp4kt: [C: 04-2] "Holding off on this until we know one way or the other whether this can be easily done in Varnish." [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/111399 (owner: 10Dr0ptp4kt) [20:55:57] YuviPanda: https://bugzilla.wikimedia.org/show_bug.cgi?id=61232 [20:56:02] aharoni: thanks! :) [20:56:12] aharoni: although I don't believe I can myself help much :( [20:56:30] aharoni: did I mention the android (and iOS) apps are going to have editing built in to them? [20:57:02] YuviPanda: I heard this somewhere... maybe from Jon. [21:00:34] dr0ptp4kt, around? could you rejoin vchat pls [21:00:46] yurik, yeah, hang on [21:01:01] yurik, wanna invit me? [21:01:07] i'm online, just hidden [21:01:15] oh, i see it [21:02:30] aharoni: the last release already had anon editing [21:02:39] aharoni: the next one (tomorrow?) will have logged in editing [21:03:03] YuviPanda: the app, you mean? [21:04:30] oh, and YuviPanda: do you happen know what's the status of the app translation in translatewiki? It's silly that I'm asking you, because it should be vice versa, but it just struck me that I haven't checked what's going there for a while. [21:05:05] I saw that the Android plurals support patch was merged a few weeks ago, but did anything else progress? And is that patch enough? [21:05:08] aharoni: I think brion was supposed to get that in there? [21:05:16] aharoni: I guess so, we just need to get the repos registered. [21:05:32] aharoni: brion was supposed to do that, but idk what the status of that is. any help or that would be welcome, I think :) [21:05:55] OK, I'll try to check what's happening with that some time very soon. You know how proud I am of our ultramegatranslated apps. [21:08:15] aharoni: :) [21:08:34] aharoni: you should maybe drop an email to brion going 'how is it going on getting thos into twn'? :) [21:08:40] OK [21:11:00] yurik, per our videoconference, revised plan: just tack outbound X-CS: header [21:11:11] yep [21:18:00] dr0ptp4kt, done [21:18:04] https://gerrit.wikimedia.org/r/#/c/112805/ [21:19:13] YuviPanda: here now! [21:19:51] violetto: just wanted to show you something, but then I broke it before taking screenshots :( will poke again when I have it working [21:20:08] YuviPanda: D: [21:20:10] okeh [21:23:10] (03CR) 10Kaldari: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/111374 (owner: 10Kaldari) [21:23:14] YuviPanda: https://bugzilla.wikimedia.org/show_bug.cgi?id=61233 [21:23:40] aharoni: :) ok [21:24:25] chrismcmalunch: having problems with jenkins: https://gerrit.wikimedia.org/r/#/c/111374/ [21:24:38] chrismcmahon: ^ [21:25:08] kaldari: sorry, was afk. I don't know much about qunit at all, looking... [21:25:35] kaldari: Jenkins gave you a +1 just now? [21:25:41] also, for some reason "recheck" doesn't get jenkins to rerun all the tests [21:26:08] chrismcmahon: Yeah, it will either give a -2 or a +1, neither of which allow me to merge it [21:26:11] kaldari: I'm going to have to send you to hashar or Krinkle, no idea [21:27:11] here's the qunit output from Jenkins (that's failing): https://integration.wikimedia.org/ci/job/mwext-MobileFrontend-qunit-mobile/2029/console [21:28:00] it seems to be failing on initialization [21:29:11] dr0ptp4kt: https://github.com/montehurd/apps-ios-wikipedia/commit/ae6a0197d1af70abc45e7265c8f9ed13097e92e8#diff-7cac9a3f82e0e22f9613ea5889d2a6d0L5 [21:30:08] kaldari: I have never used qunit, sorry to be so not helpful [22:56:27] (03PS1) 10JGonera: Fix missing overlay header in RTL languages on Android 2.3 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112822 [22:56:39] kaldari, small fix, merge when you have time ^ [22:56:46] aharoni, fixed ^ [22:57:21] (03CR) 10jenkins-bot: [V: 04-1] Fix missing overlay header in RTL languages on Android 2.3 [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112822 (owner: 10JGonera) [23:05:30] (03CR) 10JGonera: [C: 04-1] "Just a small concern." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112675 (owner: 10Zfilipin) [23:15:20] (03PS1) 10Jforrester: Regression: Fix font size for search input [extensions/MobileFrontend] (wmf/1.23wmf13) - 10https://gerrit.wikimedia.org/r/112823 [23:16:51] (03CR) 10jenkins-bot: [V: 04-1] Regression: Fix font size for search input [extensions/MobileFrontend] (wmf/1.23wmf13) - 10https://gerrit.wikimedia.org/r/112823 (owner: 10Jforrester) [23:17:58] (03Abandoned) 10Jforrester: Regression: Fix font size for search input [extensions/MobileFrontend] (wmf/1.23wmf13) - 10https://gerrit.wikimedia.org/r/112823 (owner: 10Jforrester) [23:24:49] MaxSem: failures are all reasonable. keepgoing, VE, and search failures, I think manybubbles was updating indexes out there today [23:25:11] awesome [23:50:44] kaldari, so why do you want to replace a customizable solution with a one-off formula? [23:51:12] which one are you talking about? [23:51:37] position, width, or ratio? [23:51:58] all of them [23:52:49] well, in the case of width I think you're probably right. It would make more sense to do something customizable... [23:53:37] I don't see a strong reason anyone would need to customize position or ratio though. [23:54:14] but maybe you have a different opinion on it? [23:54:15] why can't someone boost first position drastically? [23:54:42] or even the second one, if that's how articles on their wiki are organised [23:54:50] hmm, maybe so, but it's kind of micro-optimization [23:55:24] OK, OK, leave position customizable then :) [23:56:02] * MaxSem chuckles his fingers [23:56:06] what about ratio? Is someone really going to want to figure out what those numbers mean and create their own table for it? [23:56:48] 3, 5, 20, 30, 31 is just a mysterious sequence unless you dig in the code [23:57:03] it actually has a comment [23:57:10] explaining what it means [23:57:25] what about panoramas? [23:57:42] what if they're a valid PI somewhere? [23:58:14] PI? [23:58:28] Page Image [23:58:44] (03PS1) 10JGonera: Update QUnit tests to use sinon.js from core [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/112826 [23:59:02] kaldari, let us merge again ^