[04:17:44] 2Wikipedia-App-Android-App: Make it easier to access talk pages - https://phabricator.wikimedia.org/T89606#1040738 (10Deskana) p:5Triage>3Volunteer? Talk pages serve mobile users really poorly right now. In order to resolve that problem we'd need to invest a lot in to this, and we're not capable of doing t... [08:43:30] morning! [08:45:58] (03CR) 10MaxSem: [C: 032] "D'oh!" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190729 (owner: 10Florianschmidtwelzow) [08:46:25] yo joakino, do you work today? [08:46:35] yes! [08:46:44] MaxSem: how are you awake still :p [08:47:01] (03Merged) 10jenkins-bot: Hygiene: Fix false composer phpunit dependency [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190729 (owner: 10Florianschmidtwelzow) [08:47:04] it's just past midnight [08:47:27] and imma big kid [08:47:29] haha [08:47:36] MaxSem: how was the weekend [08:47:49] reasonable:P [08:48:19] note that tomorrow is also a weekend [08:48:41] amerikaa haha [08:49:00] i'm taking friday instead, going to london this next weekend [08:49:30] https://www.youtube.com/watch?v=NteVmdoo1yI [08:50:36] great song [08:51:01] wunderbaarrr [09:47:36] joakino: I responded! [09:47:45] oh, you hadn't [09:47:48] hey Yuvi|Vacation! [09:47:48] nevermind [09:47:52] what? [09:47:52] hi joakino [09:49:51] joakino: there was an email from phuedx to me, you and a bunch of others. [09:50:05] brian had replied but for some reason I thought it was you [09:50:16] haha ok :) [09:50:19] i'll read it soon [09:50:28] Yuvi|Vacation: are you on SF already? [09:50:43] joakino: no! am in Goa, near a beach [09:51:20] oh cool [09:51:30] hows the weather there [11:21:47] (03CR) 10Florianschmidtwelzow: "@bmansurov: npm and extension integration (hhvm and zend) are in the test pipeline i think, not in the check-only." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [11:21:48] (03CR) 10Florianschmidtwelzow: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [11:23:25] (03CR) 10jenkins-bot: [V: 04-1] Overlay,Panel,Drawer made to use View##events [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [13:11:29] (03PS12) 10Sumit: Overlay,Panel,Drawer made to use View##events [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) [14:42:41] 3Wikipedia-App-Android-App, Wikipedia-App-General-or-Unknown: Handle edit conflicts on the mobile app in some way - https://phabricator.wikimedia.org/T66281#1041544 (10FriedhelmW) p:5Triage>3Unbreak! [15:03:18] hi dbrant [15:03:26] hello! [15:03:35] isn't it a holiday over there? :p [15:03:58] just catching up on a few things ;) [15:05:41] Was I correct at https://ticket.wikimedia.org/otrs/index.pl?Action=AgentTicketZoom&TicketID=8125046 ? [15:07:12] Krenair: that's absolutely right [15:07:27] what about the reply? :/ [15:09:40] Hard to say... it's possible that the user missed the "/data/data/..." (two of them) path [15:15:23] dbrant, does it need root? [15:17:11] Krenair: actually... I think it might! :( All of my devices are rooted, so I didn't think twice about it. Totally missed that. [15:18:00] :( [15:20:36] (03PS3) 10Dbrant: Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 [15:23:17] (03CR) 10jenkins-bot: [V: 04-1] Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [15:27:41] dbrant, there is https://stackoverflow.com/questions/13006315/how-to-access-data-data-folder-in-android-device [15:32:29] Krenair: yeah, it can be done via adb. But ultimately the user probably just wants to copy the saved pages to another device, which we simply don't support yet :( [15:37:28] (03CR) 10Dbrant: "recheck" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [15:41:20] (03PS4) 10Dbrant: Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 [15:41:39] (03CR) 10jenkins-bot: [V: 04-1] Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [15:45:11] (03PS5) 10Dbrant: Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 [15:45:31] (03CR) 10jenkins-bot: [V: 04-1] Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [15:47:02] (03PS6) 10Dbrant: Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 [15:47:15] 3Wikipedia-App-Android-App, Wikipedia-App-General-or-Unknown: Handle edit conflicts on the mobile app in some way - https://phabricator.wikimedia.org/T66281#1041625 (10Deskana) p:5Unbreak!>3Normal Don't panic! This is a normal priority issue. :-) [15:47:31] help needed!, I've added a jquery parameter to InfoOverlay.js, which I believe is causing qunit tests to fail [15:47:35] https://gerrit.wikimedia.org/r/#/c/188673/12/javascripts/modules/wikiGrokRoulette/InfoOverlay.js [15:47:46] test result:https://dpaste.de/rM95 [15:48:12] what can I possibly do to restore the functionality? [15:55:13] codezee: it is holiday on the US so i don't think anybody is going to connect [15:55:17] i gotta leave for a while [15:55:22] sorry :( [15:55:38] joakino: alright, I'll ask sometime later :) [15:55:53] codezee: i'll be around in about 3 hours [15:56:07] bb! [15:56:16] joakino: ok!, if I'll see you, I'll jus tping [15:56:23] *just [16:04:31] (03CR) 10Sumit: [V: 04-1] "the qunit tests still fail, possibly due to adding a jquery parameter in InfoOverlay.js and UploadTutorial.js, but I'm unable to figure ou" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [16:05:42] (03CR) 10Sumit: [C: 04-1] Overlay,Panel,Drawer made to use View##events [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [16:34:49] (03PS1) 10Dbrant: Fix scrolling when tapping and dragging over lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190820 (https://phabricator.wikimedia.org/T87128) [17:02:39] 3Wikidata, Mobile-Web: Mobile version of Dutch Wikisource doesn't list other language version. - https://phabricator.wikimedia.org/T89566#1041759 (10Lydia_Pintscher) [17:14:17] (03CR) 10Bmansurov: [C: 04-1] TOC: horizontal scrolling added for long words (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190734 (https://phabricator.wikimedia.org/T89367) (owner: 10Sumit) [17:29:04] (03CR) 10Florianschmidtwelzow: [C: 032] "bring jenkins back to life" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190663 (https://phabricator.wikimedia.org/T89559) (owner: 10Florianschmidtwelzow) [17:30:20] (03Merged) 10jenkins-bot: Don't show edit pencils on main page [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190663 (https://phabricator.wikimedia.org/T89559) (owner: 10Florianschmidtwelzow) [17:34:53] (03PS1) 10Bmansurov: Add border to local notice [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190826 [17:38:48] hi codezee :) How are you? [17:39:11] (03CR) 10Bmansurov: [C: 031] "Not sure what you mean by 'breaks rotate', but it works fine for me." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190665 (https://phabricator.wikimedia.org/T89506) (owner: 10Florianschmidtwelzow) [17:39:41] FlorianSW: hi, I'm doing fine :), though in a little fix [17:40:48] FlorianSW: I've posted the comments at https://gerrit.wikimedia.org/r/#/c/188673/ [17:41:36] codezee: i took the task https://phabricator.wikimedia.org/T88499 and haven't seen, that you wanted to work on it, sorry :( Yeah, i'll take a look :) [17:42:26] bmansurov: about the change ^ :D I mean, that the rotate css rule breaks, if the height is set, iirc (i haven't found words to describe this :/) [17:42:40] (03PS1) 10Florianschmidtwelzow: TWN: Use own message for terms of use link in license warning (editor and talk) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190831 [17:42:49] FlorianSW: ohh, i thought your fix breaks it ;)) [17:43:02] You could add 'Otherwise' to the beginning of the sentence ;) [17:43:12] bmansurov: no, ok, i see, it's really bad english :D [17:43:23] FlorianSW: its ok, no problem, I didn't give time to it, I'll look around :) [17:44:00] (03PS2) 10Florianschmidtwelzow: Don't add height to watchstar button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190665 (https://phabricator.wikimedia.org/T89506) [17:44:18] FlorianSW: also, test fail results - https://dpaste.de/rM95 [17:44:31] (03CR) 10Bmansurov: [C: 032] Don't add height to watchstar button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190665 (https://phabricator.wikimedia.org/T89506) (owner: 10Florianschmidtwelzow) [17:44:34] codezee: 404 Not found [17:44:37] :D [17:45:02] FlorianSW: sorry, expired, I'll upload another :P [17:45:38] (03Merged) 10jenkins-bot: Don't add height to watchstar button [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190665 (https://phabricator.wikimedia.org/T89506) (owner: 10Florianschmidtwelzow) [17:45:47] (03PS13) 10Florianschmidtwelzow: Use View##events in Overlay, Panel and Drawer [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [17:46:07] FlorianSW: here - https://dpaste.de/k2fp [17:46:17] 3Mobile-Web: Watchstar icons in alpha and beta not aligned - https://phabricator.wikimedia.org/T89506#1041892 (10Florian) 5Open>3Resolved [17:46:50] (03CR) 10jenkins-bot: [V: 04-1] Use View##events in Overlay, Panel and Drawer [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [17:47:33] FlorianSW: the only reason I see for test fail is that I had added a jquery parameter to InfoOverlay.js [17:48:18] FlorianSW: and also to UploadTutorial.js [17:48:54] (03CR) 10Bmansurov: [C: 032] Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [17:49:16] codezee: i'll take a look :) [17:49:26] 3Mobile-Web: EditorOverlay: If VE can't be loaded, reset the clicked state of the switcher - https://phabricator.wikimedia.org/T89462#1041899 (10bmansurov) 5Open>3Resolved [17:49:32] FlorianSW: thanks... [17:50:10] (03CR) 10jenkins-bot: [V: 04-1] Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [17:50:39] (03PS1) 10Dbrant: Update launcher icons. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190835 [17:55:03] (03CR) 10Bmansurov: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [17:57:10] Deskana: yo! [17:57:18] mhurd: Good morning! [18:01:03] Deskana: we’re still doing standup? should be quick :) [18:02:15] mhurd: I think so, yeah [18:02:34] Deskana: sounds good! [18:04:49] (03CR) 10Bmansurov: [C: 04-1] "I think the bug is talking about the Watchlist in the main menu. It should take the user to http://127.0.0.1:8080/w/index.php?title=Specia" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190670 (https://phabricator.wikimedia.org/T88270) (owner: 10Florianschmidtwelzow) [18:08:08] (03CR) 10Florianschmidtwelzow: "correct. If you never clicked on the watchlist (so you're a new user e.g.), you will be redirected to the default watchlist. For all futur" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190670 (https://phabricator.wikimedia.org/T88270) (owner: 10Florianschmidtwelzow) [18:09:50] (03CR) 10BearND: [C: 032] "Still feel there is missing something on the top of the nav drawer. So I hope there will be some design improvements following." [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [18:12:00] (03Merged) 10jenkins-bot: Make nav drawer overlap over search bar. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/187849 (owner: 10Dbrant) [18:13:51] (03CR) 10BearND: [C: 032] Update launcher icons. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190835 (owner: 10Dbrant) [18:14:10] (03Merged) 10jenkins-bot: Update launcher icons. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190835 (owner: 10Dbrant) [18:15:02] FlorianSW: how can jenkins be made to run all tests instead of just two, which it was doing in that patch? [18:15:19] codezee: one second, ok :) [18:15:43] bmansurov: https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/javascripts/modules/wikigrok/WikiGrokDialogC.js#L96 why do you call Drawer's prototype postRender? wikiGrok uses panel and :/ [18:16:24] (03CR) 10Bmansurov: "Why not do it in css? Something like this:" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:17:01] FlorianSW: because wikigrok version C is supposed to be fixed at the bottom of the page [18:17:06] i.e. drawer [18:17:24] FlorianSW: also lines 15 and 16 [18:17:51] codezee: All "new" users are in a test pipeline called "check-only" which runs "Only" basic tests like lint's. For other tests (which takes much more resources), e.g. npm, qunit and so on, you need to be a wmf staff or a so called "trusted user". So for you it's actually not possible to trigger the test pipeline (which includes npm, qunit and so on) :/ You should run the tests locally and submit them if there isn't an error [18:18:21] ...rebase the change, or use the "recheck" key, zuul wil trigger all jobs connected with our accounts, which will normally be all tests :) [18:18:45] FlorianSW: ok :) [18:19:36] bmansurov: hmm, ok :/ because it breaks the change from codezee (https://gerrit.wikimedia.org/r/#/c/188673/13/javascripts/Drawer.js) because postRender references a function that does exist in Drawer class, but not in WikiGrokDialogC :/ [18:20:34] FlorianSW: which function? [18:20:58] bmansurov: new functions, onShowDrawer and onHideDrawer [18:21:09] (03CR) 10Florianschmidtwelzow: [C: 04-1] Use View##events in Overlay, Panel and Drawer (035 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/188673 (https://phabricator.wikimedia.org/T87192) (owner: 10Sumit) [18:21:53] FlorianSW: ok then we should initialize Drawer in WikigrokC then [18:22:40] yap, or you extend WikiGrokDialogB and Drawer, that should work, too [18:22:44] bmansurov: ^ [18:23:02] yes [18:23:29] codezee: do you read it? ^ [18:23:42] FlorianSW: yes, I'm reading it, I see the problem now [18:23:58] great :) [18:24:45] (03CR) 10BearND: [C: 04-1] "does not compile" (031 comment) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190820 (https://phabricator.wikimedia.org/T87128) (owner: 10Dbrant) [18:25:44] (03CR) 10Florianschmidtwelzow: [C: 04-1] "hmm, haven't thought about it :)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:28:58] (03PS2) 10Dbrant: Fix scrolling when tapping and dragging over lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190820 (https://phabricator.wikimedia.org/T87128) [18:29:08] (03PS2) 10Florianschmidtwelzow: Add clear:both before the page-secondary-actions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) [18:29:51] 3Mobile-Web: Main Page has 100% high edit button - https://phabricator.wikimedia.org/T89559#1041976 (10Florian) 5Open>3Resolved [18:31:19] (03CR) 10Bmansurov: [C: 04-1] Ask the user if he want to leave the editor (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [18:33:11] (03CR) 10Bmansurov: [C: 04-1] Add clear:both before the page-secondary-actions (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:35:10] mhurd Deskana is the lead image supposed to be shown in landscape? [18:35:22] Deskana: no [18:35:25] mhurd since you're looking at the native "header" component [18:35:25] Deskana: by design [18:35:29] why? [18:36:00] bgerstle: mvp was spec’ed to be portrait only for now. [18:36:08] For the MVP, it was agreed that we wouldn't think about it in landscape mode because the aspect ratio makes it harder than in portrait. [18:36:27] hm [18:36:28] ok [18:36:44] Rather than do something which might kind of work then open a can of design worms, we just decided to not deal with that. [18:37:24] bgerstle: especially on tablets - stretching images out etc. we expected to revisit at some point in future [18:37:29] Deskana mhurd in either case, should the lead image show when you rotate back to portrait? [18:37:37] bgerstle: it does [18:37:41] bgerstle: Yep [18:37:43] mhurd not in my experience [18:37:44] not always [18:37:58] bgerstle: did the artifcle have a lead image? [18:38:14] yeah, i'll have to try it some more to get exact reproduction steps [18:38:18] bgerstle: def record any instances you find though :) [18:38:21] yeah [18:38:37] mhurd for the scaling, though, it should only be a matter of scale aspect fill [18:38:44] whether it looks good is another question [18:39:03] but, whether any of our lead images look good is kind of a toss-up [18:39:07] bgerstle: it was more an issue of stretching lower res image out making it look pixelated [18:39:17] right [18:39:27] makes sense [18:39:34] And also, how much of the screen you want it to take up [18:39:38] (03CR) 10BearND: [C: 04-1] "If starting to drag up from the bottom of the lead image (roughly where the description is) and continuing up to the edge of the screen I " [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/190820 (https://phabricator.wikimedia.org/T87128) (owner: 10Dbrant) [18:39:40] We wanted to keep the text above the fold [18:39:40] bgerstle: we had/have other lead image layout challenges as it is, so scoped it narrowly [18:40:12] So then you need an image that takes up around half of the height of the screen, and since the screen is much wider than it is tall in landscape mode, you'd end up with a very wide image [18:40:20] mhurd so you'll hide the lead image when in landscape, but show the other components of your header? [18:40:28] bgerstle: yes [18:40:36] Deskana right [18:40:38] bgerstle: that’s how it presently works [18:41:16] These aren't problems that we can't figure out, but we didn't want to have to figure them out to move forwards on this [18:41:23] Portrait is much easier due to the aspect ratios in that mode [18:42:06] sure [18:42:42] and other things are probably more important than supporting landscape. just wondering if we're making more work for ourselves by special-casing our orientation support [18:43:10] but if it's simple as a "hide lead image when landscape" piece of logic, should be fine [18:43:38] wanted to make sure it was a feature, not a bug [18:43:51] i'll let you know if i can narrow down a scenario when it doesn't reappear [18:44:35] bgerstle: yeah, making it do so was a bit more work, but it’s done such that the change takes place as part of the rotation animation, so it looks smooth - you’ll notice the lead title text color changes as part of this animation as well [18:44:44] (03PS1) 10Florianschmidtwelzow: Hygiene: Use Config object in SpecialPages [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190841 (https://phabricator.wikimedia.org/T87614) [18:45:59] bmansurov: can you re-+2 this: https://gerrit.wikimedia.org/r/#/c/190474/ ? :) [18:46:08] mhurd also, i set the thumb width/height constraints to the max screen dimension (so landscape or portrait will at most fill the screen) [18:46:12] but now you can't zoom [18:46:20] (03CR) 10Bmansurov: [V: 032] Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [18:46:25] unless you want to zoom farther than 1:1 [18:47:15] (03CR) 10Florianschmidtwelzow: [C: 04-1] Add clear:both before the page-secondary-actions (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:48:12] bgerstle: And about the merging, generally getting stuff merged in small chunks will let me see it sooner without you having to set up a meeting to show it to me. [18:48:20] bmansurov: you need to remove your +2 on review column and add it again :/ [18:48:29] bgerstle: That's good for both of us, because the earlier you get the feedback the less you might have to change after the fact. :-) [18:48:38] (03CR) 10Bmansurov: Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [18:48:40] bgerstle: for the higher res version i think this should be a fixed resolution consistent with what android uses so we don’t cause unknown number of new resolutions of image to be created/cached. [18:48:46] (03CR) 10Bmansurov: [C: 032] Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [18:48:51] Deskana trust me, i would have if i knew i could merge it w/o it going out prematurely [18:48:58] (03PS3) 10Florianschmidtwelzow: Add clear:both before the page-secondary-actions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) [18:49:00] AFAIK we don't have feature flags [18:49:46] We need a proper alpha for the iOS app. [18:49:46] mhurd if it's any smaller, it will look blurry [18:49:56] TestFlight is kind of both alpha and beta right now and it's annoying. [18:49:59] it's not a matter of alpha/beta [18:50:00] (03Merged) 10jenkins-bot: Reset clicked state of VE editor switcher when loading fails [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190474 (https://phabricator.wikimedia.org/T89462) (owner: 10Florianschmidtwelzow) [18:50:04] it's a matter of having control over what users see [18:50:15] and being able to merge code w/o potentially blocking release [18:50:33] bgerstle: the image info query will return you the closest resolution it has to the one you requested, it will not scale up [18:50:43] And whether we're happy with users seeing it depends on whether it's relatively stable but still untested (beta) or unstable but kind of working (alpha)... so, actually, it is about alpha and beta. [18:50:47] *will return url* [18:50:51] mhurd that's not what i'm seeing. it's scaling up to 2048 [18:50:56] It's about both [18:51:25] bgerstle: hmm what query parameters are you using? [18:51:28] (03CR) 10Bmansurov: [C: 032] Add clear:both before the page-secondary-actions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:51:38] 3Mobile-Web: Alpha: Wrong position for the last edited bar when images are present on the page - https://phabricator.wikimedia.org/T88499#1042072 (10bmansurov) 5Open>3Resolved [18:52:17] bgerstle: i think i found ‘em… testing... [18:52:31] (03Merged) 10jenkins-bot: Add clear:both before the page-secondary-actions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190673 (https://phabricator.wikimedia.org/T88499) (owner: 10Florianschmidtwelzow) [18:53:16] Deskana right, but if i were to just merge the image gallery in the state it's in now, wouldn't you get a flood of noise about a feature you know isn't ready to be seen? [18:53:30] Noise from where? [18:53:36] whoever's using the app? testers? [18:53:51] people in alpha/beta? [18:54:20] We've never really had that problem in our experience. [18:54:31] The feedback tends to be very manageable. [18:54:59] And also, it's kind of a chicken and egg problem, because I don't know whether it's ready to be shown to users because I don't know how it looks :P [18:55:39] right [18:55:43] that's exactly the problem [18:55:48] if i merge the image gallery [18:55:53] and it's not ready [18:55:55] but we want to release..? [18:56:08] this is where feature flags come in handy [18:57:08] i _want_ to show you [18:57:16] (03CR) 10Florianschmidtwelzow: Ask the user if he want to leave the editor (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [18:57:20] but my concern was that putting it in would have blocked the TF build [18:57:57] bgerstle: quick hangout? i think i see the problem [18:58:10] mhurd i have a 1:1 w/ tomasz in a minute. maybe after? [18:58:16] bgerstle: ya sure [18:58:32] (03PS5) 10Florianschmidtwelzow: Ask the user if he want to leave the editor [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) [19:00:50] (03CR) 10jenkins-bot: [V: 04-1] Ask the user if he want to leave the editor [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [19:03:16] (03CR) 10Florianschmidtwelzow: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [19:06:11] (03CR) 10Bmansurov: [C: 032] Ask the user if he want to leave the editor [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [19:06:45] mhurd it appears tfinc isn't working today, call me whenever [19:07:09] bgerstle: good to know - i had 1:1 schedules today as well... [19:07:17] (03Merged) 10jenkins-bot: Ask the user if he want to leave the editor [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190163 (https://phabricator.wikimedia.org/T88949) (owner: 10Florianschmidtwelzow) [19:08:04] 3Mobile-Web: The mobile Web wikicode editor loses the user-entered text - https://phabricator.wikimedia.org/T88949#1042110 (10bmansurov) 5Open>3Resolved @Nnemo, since the editor is constructed on the fly without extra work to save the user's changes, the default browser behavior doesn't apply here. [19:09:27] FlorianSW: what does TWN stand for? [19:09:38] translatewiki.net :D [19:09:40] sorry :P [19:09:47] np, thanks [19:09:59] [[TWN]] [19:10:10] @link [19:10:11] http://enwp.org/TWN [19:10:33] fu**, false, it should be mediawiki.org, not enwiki :) [19:10:39] [[mediawiki:TWN]] [19:10:40] @link [19:10:40] http://enwp.org/mediawiki:TWN [19:10:51] *grrr* [19:11:01] [[mediawikiwiki:TWN]] [19:11:02] @link [19:11:02] http://enwp.org/mediawikiwiki:TWN [19:11:13] aha :D bmansurov ^ [19:11:30] :) [19:13:30] (03CR) 10Bmansurov: "Please change these in the commit message:" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190831 (owner: 10Florianschmidtwelzow) [19:13:50] 3Wikipedia-App-Android-App: Storing current scroll location for each article? - https://phabricator.wikimedia.org/T89607#1042116 (10Deskana) p:5Triage>3Low This is a nice idea to support the "I'm not finished reading, and want to read it later" use case. We'd need to think about how to do this in a way that... [19:16:39] (03PS2) 10Florianschmidtwelzow: TWN: Use own message for terms of use link in license warning (editor and talk) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190831 [19:17:07] (03CR) 10Florianschmidtwelzow: [C: 04-2] "I want to wait for a response of the user on TWN, if this would solve the problem :)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190831 (owner: 10Florianschmidtwelzow) [19:18:41] mhurd: Aha, Adam was using the epic card to track the final set of design changes. [19:18:54] mhurd: So I copied that checklist onto a separate card and put the epic in "Done [19:29:27] 3Wikipedia-App-Android-App, Mobile-Apps, Wikipedia-App-iOS-App: Upload photos on Wikipedia mobile app's edit screen - https://phabricator.wikimedia.org/T88746#1042136 (10Deskana) [19:31:01] sup [19:33:50] 3Wikipedia-App-Android-App, Mobile-Apps, Wikipedia-App-iOS-App: Upload photos on Wikipedia mobile app's edit screen - https://phabricator.wikimedia.org/T88746#1042143 (10Deskana) >>! In T88746#1019516, @Tobias1984 wrote: > I think it should be more a discussion about the needs of the community. If the community... [20:23:37] Deskana: running to lunch [20:24:04] (03PS1) 10Florianschmidtwelzow: TWN: mf-profile-last-thanked should really say "last thanked by" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190854 [20:26:19] (03CR) 10Nemo bis: [C: 031] "Thanks. I guess the original author of this message thought the "last" was an unnecessary detail, but it's actually something which got me" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190854 (owner: 10Florianschmidtwelzow) [21:22:34] Deskana: should https://meta.wikimedia.org/wiki/Schema:MobileWikiAppShareAFact be updated for the case where no text is selected and the share menu item from the overflow menu is used? Also do we want to track "share as text" results? [21:24:25] Hmm. [21:24:50] In the case where no text gets selected there would not be a highlight event. [21:25:41] Yeah, that's interesting. [21:25:45] I'll need to think about this. [21:25:57] The new UX might change exactly what we want to measure. [21:25:57] sure [21:26:32] bearND: I'll try to give you an answer by tomorrow. Any changes we need to make will be minor but we should make them quite fast so we can get it out soon. [21:27:44] Deskana: I just came across it as I was refactoring the code, and I felt I needed to pass the funnel into the new common ShareHandler class [21:28:59] bearND: Is my lack of an answer right now blocking you from proceeding? [21:29:06] No. I'll just add a null check for the funnel for now [21:29:07] bearND: If it is I can drop what I'm doing. [21:29:20] Alright. Thanks! [21:46:32] 3Wikipedia-App-iOS-App: Wikipedia-ios-share-article-name-on-wikipedia/ja - https://phabricator.wikimedia.org/T89604#1042340 (10Nemo_bis) [21:47:12] 3Wikipedia-App-iOS-App: Wikipedia-ios-share-article-name-on-wikipedia/ja - https://phabricator.wikimedia.org/T89604#1040181 (10Nemo_bis) [22:13:45] 3Wikipedia-App-iOS-App: Wikipedia-ios-share-article-name-on-wikipedia/ja - https://phabricator.wikimedia.org/T89604#1042379 (10Deskana) @Takot Thanks for the report. We've had issues with the same string on Android, mainly because the purpose of the string is complicated and we're not adequately explaining what... [22:14:24] (03PS1) 10Deskana: Improve descriptions for some strings in Share a Fact feature. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/190941 (https://phabricator.wikimedia.org/T89604) [22:14:44] mhurd: ^ Can you give this a quick review? [22:14:55] mhurd: Pretty minor string change, should only take a minute. [22:15:17] Wait, I found a typo. [22:15:35] Deskana: k [22:15:35] (03PS2) 10Deskana: Improve descriptions for some strings in Share a Fact feature. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/190941 (https://phabricator.wikimedia.org/T89604) [22:15:41] mhurd: Okay, now it's good ^ [22:15:51] Deskana mhurd was addressing some code-review comments from corey, nearing completion but need to break for a bit [22:15:57] will be on again later [22:16:07] bgerstle: Good stuff. Thanks! [22:16:12] bgerstle: k [22:16:18] o/ [22:16:32] mhurd: No, wait, sorry [22:16:39] mhurd: I wrote something stupid [22:16:47] Deskana: no worry [22:16:51] mhurd: What do I use to escape quotes in this? A backslash? [22:17:26] Deskana: are there other quotes being escaped somewhere in that file? i’d have to check [22:17:42] Good thinking. Yep, it's a backslash. [22:17:59] cool! [22:18:14] (03PS3) 10Deskana: Improve descriptions for some strings in Share a Fact feature. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/190941 (https://phabricator.wikimedia.org/T89604) [22:18:16] mhurd: Alright, I'm done. [22:18:23] patchfinalfinalfinalfinal.pdf [22:18:27] hehe [22:19:20] Deskana: looks good. do you want that merged now so twn gets what they need asap? [22:19:26] mhurd: Yes please [22:19:30] Deskana: k [22:21:04] (03CR) 10Mhurd: [C: 032 V: 032] "Per Dan merged this now so TWN gets clarification a.s.a.p." [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/190941 (https://phabricator.wikimedia.org/T89604) (owner: 10Deskana) [22:21:11] mhurd: Thank you sir! [22:21:16] Deskana: np! [22:21:21] Deskana: good catch [22:22:23] 3Wikipedia-App-iOS-App: Wikipedia-ios-share-article-name-on-wikipedia/ja - https://phabricator.wikimedia.org/T89604#1042406 (10Deskana) 5Open>3Resolved a:3Deskana I'm closing this as resolved as, hopefully, the improved description which was just merged (and will be synced with translatewiki soon) will hel... [22:22:49] 3Wikipedia-App-iOS-App: Wikipedia-ios-share-article-name-on-wikipedia/ja - https://phabricator.wikimedia.org/T89604#1042410 (10Deskana) p:5Triage>3High [22:23:41] mhurd: It's just nice to get these minor little that you can knock out and merge in 5 minutes done the second you see them, otherwise they have a tendency to just sit there. [22:23:47] mhurd: So, thanks again. :-) [22:45:00] Deskana: a small feature request ? [22:46:41] on alpha add some sort of release notes, in the more menu or somewhere, it can even be a list of gerrit patches. doable ? [22:48:55] matanya: Hmm, that would only really serve a very, very small portion of our user base. [22:49:27] matanya: So if we expended effort on something like that, I'd rather do it a more generic version of it that's useful to everyone. [22:50:05] Deskana: i personally find it useful, but it might be only me [22:50:38] matanya: I would too. But you and I are not the typical mobile app user. :-) [22:51:10] I think the root of the request, namely "Let me see everything that's changed in this version of the app" is sound. [22:51:23] But we'd have to think about how to serve it better than a big list of patches. [22:52:02] even a link to a gerrit tag/branch/sprint [22:52:30] Even those are very technical things. Your average user doesn't know what a repository branch or sprint board are. [22:53:01] yeah, i fail to remember that from time to time :) [22:53:25] It depends who the audience is. [22:53:37] ok, i requested, you will find the right way to make it happen, or not :) [22:53:50] I think a page on mediawiki.org with a list of every commit new to a particular release would be good [22:53:59] Note the difference there; on mediawiki.org. The audience is not the user. :-) [22:54:26] but it needs to be maintained in some way [22:54:35] We could probably write a script to look for commits that are indicative of a release that then builds it automatically [22:54:44] Indeed, yes [22:54:48] That is the reason we've not done it yet [22:54:51] Overhead! [22:55:17] would you be interested in help here, if i can ? [22:55:26] isn’t that what the commit messages in git already do? [22:55:37] Also, bear in mind that the app store the user got the app from displays a "recently changed" bit for each app version [22:56:01] "What's New" on this, for example: https://play.google.com/store/apps/details?id=org.wikipedia [22:56:04] mhurd: how would you know what patch made it to what release? [22:56:18] i don't use play [22:56:33] matanya: oh, from the release date of the last patch. just look at commits since then [22:56:33] matanya: Right, there are people that get their apps from other sources on Android. [22:57:03] ah hadn’t considered the “other sources” angle [22:58:23] + the alpha is not on play iirc. so that wouldn't help even if i used play [23:00:26] Correct, it's not. [23:00:35] So would you say this is more for people that are testing? [23:00:54] that is way i asked for the alpha in the first place [23:01:00] "As a volunteer tester, I would like a list of changes that are present in the version of the app that I am using, so I can test them thoroughly"? [23:01:09] yes [23:01:10] Deskana: matanya not ideal, but i suppose you could bundle a release notes file (linked to in the “about the wikipedia app” menu item) which would be appended with respective notes for each release [23:02:11] Deskana: matanya like this one: https://www.dropbox.com/release_notes [23:02:41] Right [23:02:47] that is nice. [23:03:03] But of course there are complications [23:03:22] Beta and production may overlap a lot or not at all [23:03:27] In terms of patches, I mean [23:03:37] That two patches were in the same beta doesn't mean they were in the same production release [23:03:46] So provided you choose one version to do it for, it's simpler [23:03:48] Like production [23:03:56] We could host that on mediawiki.org easily enough [23:04:00] https://git.wikimedia.org/blob/mediawiki%2Fcore.git/REL1_24/RELEASE-NOTES-1.24 [23:04:28] I'd rather we put it on mediawiki.org personally :-) [23:04:33] https://www.mediawiki.org/wiki/MediaWiki_1.24 [23:06:05] Deskana: there could be 15 patches enabling a feature. this seems a better fit for higher level update descriptions. it seems like some git-fu would better suit this use-case. maybe post a couple git commands for seeing the patches that went into a given release? [23:06:56] mhurd: One could massage that into a higher-level thing anyway [23:07:12] I wouldn't mind taking the big list of commits and massaging that into high-level bullet points every release [23:08:35] Deskana: taking something like https://www.mediawiki.org/wiki/MediaWiki_1.25/wmf1 and converting to https://www.mediawiki.org/wiki/MediaWiki_1.24 [23:17:11] (03CR) 10Bmansurov: Change default watchlist from a-z to feed [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/190670 (https://phabricator.wikimedia.org/T88270) (owner: 10Florianschmidtwelzow) [23:17:24] matanya: Yes, but I'd say one page for the *entire app* [23:17:26] matanya: Not per release [23:17:45] matanya: Like that Dropbox list. [23:20:02] that more than works for me [23:21:44] Deskana: need a ticket for this request, or it is on the task list already ? [23:22:08] matanya: I'll make one then you can check it to make sure we're on the same page. :-) [23:22:19] thanks! [23:25:49] 3Wikipedia-App-Android-App, Mobile-Apps, Wikipedia-App-iOS-App: Create a page on mediawiki.org listing all major changes to the app - https://phabricator.wikimedia.org/T89683#1042534 (10Deskana) [23:25:52] matanya: ^ [23:28:49] yes, that thanks a lot [23:35:50] matanya: So is the intended audience meant to be an average app user or an engineer? [23:36:02] matanya: I said it was an average app user, in which case I don't think we really need a script. [23:36:16] matanya: We just need me (the product owner) to write a high level summary by looking at the commit logs in git [23:37:13] i see two use cases: 1. me looking at each commit and testing the feature. 2. me sending the app to friend and telling them "test item #4 on whats new" [23:37:33] 1) is already served by looking at gerrit :-) [23:37:43] 2) Could be served by a mediawiki.org page linked from the app, yeah [23:38:08] so yes, i do use case 1 by looking at gerrit [23:38:50] i do use case 2 by a how to test in my mails, that is not working very well [23:39:04] Yeah, that's true [23:39:31] matanya: We do have this: https://www.mediawiki.org/wiki/Mobile/Release_history#Android [23:40:30] i tried that once, the guy responded: "do you even know me?, I would never read such a page. what do you want me to test" [23:40:31] If we exposed it to users we want it to look a lot nicer than that [23:40:53] Well if you want to make a page that a guy who doesn't want to read pages will read, then that's pretty impossible :P [23:41:52] "we do the hard things right away, impossible takes us sme time" [23:42:00] *some time [23:44:52] The Difficult We Do Immediately. The Impossible Takes a Little Longer. <-- correct quote [23:46:43] There's truth to that. [23:47:13] But, this is a problem we don't have the time to devote that level of thinking too. [23:47:16] *to [23:47:18] We're still a small team [23:47:22] That's why we're not actively tackling it [23:51:45] 3Mobile-Web: Users who click on a link shared from Tweet a Fact in the app should see a banner telling them about it - https://phabricator.wikimedia.org/T89403#1042622 (10bmansurov) @KHammerstein @Msyed: can you guys upload a design? Thanks! [23:52:39] 3Mobile-Web: Put link to desktop version in mobile front end at the top rather than at the bottom - https://phabricator.wikimedia.org/T89448#1042625 (10bmansurov) @KHammerstein @Msyed what do you guys think?