[00:03:10] 10Collab-Notifications-Page, 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 13Patch-For-Review: [regression pre-wmf.9] Clicking on notification badges gives TypeError - https://phabricator.wikimedia.org/T155800#2963573 (10Etonkovidova) Checked the fix in betalabs. **QA recommendation: Resolve.** [00:07:12] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): [minor] monobook- Notification drop-down panel needs additional padding for bundled notifications. - https://phabricator.wikimedia.org/T156080#2963579 (10Etonkovidova) [00:12:13] Going for a little bit, be back in about an hour+ and continue working. [00:17:17] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page, 05MW-1.29-release-notes, 13Patch-For-Review, 05WMF-deploy-2017-01-17_(1.29.0-wmf.8): Build user interface for the Filter Search Bar - https://phabricator.wikimedia.org/T149435#2963613 (10Etonkovidova) @Mooeypoo... [00:25:09] etonkovidova, when you did your screen share, it looks like VE was parsing {{int:}} with the correct language (even though it wasn't correct on save). [00:25:16] What were you testing with, since I can't reproduce this? [00:26:06] Do you remember which wiki, the user's language, and the input text? [00:29:20] matt_flaschen: yes, it's tricky to reproduce - so it was cawiki. The input text was from the ticket's link - I will add here some steps in a moment [00:31:20] matt_flaschen: 1) go to https://www.wikidata.org/wiki/Topic:Sphnk0g2x4zswkow?uselang=fr 2) Copy (without opening any editor): À voir ce que cela donne en pratique mais ajouter natural heritage (Q386426) en plus de cultural heritage (Q210272) dans la contrainte Type me semble une bonne idée. [00:31:57] matt_flaschen: 3) Open cawiki VE - paste it. Should work. [00:37:41] Thanks, etonkovidova [00:38:02] (03PS1) 10Catrope: Remove FloatableElement hack, no longer necessary. [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333816 [00:38:49] (03CR) 10Catrope: [C: 04-2] "This needs to wait until a version of OOjs UI is released that has I22f2ca72cf7c2f in it." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333816 (owner: 10Catrope) [00:43:14] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963645 (10jmatazzoni) [00:49:19] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963663 (10Cenarium) @jmatazzoni: I'm actually working on implementing anon notifications in echo, which I think is the onl... [00:49:27] 06Collaboration-Team-Triage, 10Notifications, 07Epic, 13Patch-For-Review: Provide access to Notifications for anonymous users - https://phabricator.wikimedia.org/T58828#2963666 (10jmatazzoni) The project we have in hand and for which this patch was presumably intended is [[ https://en.wikipedia.org/wiki/W... [00:53:33] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963681 (10jmatazzoni) >>! In T156083#2963663, @Cenarium wrote: > @jmatazzoni: I'm actually working on implementing anon no... [01:04:38] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963699 (10Cenarium) >>! In T156083#2963681, @jmatazzoni wrote: > > Hi Cenarium. Our messages crossed paths. As you can s... [01:24:10] (03PS2) 10Catrope: Update coment for overflow-x hack [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333816 [01:31:51] 06Collaboration-Team-Triage, 10Notifications, 06Wikipedia-Android-App-Backlog, 06Wikipedia-iOS-App-Backlog: Echo notifications on Wikipedia mobile apps (Android & iOS) - https://phabricator.wikimedia.org/T123155#2963746 (10Liuxinyu970226) Who can tell me that why "JMinor raised the priority of this task fr... [01:56:50] 10Collab-Notifications-Page, 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 13Patch-For-Review: [regression pre-wmf.9] Clicking on notification badges gives TypeError - https://phabricator.wikimedia.org/T155800#2963791 (10jmatazzoni) 05Open>03Resolved [02:12:48] 06Collaboration-Team-Triage, 10Notifications, 07Epic, 13Patch-For-Review: Provide access to Notifications for anonymous users - https://phabricator.wikimedia.org/T58828#2963828 (10Cenarium) >>! In T58828#2963503, @Mattflaschen-WMF wrote: > Issues to discuss include: > * Random expiration when server caches... [02:21:38] (03PS3) 10Catrope: Remove FloatableElement hack, no longer needed [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333816 [03:57:14] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests: Remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T63729#2963875 (10MZMcBride) [03:59:11] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests: Remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T63729#663384 (10MZMcBride) I think it's reasonable to wait for {T154816} to be resolved, if possible, but we should set a firm deadline. Is four weeks enough... [03:59:42] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10MediaWiki-Export-or-Import: Migrate ORES_paper flow board from Meta to Mediawiki.org - https://phabricator.wikimedia.org/T154816#2924843 (10MZMcBride) I left a relevant comment at T63729#2963875. [04:12:46] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963892 (10Mattflaschen-WMF) @jmatazzoni wrote: > Evaluating edits for deferral presumably causes some delay. How long will... [04:21:18] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2963904 (10Catrope) >>! In T156083#2963892, @Mattflaschen-WMF wrote: >> E.g., Deferred Changes applies to new and anonymous... [04:22:26] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10MediaWiki-Export-or-Import: Migrate ORES_paper flow board from Meta to Mediawiki.org - https://phabricator.wikimedia.org/T154816#2963905 (10Mattflaschen-WMF) [04:24:06] TIL the hard way that 'user' implies 'noexec'. [04:26:47] (in fstab) [04:27:16] RoanKattouw, you're a giant-hack-comment party-pooper. [04:27:18] just saying. [04:27:26] (CR) Mooeypoo: "Fine, fine. However, I demand the return of "this is a hack" comment. Beyond the amusement it will provide us for months -- it will also m" [core] - https://gerrit.wikimedia.org/r/332932 (owner: Mooeypoo) [04:27:29] RoanKattouw, ^ [04:27:30] ;) [04:28:00] (Seriously, though, I think you should add "hack" comment on top of your fix, as I commented on it. It really is a hack, and we should really try not to keep it in the code for very long. [04:28:02] ) [04:28:24] Sure, I can put the giant hack comment back in [04:28:36] Doesn't have to be *that* giant hack -- just a hack comment in general [04:28:56] I'm having trouble seeing how we'd do this better in the backend; I guess we'd add data attributes telling us which filter each link is associated with? [04:28:58] I mean, I was amused... but I don't mind having that specific one in -- we just need to make sure *something* says "we should really not leave this in the code forever" [04:29:11] (Note BTW that the ORES damaging filter link doesn't have a class in the correct format!) [04:29:17] RoanKattouw, if we know that this is what we want to do, we could get rid of the "|" in general and add them PROPERLY IN CSS [04:29:28] I suppose so [04:29:38] And then add either data-name="foo" or whatever [04:29:41] so we can pick them out better [04:29:46] if we want to, we can do it right [04:29:56] This isn't The Right Way[tm] [04:29:59] Re pipes in CSS, that would not be very old-browser-friendly [04:30:00] It's just The Fast Way [04:31:07] Even if we leave the pipes in, we can set the data on the elements better, and even get the pipes inside the so when we remove it, it is actually being removed... ther are many ways, but the current way is dirty and only done because we kinda don't want to fix the backend output [04:31:10] We could use a CSS class (new-implementation or whatever), but include both the text and the pipe in the element, so you can hide both. [04:31:33] On the backend I mean. [04:31:39] Ex, one issue is that it's hard to find the name of the filter from its element [04:31:55] we can fix that in the backend already so it doesn't bother the current version but is better handled in the front end [04:32:15] Mathematically speaking I don't think you can put the pipe inside [04:32:20] But to be fair, the entire decision about whether we even *want* to show unimplemented filters at all if a user has the "new system" is a product decision [04:32:25] Consider what happens if you need to remove the first element, and also the last element [04:32:46] ... I can't compute your latest sentence, it does not start with "Assume" [04:32:50] (But you have a point) [04:32:57] Hmm I guess ideally once all filters are implemented we can just remove that whole list [04:33:28] RoanKattouw, whoops, good point. [04:33:41] RoanKattouw, okay, so, I'm actually not sure which way to go: I am dismayed that we don't support 3rd party installations, and in that aspect, I'd appreciate having a "dumping area" for filters that come from extensions that haven't updated their filter definition so external 3rd party wikis can still have them [04:34:10] Then again, if we *state clearly* that we changed the filter structure definition, we can also decide product-wise that we expect 3rd party extensions to adhere to the new structure if they want to be added ot the view [04:34:27] mooeypoo, no, we need back-compat at least temporarily. [04:34:29] Either way, we should make that decision with that in mind, rather than "just" not care about 3rd parties :\ [04:34:43] matt_flaschen, yeah I'd like back-compat for a while, honestly. [04:34:54] We can decide how long 'temporarily' is, but IMHO dropping it immediately is not acceptable. Plus, it's a beta initially anyway, so it's pretty unfair to make them implement just for that. [04:34:55] but that's a product decision of how long this lasts... [04:35:03] oh no it won't be immmediately -- hence the hack [04:35:11] but even if the temporary is long, hacks are acceptable [04:35:22] mooeypoo, I meant 'immediately' as in 'when it became a production beta feature' [04:35:25] but if we decide that we want to backwards-compat as a product decision, then we should not hack, and do it right [04:35:31] * mooeypoo nods [04:35:52] OK, new patchset has significantly more HACK comments [04:36:01] I have a general issue with how we (don't) care for 3rd party installation, honestly [04:36:08] mooeypoo, I can add metadata so the pipe list can just be generated (basically what RoanKattouw said). Please add what you what metadata you want to T152754 [04:36:09] T152754: Configure filters in a single extensible place - https://phabricator.wikimedia.org/T152754 [04:36:10] We care incidentally [04:36:34] mooeypoo, basically data attributes in such, but then reflow the list (to deal with pipe issues)? [04:36:37] matt_flaschen: What do you mean by that? You mean generate the list server-side? [04:37:27] matt_flaschen, it's not just the pipe, to be fair, it's also making sure we hide the correct filters (for instance, 'hidebyme' --> 'hidemyself' etc) [04:37:47] btw that one seems to be a symptom of maybe an error in setting it up? [04:37:52] why would the name be different...? [04:39:24] RoanKattouw, no, I mean: [04:39:38] 1. Generate the pipe list on server-side basically as it already does, just add data attributes. [04:40:00] 2. Regenerate it client-side (similar to the HACK, but without the "This is absolutely terrible" part). [04:40:08] Is it just me, or does the current HACK either: [04:40:43] mooeypoo, I was just saying you'd have to regenerate the list, but the data attributes would make doing so easier by providing that hidebyme->hidemyself whatever metadata. [04:40:49] a. Drop the pipe. [04:41:26] b. Append it only because the pipe doesn't have a class so it's not a known filter [04:41:28] ? [04:41:51] Don't see where it's handled. But anyway, mooeypoo, it seems like more metadata would at least sole part. If so, please comment at T152754 . [04:41:51] T152754: Configure filters in a single extensible place - https://phabricator.wikimedia.org/T152754 [04:43:49] matt_flaschen: My hack, as written, deletes the pipe after the link, except if there isn't one then it deletes the pipe before the link [04:44:16] But yeah we definitely need a data attribute for the filter name, both my hack and Moriel's extract the filter name from the class name [04:44:32] (and then manually remap hidebyme->hidemyself because that one is weird) [04:47:17] it's also new, could it be that it was a typo by stephane? [04:47:25] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page: Configure filters in a single extensible place - https://phabricator.wikimedia.org/T152754#2963929 (10Mattflaschen-WMF) [04:47:36] I think he named it that at first and then changed -- could just have been a typo we missed in CR [04:47:53] I don't think it affects much, we can/should just fix that class name in the backend [04:48:01] No I don't think that one is new, I think that one is old [04:48:11] I'd rather fix the data attribute value [04:48:22] Let's not change the class name because that would break CSS written for the old class name [04:48:28] Which has probably been around for 10+ years [04:48:57] I thought it was new [04:49:08] but yeah, data attribute for a filter name is best [04:49:13] I don't think that we added links for any of the new filters we added [04:49:25] If we want to, btw, we can have the back end do the calculation btw [04:49:39] so the backend could go "does this one exist? yes? add a "existsinjs" class or something [04:49:49] mooeypoo, oh yeah, I mentioned that before, but forgot. [04:49:50] then the UI just needs .existsinjs { display: none; } [04:50:11] or a simpler jquery that takes anything but existsinjs classes and takes off the span + pipe [04:50:18] mooeypoo, no, because of the pipe, but it at least saves looking it up in JS (though I guess it doesn't matter really). [04:50:20] instead of the front end doing the whole loop calculation [04:51:20] I'll already need to have that info loaded to hide the ones that are *only* in the new implementation, so I might as well tag it in the backend. [04:51:24] yeah but we can make the job of the front end easier. Doing $( '.filterClassThing:not[.existsInJS]' ).each( function () { /* remove all of these + the following spans if exist, blah */ } ); is easier than "do the entire claculation and remove" [04:51:49] Yeah I imagined that the backend already does that anyways (goes over implemented filters) so it should be relatively easy? [04:51:53] ish? [04:51:58] No ish [04:53:04] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page: Configure filters in a single extensible place - https://phabricator.wikimedia.org/T152754#2963944 (10Mattflaschen-WMF) [04:53:19] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page: Configure filters in a single extensible place - https://phabricator.wikimedia.org/T152754#2859076 (10Mattflaschen-WMF) [04:53:46] mooeypoo, RoanKattouw, quick Vagrant patch: https://gerrit.wikimedia.org/r/#/c/333839/ [04:54:43] matt_flaschen, were these removed from parsoid? [04:55:16] mooeypoo, they're client-side config variables. It doesn't directly touch Parsoid variable names. [04:55:49] They are still supported in Flow, but not used if VirtualRESTServiceClient is used for Flow/Parsoid (which it is on Vagrant). [04:56:34] mooeypoo, see Flow/includes/Conversion/Utils.php, getVRSObject [05:11:33] 06Collaboration-Team-Triage, 10Flow, 07I18n: Vagrant Parsoid should be configured (separately) for all wikis - https://phabricator.wikimedia.org/T156097#2963980 (10Mattflaschen-WMF) [05:11:52] That's a precusor to T156097 (I don't know how we didn't catch that before, it's a big problem when using Parsoid locally). [05:11:52] T156097: Vagrant Parsoid should be configured (separately) for all wikis - https://phabricator.wikimedia.org/T156097 [05:13:22] matt_flaschen, ok it looks good to me, I don't mind +2'ing but how deeply should I test this? [05:13:31] I am a little eh about reprovisioning atm [05:14:51] mooeypoo, well, that's the only way you can test it. Technically I also didn't really provision (I just removed it from settings.d/puppet-managed file then tested). [05:15:13] mooeypoo, but Roan already merged it so you're in the clear. [05:15:17] matt_flaschen: werdna started working on multi-wiki for parsoid just before he left. The patch is probably very rotted by now though -- https://gerrit.wikimedia.org/r/#/c/199591/ [05:16:05] bd808, ah, thanks. [05:16:06] we probably need much better site matrix support :/ [05:16:56] matt_flaschen, oh. ha. well. nevermind then. [05:17:12] matt_flaschen, sorry for the delay, I'm battling jquery and ooui for a popup on hover. [05:17:16] ... it does not cooperate. [05:17:49] mooeypoo, it doesn't matter, it was just to make me less confused later when working on T156097 . And I need to leave anyway. [05:17:49] T156097: Vagrant Parsoid should be configured (separately) for all wikis - https://phabricator.wikimedia.org/T156097 [05:17:58] * mooeypoo nods [05:22:05] 06Collaboration-Team-Triage, 10Flow, 07I18n: Flow does not support varying language of parts of content based on user interface language (e.g. {{int:}}) - https://phabricator.wikimedia.org/T106687#1475109 (10Mattflaschen-WMF) Short answer: There might be a way to fix this without changing the architecture.... [05:23:42] yeah I should stop too [05:23:47] eh. Okay, see you guys tomorrow. [05:25:40] 06Collaboration-Team-Triage, 10Flow, 07I18n: Flow does not support varying language of parts of content based on user interface language (e.g. {{int:}}) - https://phabricator.wikimedia.org/T106687#1475109 (10Catrope) >>! In T106687#2964050, @Mattflaschen-WMF wrote: > As you can see, it preserves all the data... [05:31:58] 06Collaboration-Team-Triage, 10Flow, 07I18n: Flow does not support varying language of parts of content based on user interface language (e.g. {{int:}}) - https://phabricator.wikimedia.org/T106687#2964057 (10Mattflaschen-WMF) >>! In T106687#2964054, @Catrope wrote: > To clarify, are you suggesting we write a... [05:32:08] Have a good night, all. [06:24:57] mooeypoo: FYI 17:24:38  (CR) Catrope: [C: -1] "I rebased this patch and amended it to add existing defaults to the init file. Then I noticed it doesn't initialize correctly." (1 comment) [core] - https://gerrit.wikimedia.org/r/326064 (https://phabricator.wikimedia.org/T149391) (owner: Mooeypoo) [07:54:07] (03CR) 10Legoktm: [C: 032] build: Re-use MessageDirs in banana conf [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333619 (owner: 10Esanders) [07:54:13] (03CR) 10Legoktm: [C: 032] build: Re-use MessageDirs in banana conf [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333618 (owner: 10Esanders) [08:05:31] (03Merged) 10jenkins-bot: build: Re-use MessageDirs in banana conf [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333619 (owner: 10Esanders) [08:06:32] (03Merged) 10jenkins-bot: build: Re-use MessageDirs in banana conf [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333618 (owner: 10Esanders) [08:10:18] (03CR) 10jenkins-bot: build: Re-use MessageDirs in banana conf [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333618 (owner: 10Esanders) [08:15:20] (03CR) 10jenkins-bot: build: Re-use MessageDirs in banana conf [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333619 (owner: 10Esanders) [08:45:11] 06Collaboration-Team-Triage, 10Flow, 13Patch-For-Review: Flow: Prettify thread permalink URLs/Titles (they are not human readable!) - https://phabricator.wikimedia.org/T59154#601146 (10geraki) Please check https://www.mediawiki.org/wiki/Talk:Flow/2017_community_feedback_concerning_URLs_format In non latin sc... [08:47:24] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration: Page Curation: Speedy deletion G12 (Copyvio) hangs on blacklisted site - https://phabricator.wikimedia.org/T156110#2964338 (10Hydronium_Hydroxide) [09:11:16] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964406 (10MarcoAurelio) [09:32:40] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests, 13Patch-For-Review: Remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T63729#2964469 (10MarcoAurelio) Proposed workflow: # {T154816}, then # {T156113}, then # SWAT https://gerrit.wikimedia.org/r/#/c/333860/ [10:29:45] (03PS4) 10Addshore: DiscussionParserTest only create users when needed [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333673 [10:43:17] 06Collaboration-Team-Triage, 10Flow: Page loses focus while trying to summarize thread - https://phabricator.wikimedia.org/T156118#2964575 (10Elitre) [10:44:46] (03PS1) 10WMDE-Fisch: [WIP] Fix mentions failing editing empty pages [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333879 (https://phabricator.wikimedia.org/T155998) [10:46:34] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964406 (10Catrope) This would only affect https://meta.wikimedia.org/wiki/Talk:Flo... [10:48:48] 06Collaboration-Team-Triage, 10Flow, 06Community-Liaisons, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964612 (10MarcoAurelio) @Catrope Would conversion still be required if we delete t... [10:50:16] 06Collaboration-Team-Triage, 10Flow: Page loses focus while trying to summarize thread - https://phabricator.wikimedia.org/T156118#2964616 (10Elitre) Actually, it just happened again while trying to mention in a regular thread. [10:51:33] (03CR) 10jerkins-bot: [V: 04-1] [WIP] Fix mentions failing editing empty pages [extensions/Echo] - 10https://gerrit.wikimedia.org/r/333879 (https://phabricator.wikimedia.org/T155998) (owner: 10WMDE-Fisch) [10:54:33] 06Collaboration-Team-Triage, 10Flow, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jan-Mar 2017): Ask communities about how to reformat Flow's URLs - https://phabricator.wikimedia.org/T155837#2964628 (10Trizek-WMF) The discussion is open. [10:57:15] 06Collaboration-Team-Triage, 10Notifications, 06Developer-Relations (Jan-Mar-2017), 10Wikimedia-Developer-Summit (2017): Hacking Session: Learn to make new notification types - https://phabricator.wikimedia.org/T149246#2964631 (10Qgil) p:05Triage>03Normal Is there any additional outcome expected from,... [11:22:15] 06Collaboration-Team-Triage, 10Flow, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964692 (10Qgil) (#community-liaisons is already watching the parent task.) [11:32:28] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2964736 (10ovasileva) {F5346425} @Nirzar - maybe it's just me, but the edges are looking a little pixelated? [11:56:57] 06Collaboration-Team-Triage, 10Flow, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964757 (10Catrope) >>! In T156113#2964612, @MarcoAurelio wrote: > @Catrope Would conversion still be requi... [12:00:00] 06Collaboration-Team-Triage, 10Flow, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2964759 (10MarcoAurelio) @Catrope Better to run the conversion script then. Thanks. [12:57:09] (03CR) 10Catrope: [C: 032] Rephrase Flow notifications preference [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333278 (https://phabricator.wikimedia.org/T149365) (owner: 10Sbisson) [13:02:49] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Notifications, 13Patch-For-Review: Problem with marking mention notifications as read when visiting the originating page - https://phabricator.wikimedia.org/T154057#2964970 (10Catrope) Hmm, I'm not sure if this is a good feature or a bad one. For l... [13:16:08] (03Merged) 10jenkins-bot: Rephrase Flow notifications preference [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333278 (https://phabricator.wikimedia.org/T149365) (owner: 10Sbisson) [13:16:25] (03CR) 10jenkins-bot: Rephrase Flow notifications preference [extensions/Flow] - 10https://gerrit.wikimedia.org/r/333278 (https://phabricator.wikimedia.org/T149365) (owner: 10Sbisson) [13:48:17] (03PS13) 10Cenarium: Support local anon notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/329374 (https://phabricator.wikimedia.org/T58828) (owner: 10Cen.temp) [13:56:00] (03CR) 10jerkins-bot: [V: 04-1] Support local anon notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/329374 (https://phabricator.wikimedia.org/T58828) (owner: 10Cen.temp) [14:50:13] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Flow, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jan-Mar 2017), 07User-notice-collaboration: Publication of the Flow Satisfaction Survey results including an analysis and raw da... - https://phabricator.wikimedia.org/T144730#2965261 [15:01:26] 06Collaboration-Team-Triage, 10Flow, 07I18n: Flow does not support varying language of parts of content based on user interface language (e.g. {{int:}}) - https://phabricator.wikimedia.org/T106687#2965312 (10Trizek-WMF) That task has been raised up because of Wikidata, where Properties are not displayed in t... [15:08:01] 06Collaboration-Team-Triage, 10Flow, 13Patch-For-Review: Flow: Prettify thread permalink URLs/Titles (they are not human readable!) - https://phabricator.wikimedia.org/T59154#2965377 (10Trizek-WMF) For practical reasons, please do not comment both on the proposal and here. @geraki, I've replied to your fee... [16:03:32] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Flow, 07User-notice-collaboration: Rich text mode should be the default in Flow replies - https://phabricator.wikimedia.org/T155665#2965558 (10SBisson) This is what I've found in my testing so far: == Saving a new topic sets the preference to the... [16:57:02] 06Collaboration-Team-Triage, 10Flow, 13Patch-For-Review: Flow: Prettify thread permalink URLs/Titles (they are not human readable!) - https://phabricator.wikimedia.org/T59154#2965748 (10Tgr) All modern browsers percent-decode URLs before display; you never see the `%`s unless you use IE8 or something (in whi... [16:57:40] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2965749 (10MGChecker) I don't exactöy know how FlaggedRevs works in English Wikipedia, but in German Wikipedia it's well po... [17:13:18] (03PS14) 10Cenarium: Support anon notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/329374 (https://phabricator.wikimedia.org/T58828) (owner: 10Cen.temp) [17:15:43] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2965799 (10Nirzar) this needs work. i wanted to post actual fixes but just keeping this comment so it does... [17:36:44] 06Collaboration-Team-Triage, 10Flow, 13Patch-For-Review: Flow: Prettify thread permalink URLs/Titles (they are not human readable!) - https://phabricator.wikimedia.org/T59154#2965883 (10Trizek-WMF) >>! In T59154#2965748, @Tgr wrote: > All modern browsers percent-decode URLs before display; you never see the... [17:55:32] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): [wmf.8] Save Summary with open Link/Mention inspector gives "Uncaught TypeError: Cannot read property 'ownerDocument' of undefined" - https://phabricator.wikimedia.org/T156156#2965904 (10Etonkovidova) [18:11:21] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2965944 (10bmansurov) No pixelation on my end: {F5350344} @Nirzar, can you also include steps to reprodu... [18:19:55] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jan-Mar 2017): Contact Portuguese Wikipedia about testing ERI Recent Changes Beta project - https://phabricator.wikimedia.org/T156157#2965957 (10Trizek-W... [18:45:49] 06Collaboration-Team-Triage, 10Flow, 10Wikimedia-Site-requests, 07Community-consensus-needed: Enable Flow on one page (Research talk:ORES paper) on Meta - https://phabricator.wikimedia.org/T153412#2966047 (10Trizek-WMF) 05Open>03Invalid That task has been completed, then reopened. then a RfC that has c... [18:47:47] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 06Reading-Web-Backlog, and 2 others: Mobile mentions should link to mobile view and not desktop view - https://phabricator.wikimedia.org/T154750#2966053 (10Trizek-WMF) p:05Triage>03High I've tested it: # I'm browsing https://en.m.wikipedia... [19:07:50] 06Collaboration-Team-Triage, 10Flow: Remove the Topic: namespace from Search until topics Search is supported - https://phabricator.wikimedia.org/T155450#2966091 (10Trizek-WMF) 05Open>03Invalid I don't see Topic NS on any advanced search forms anymore. [19:13:11] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page: Release strategy for RC page improvements: what wikis get the new features when? - https://phabricator.wikimedia.org/T150715#2966107 (10Trizek-WMF) [19:13:13] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jan-Mar 2017): Contact communities for ERI Recent Changes Beta project - https://phabricator.wikimedia.org/T146972#2966104 (10Trizek-WMF) 05Open>03st... [19:20:30] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 06Reading-Web-Backlog, and 2 others: Mobile mentions should link to mobile view and not desktop view - https://phabricator.wikimedia.org/T154750#2966141 (10Jdlrobson) Note MobileContext in MobileFrontend does provide a method getMobileUrl for... [19:27:48] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2966147 (10Nirzar) There's no pixelation. but here are visual fixes that are needed, i'm just listing down... [19:28:12] 06Collaboration-Team-Triage, 10Flow: What links here use the topic code and not the topic title - https://phabricator.wikimedia.org/T155242#2938209 (10Trizek-WMF) {T59154} may solve that. We are investigating. [19:33:40] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): RCFilters UI: Use mixins and variables in LESS files - https://phabricator.wikimedia.org/T156164#2966170 (10Mooeypoo) [19:52:19] matt_flaschen: re T155026 Page curation creates double page headings on user talk pages -- are you going to do something or you've done smth already? [19:52:19] T155026: Page curation creates double page headings on user talk pages - https://phabricator.wikimedia.org/T155026 [19:53:21] matt_flaschen: in betalabs the issue does not exist ... [20:34:08] 06Collaboration-Team-Triage, 10Flow, 10UI-Standardization, 10VisualEditor, 07Design: Come up with a better re-usable UI concept for a button to switch editor-mode; the current one is confusing, and hard to discover - https://phabricator.wikimedia.org/T116417#2966487 (10RHo) +1 @Pginer-WMF – think it make... [21:00:08] (03PS15) 10Cenarium: Support anon notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/329374 (https://phabricator.wikimedia.org/T58828) (owner: 10Cen.temp) [21:36:12] etonkovidova, I'm aware of the issue and what needs to be fixed. It's high on my TODO list. [21:36:28] etonkovidova, thanks for the note that it works on Beta, though. I guess I did test the import, there. I can verify. [21:37:40] matt_flaschen: ok then :) [21:48:26] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2966837 (10bmansurov) @Nirzar, some of the points don't belong to this task. Can you add those to the pare... [21:48:50] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2966839 (10bmansurov) a:03bmansurov [21:49:06] 06Collaboration-Team-Triage, 10Notifications, 10MobileFrontend, 10Reading-Community-Engagement, and 2 others: Change how the notification count is displayed - https://phabricator.wikimedia.org/T152457#2966842 (10Nirzar) @bmansurov sure thing, i was in the zoneeeeeeee [21:53:21] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2966860 (10Cenarium) >>! In T156083#2963904, @Catrope wrote: >>>! In T156083#2963892, @Mattflaschen-WMF wrote: >>> E.g., De... [21:57:56] 06Collaboration-Team-Triage, 10Notifications, 07Epic, 13Patch-For-Review: Provide access to Notifications for anonymous users - https://phabricator.wikimedia.org/T58828#2966892 (10Cenarium) Assuming we can alter the db, I've modified the patch set to offer the same level of functionality than notifications... [21:59:45] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Flow, 07User-notice-collaboration: Rich text mode should be the default in Flow replies - https://phabricator.wikimedia.org/T155665#2966909 (10jmatazzoni) >>! In T155665#2965558, @SBisson wrote: > This is what I've found in my testing so far: > >... [22:02:02] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Flow, 07User-notice-collaboration: Rich text mode should be the default in Flow replies - https://phabricator.wikimedia.org/T155665#2966925 (10jmatazzoni) Before we can discuss the optimal behavior, I suppose I need to know what we think the curren... [22:02:37] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-ReviewStream, 10Analytics, 10ChangeProp, and 4 others: Set up the foundation for the ReviewStream feed - https://phabricator.wikimedia.org/T143743#2966929 (10Ottomata) Today we had a ReviewStream meeting. We had original... [22:19:41] 06Collaboration-Team-Triage, 10Flow: Page loses focus while trying to summarize thread - https://phabricator.wikimedia.org/T156118#2964575 (10Etonkovidova) @Elitre The issue looks familiar and scrolling up is really annoying. Unfortunately, I do not have such combination (Win 10 and Chrome) to test. In Win 7 w... [22:20:04] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017): Consider how to best inform anonymous users that their edit has been actively deferred - https://phabricator.wikimedia.org/T156083#2967038 (10Cenarium) Regarding the feasibility of anon notifications (T58828), I had made a rough patch that didn't alter t... [22:21:23] 06Collaboration-Team-Triage, 10Flow, 10UI-Standardization, 10VisualEditor, 07Design: Come up with a better re-usable UI concept for a button to switch editor-mode; the current one is confusing, and hard to discover - https://phabricator.wikimedia.org/T116417#2967039 (10jmatazzoni) @RHo - +1 to putting... [22:26:29] 06Collaboration-Team-Triage, 10Notifications, 07Epic, 13Patch-For-Review: Provide access to Notifications for anonymous users - https://phabricator.wikimedia.org/T58828#2967047 (10jcrespo) echo_notification table for enwiki: ``` CREATE TABLE `echo_notification` ( `notification_event` int(10) unsigned NO... [22:35:02] 06Collaboration-Team-Triage, 10Flow, 07Mobile: Impossible to reach the link "Disable mobile version" on a Flow page - https://phabricator.wikimedia.org/T155480#2944340 (10Etonkovidova) @Thibaut120094 Did you mean to "Desktop" link instead of "Disable mobile version"? The issue is valid and moved to the cur... [22:40:06] 06Collaboration-Team-Triage, 10Flow, 10Wikimedia-Site-requests: Maintenance script run to convert Flow boards to Wikitext prior to remove Flow from Meta-Wiki - https://phabricator.wikimedia.org/T156113#2967112 (10MarcoAurelio) p:05Triage>03Normal [22:45:50] 10Collab-Team-2016-Apr-Jun-Q4, 06Collaboration-Team-Triage, 10Flow, 06Editing-Department, 10MobileFrontend: Edit pencil icon displayed as protected unnecessarily when viewing Flow page in MobileFrontend - https://phabricator.wikimedia.org/T130322#2967119 (10Etonkovidova) Re-opening since the issue has re... [22:49:07] 10Collab-Team-2016-Apr-Jun-Q4, 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Flow, 06Editing-Department, and 2 others: [regression wmf.9] Edit pencil icon displayed as protected unnecessarily when viewing Flow page in MobileFrontend - https://phabricator.wikimedia.org/T130322#2967122 (10Etonk... [22:57:37] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 07Mobile, 07Regression: [mobile regression wmf.9] Cannot switch to VE on Flow post - https://phabricator.wikimedia.org/T156201#2967142 (10Etonkovidova) [23:13:08] 06Collaboration-Team-Triage, 10Flow, 10UI-Standardization, 10VisualEditor, 07Design: Come up with a better re-usable UI concept for a button to switch editor-mode; the current one is confusing, and hard to discover - https://phabricator.wikimedia.org/T116417#2967191 (10RHo) Hi @jmatazzoni – apologize for... [23:31:13] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page, 05MW-1.29-release-notes, 13Patch-For-Review, 05WMF-deploy-2017-01-31_(1.29.0-wmf.10): Build user interface for Active Filter Display Area - https://phabricator.wikimedia.org/T149391#2967265 (10jmatazzoni) [23:32:19] 06Collaboration-Team-Triage (Collab-Team-Q3-Jan-Mar-2017), 10Edit-Review-Improvements-RC-Page: Approved interface text for RC page interface elements - https://phabricator.wikimedia.org/T149385#2967283 (10jmatazzoni)