[00:00:55] 10VisualEditor, 10VisualEditor-MediaWiki: [Regression pre-wmf5] The font for headings/texts etc are appearing bigger after opening VE and switching to read mode - https://phabricator.wikimedia.org/T98193#1263286 (10Jdforrester-WMF) [00:00:59] 10VisualEditor, 10VisualEditor-MediaWiki: [Regression pre-wmf5] The font for headings/texts etc are appearing bigger after opening VE and switching to read mode - https://phabricator.wikimedia.org/T98193#1262057 (10Jdforrester-WMF) Oh dear. [00:01:39] 10VisualEditor, 10VisualEditor-MediaWiki: [Regression pre-wmf5] Switching from VE to read mode re-embiggens the text again and makes it impossible to edit twice in one session - https://phabricator.wikimedia.org/T98193#1263291 (10Jdforrester-WMF) p:5High>3Unbreak! [00:13:09] 10WikiEditor, 3Editing Department 2014/15 Q4 blockers, 5Patch-For-Review: action.abort.type in Schema:Edit not populated for Wikitext - https://phabricator.wikimedia.org/T95938#1263337 (10Jdforrester-WMF) This is now deployed to production as of 2015-05-06 ~00:05 UTC. Please confirm. [00:23:27] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: [Usability Suggestion] In link inspector, after switching back from External link to Search pages while a text is selected, retaining the original text in the text field will be helpful - https://phabricator.wikimedia.org/T98210#1263371 (10Nirzar) [00:27:33] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: [Usability] Consistency between inspector and context menu - https://phabricator.wikimedia.org/T98271#1263401 (10Nirzar) 3NEW [00:29:28] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: Move the delete action outside of edit for link inspector and language inspector - https://phabricator.wikimedia.org/T98272#1263415 (10Nirzar) 3NEW [00:35:55] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: Adding link flow after the first dialog - https://phabricator.wikimedia.org/T98273#1263429 (10Nirzar) 3NEW [00:41:27] James_F: we probably need a pull through for this https://phabricator.wikimedia.org/T98080 ? I dont see it fixed in betalabs. [00:42:58] ryasmeen: Worse, it needs an OOUI release [00:43:12] Those usually only happen once a week [00:45:19] oh no! :( [00:45:20] RoanKattouw: It was released. [00:45:54] Hah you're righ [00:45:59] The release I did yesterday [00:46:11] Yes. [00:46:18] (helped with) [00:46:28] so it should be working in Betalabs? [00:47:01] Theoretically. [00:47:12] 10VisualEditor, 6WMF-Design-Research: VE image updating: test two concepts side by side - https://phabricator.wikimedia.org/T96979#1263482 (10aripstra) a:5KHammerstein>3None [00:47:14] ok but its not practically :( [00:47:23] re-opening it then [00:47:33] Which template? [00:47:57] Hmm. [00:48:02] I think Ed mis-triaged that bug. [00:48:16] The thing he linked to was for making it not red on initial focus. [00:48:20] Which is indeed fixed. [00:48:46] ah right! [00:49:18] Which is https://phabricator.wikimedia.org/T98202 [00:49:23] 10VisualEditor: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1263506 (10Ryasmeen) 5Resolved>3Open [00:49:37] James_F: yeah [00:49:42] 10VisualEditor: [Regression pre-wmf5] Link inspector opens with a red highlight around the empty text field except for the first time of every edit session - https://phabricator.wikimedia.org/T98202#1263512 (10Jdforrester-WMF) 5Open>3Resolved Fixed in https://gerrit.wikimedia.org/r/#/c/208694/ [00:50:14] 10VisualEditor, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Link inspector opens with a red highlight around the empty text field except for the first time of every edit session - https://phabricator.wikimedia.org/T98202#1263517 (10Jdforrester... [00:50:21] 10VisualEditor, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Link inspector opens with a red highlight around the empty text field except for the first time of every edit session - https://phabricator.wikimedia.org/T98202#1262287 (10Jdforrester... [00:51:02] James_F: no, that one is also not fixed [00:51:15] You sure? [00:51:17] * James_F re-tests. [00:51:32] yeah so the initial focus is not red for the first time when you open link inspector [00:51:42] But it is when you switch to external. [00:51:43] Bah. [00:51:59] but after that it again comes back on each try [00:53:48] * James_F sighs. [01:00:06] 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: [Regression?] Removing edited articles from watchlist at it.wp - https://phabricator.wikimedia.org/T97838#1263606 (10Jdforrester-WMF) [01:00:46] 10VisualEditor: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1263607 (10Ryasmeen) This also happens in Media Settings dialog {F161038} [01:42:12] (03PS2) 10Jforrester: Update extension.json and empty out PHP entry point [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208057 (https://phabricator.wikimedia.org/T87992) [01:42:43] (03CR) 10Jforrester: "PS2 is a re-generation." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208057 (https://phabricator.wikimedia.org/T87992) (owner: 10Jforrester) [01:44:26] (03CR) 10jenkins-bot: [V: 04-1] Update extension.json and empty out PHP entry point [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208057 (https://phabricator.wikimedia.org/T87992) (owner: 10Jforrester) [05:30:40] 10Mediawiki-extensions-Graph-VE: Community bonding evaluation for "Enable VisualEditor support in Graph extension" - https://phabricator.wikimedia.org/T97502#1263847 (10Qgil) [09:30:03] 10Mediawiki-extensions-Graph-VE, 10VisualEditor, 10Graph, 10Parsoid, and 2 others: Enable VisualEditor support in Graph extension (GSoC 2015 Proposal) - https://phabricator.wikimedia.org/T93585#1264155 (10Yurik) The graph extension has been launched on all production wikis. [10:31:49] (03PS1) 10Esanders: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) [11:45:16] Hello everyone [11:45:40] I'm trying to install VisualEditor on my mediawiki server and I seem to have run into a problem [11:46:40] it's running on an ubuntu server, I installed parsoid without any issues but when I try to run the visualeditor I get an SSL error [11:47:02] Is there a way to make parsoid ignore the fact that the wiki is using a self signed SSL cert? [12:04:42] (03PS2) 10Esanders: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) [12:09:28] (03CR) 10Esanders: "needs review asap" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [12:10:10] (03CR) 10Esanders: [C: 032] jshint: Kill file-level globals and add "moment" to the list [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209044 (owner: 10Jforrester) [12:13:20] (03CR) 10Esanders: "Underline has two problems: (1) spaces don't get underlined due to how jquery.highlight works, so you get 'Barack Obama' and" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209022 (owner: 10Esanders) [12:17:05] (03CR) 10Alex Monk: [C: 031] "not tested yet" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [12:22:51] Krinkle|detached, do you still have the MW/core, MW/extensions folder structure? [12:28:37] I think I have that on my desktop [12:28:50] which I don't use for MW development anymore so haven't updated [12:28:59] I might've gotten around to fixing it at some point [12:50:58] hello guys, i#m looking for a solution that the visualeditor inserts pdf files with the tags filename instead of the usual [[ ]]. Can you help me? [12:51:51] what extension provides that syntax? [12:52:13] it's embedPDF [12:54:11] Krenair, https://gerrit.wikimedia.org/r/#/c/209216/ stuck by a load order bug [12:54:51] SMC, there currently isn't a generic tool for inserting extension tags, just editing them [12:56:52] SMC, the simplest example of creating an insert tool for a specific extension can by found in wikihiero (wikihiero/modules/VisualEditor) [12:57:19] fun [12:58:52] my first thinking was that a make a check when using the insert media funktion of the filename ending .. if it is an pdf the function has to use the tags [13:06:18] Krenair, shouldn't extensions just register __DIR__ . "extension.json" instead of expecting core to know where they are? [13:06:51] SMC, it would be difficult to override the media insert dialog - plus that generates very different HTML to an extension [13:07:16] you'd be better off copying the wikihiero approach [13:08:03] VisualEditor is not a wikitext editor, it is a Parsoid HTML editor, so you aren't generating filename [13:09:32] hm ok, maybe that would be the problem.. [13:09:45] but
[13:10:38] the logic for doing that is already in generic was is already in the MWExtenion* classes, so you just need to extend them, like wikihiero does [13:11:19] * the logic for doing that is already implemented generically in the MWExtenion* classes ... [13:21:01] edsanders, I think that extensions being in weird non-standard places is considered a bit of an edge case [13:21:24] as mentioned in the bug we already have globals to support it [13:21:36] before the .json commit it wasn't a problem [13:42:33] (03CR) 10Jforrester: [C: 032] Remove bold from result titles so that query highlighting works [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209022 (owner: 10Esanders) [13:42:57] thanks edsanders .. maybe i'll find a way [13:44:52] (03Merged) 10jenkins-bot: Remove bold from result titles so that query highlighting works [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209022 (owner: 10Esanders) [14:15:14] edsanders, how do you actually trigger GeneralReferenceDialog? [14:55:25] Krenair: It's currently only triggered from the Citoid extension. [14:55:39] Krenair: We're changing that soon-ish, but it's not available right now. [14:56:04] Where is the Citoid part of https://gerrit.wikimedia.org/r/208971 then? [14:56:29] Krenair: getSurface().execute( 'window', 'open', 'generalreference', { …} ) [14:57:11] Krenair: https://gerrit.wikimedia.org/r/#/c/209198/ but it's not a breaking change, is it? [14:57:23] no [14:57:44] as long as it works as before then it's good to merge [14:58:14] you will need to have citoid extension enabled to test it though, as James points out [14:58:28] Citoid extension installation is relatively easy. [14:58:42] You need the extension and a few MW: messages set. [14:59:04] yeah I already installed that [15:06:22] edsanders, "We couldn't make a citation for you. You can make one by using the ." [15:06:37] it doesn't provide a proper way to try again either [15:06:40] ah [15:07:03] (03PS2) 10Divec: WIP: Zero-width space to avoid empty annotation cursoring bug [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208355 (https://phabricator.wikimedia.org/T96307) [15:07:05] this is the Citoid patch yes? [15:08:33] yes [15:09:49] (03CR) 10jenkins-bot: [V: 04-1] WIP: Zero-width space to avoid empty annotation cursoring bug [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208355 (https://phabricator.wikimedia.org/T96307) (owner: 10Divec) [15:22:43] (03PS2) 10Esanders: Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 [15:23:16] (03PS1) 10Esanders: Fix preview widget class name [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/209241 [15:23:40] (03CR) 10Jforrester: [C: 031] "Whoops." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/209241 (owner: 10Esanders) [15:40:55] (03PS3) 10Esanders: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) [16:32:08] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers, 5Patch-For-Review: Remove dropdown for other cite options from the toolbar. Options will be accessible from the auto-cite tool - https://phabricator.wikimedia.org/T96946#1265395 (10Jdforrester-WMF) [16:32:12] 10Citoid, 10VisualEditor, 7Design, 3Editing Department 2014/15 Q4 blockers, 5Patch-For-Review: Consolidate citation workflow into an inspector - https://phabricator.wikimedia.org/T97390#1265394 (10Jdforrester-WMF) [16:33:38] edsanders: Yep. [16:33:52] edsanders: mediawiki/core, mediawiki/extensions/ [16:34:06] Krinkle, see https://phabricator.wikimedia.org/T98319 and patch [16:34:16] edsanders: Its the only workable set up if you deal with wmf branches and older release branches. [16:34:26] Because otherwise skins/ and extensions/ get screwed up [16:34:57] I use require_ from the other directory directly and set wgExtensionAssetsPath accordingly for front-end references. [16:35:23] have you got a fix for the backend problem^^ ? [16:36:07] What backend problem? This set up has worked without issues for over 5 years. Afaik Roan and Reedy (used to?) have the same set up, as do most "old timer" MW devs that predate me :) [16:37:34] have you read the bug? [16:38:31] 10VisualEditor, 6Community-Liaison, 10WMF-Product-Strategy, 3Editing Department 2014/15 Q4 blockers, 3Search-and-Discovery-Research-and-Data-Sprint: How many active editors are using VisualEditor at large Wikipedias? - https://phabricator.wikimedia.org/T94637#1265438 (10Ironholds) [16:39:09] Yes, I raised the issue and caused the bug to get filed. [16:39:26] https://github.com/Krinkle/dotfiles/blob/master/hosts/KrinkleMac/templates/mw-LocalSettings.php#L24-L29 [16:39:37] https://github.com/Krinkle/dotfiles/blob/master/hosts/KrinkleMac/mw-CommonSettings.php#L288-L292 [16:39:44] There is a $path parameter for a reason [16:39:49] Though your patch would make things a lot easier [16:40:07] Yes, but it doesn't work - Setup.php which sets the defaults, hasn't run yet [16:40:43] Such a global var has never been needed because MediaWIki has no concept of an extension directory in the backend. Code gets loaded in LocalSettings, anything in the backend expands relatively, not absolutely. [16:41:38] Typically using __DIR__, e.g. when registering modules or classes. [16:42:12] edsanders: Perhaps $path can optionally be a directory, to make it slightly easier to use. [16:42:34] I seem to recall maintenance scripts breaking with that directory setup [16:42:40] Nope. [16:42:57] If you've got more than one wiki, you should have MW_INSTALL_PATH set in the env [16:43:03] same as in prod [16:43:26] I had only one wiki with that setup [16:43:36] you run the extension php file directly, and that usees the env to discover mediawiki core [16:43:51] https://github.com/Krinkle/dotfiles/blob/master/hosts/KrinkleMac/modules/setup.sh#L25-L28 [16:44:08] for Maintenance that is [16:45:00] Krenair: Yeah, that works well. But it becomes more or less mandatory to have the env variables when working with multiple hostnames. [16:45:30] This was a setup on my desktop that had only one wiki but core in it's own directory [16:45:40] I use *.mediawiki.dev for a basic wiki farm using the same source code. And mediawiki.dev for an isolated simple wiki. [16:46:00] You were sitting next to me when I set up my laptop :) [16:46:26] Right [16:50:29] 10VisualEditor: A mouse icon appears upon hovering over the search and external link icon in link inspector but none of them are clickable object - https://phabricator.wikimedia.org/T98213#1265527 (10Ryasmeen) [16:52:39] 10VisualEditor: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1265548 (10Ryasmeen) We changed something around this in this week: https://phabricator.wikimedia.org/T97516 . May be related? [17:07:45] 10VisualEditor, 6Community-Liaison, 10WMF-Product-Strategy, 3Editing Department 2014/15 Q4 blockers, 3Search-and-Discovery-Research-and-Data-Sprint: How many active editors are using VisualEditor at large Wikipedias? - https://phabricator.wikimedia.org/T94637#1265607 (10Ironholds) [17:08:50] (03PS1) 10Bartosz Dziewoński: Revert "Be more selective in application of 0.8em font size" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209273 (https://phabricator.wikimedia.org/T98193) [17:11:20] 10VisualEditor, 10VisualEditor-MediaWiki, 5Patch-For-Review: [Regression pre-wmf5] Switching from VE to read mode re-embiggens the text again and makes it impossible to edit twice in one session - https://phabricator.wikimedia.org/T98193#1265634 (10matmarex) If this is "Unbreak Now!" priority, I recommend me... [17:13:28] (03PS4) 10Mvolz: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [17:13:39] (03PS3) 10Mvolz: Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [17:15:50] 10WikiEditor, 3Editing Department 2014/15 Q4 blockers, 5Patch-For-Review: editSessionID is duplicated in Edit:Schema for Wikitext - https://phabricator.wikimedia.org/T95919#1265671 (10Halfak) Hey folks, I ran the usual set of tests against this again and while I'm *finally* seeing sessions logged when I hit... [17:18:29] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Link inspector tabs (Search/External) not consistent with VE/OOUI design pattern - https://phabricator.wikimedia.org/T98370#1265682 (10Krinkle) 3NEW [17:20:49] Krinkle, Krenair so any ideas on how we should work around the load order issues of Setup.php? [17:21:29] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1265727 (10Ryasmeen) [17:21:41] edsanders: The value is only significant if it's not the default, right? [17:21:45] So it should work fine [17:21:54] if false, use $IP/extensions. If not false, use it. [17:22:07] 10VisualEditor, 10VisualEditor-MediaWiki-Links: A mouse icon appears upon hovering over the search and external link icon in link inspector but none of them are clickable object - https://phabricator.wikimedia.org/T98213#1265731 (10Ryasmeen) [17:22:12] Krenair, $IP/extensions gets set in Setup.php [17:22:24] extensions are registered before that [17:22:35] I see that, but that's only for the main run time after glbal functions and hooks are run [17:22:37] Use it inline. [17:22:45] In the relevant global function [17:22:48] 10VisualEditor, 10VisualEditor-MediaWiki-Links: The URL in the context menu of an external link is clickable after adding label to it but not otherwise - https://phabricator.wikimedia.org/T98211#1265732 (10Ryasmeen) [17:22:53] use what inline? [17:23:27] do the fallback twice? [17:23:48] Even if you can get in there before some critical point in setup, it's pointless. because that might give you a non-false value as default, but a non-default value (which is what this whole thing is for) would be et by the user in LocalSettings.php in the same synchronous execution path that calls wfLoadEtension so there's no possible place to get it set. [17:23:51] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: [Usability Suggestion] In link inspector, after switching back from External link to Search pages while a text is selected, retaining the original text in the text field w... - https://phabricator.wikimedia.org/T98210#1265754 [17:24:31] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Link inspector opens with a red highlight around the empty text field except for the first time of ... - https://phabricator.wikimedia.org/T98202#1265758 [17:24:39] Actually, $IP is set before LocalSettings.php [17:24:48] Just use it direcrtly in DefaultSettings.php [17:25:28] $wgExtensionPath = "$IP/extensions"; ? [17:25:33] Yup [17:25:44] why aren't the others like that? [17:25:52] They are, just not the ones from the stone age. [17:26:08] Because $IP used to be a substituted assignment in LocalSettings.php generated by the installer [17:26:18] But that hasn't been the case for many years. [17:26:41] Search for $IP in Defaultsettings. Plenty do use it. [17:30:12] (03CR) 10Jforrester: [C: 032] Revert "Be more selective in application of 0.8em font size" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209273 (https://phabricator.wikimedia.org/T98193) (owner: 10Bartosz Dziewoński) [17:30:58] 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Switching from VE to read mode re-embiggens the text again and makes it impossible to edit twice in one session - https://phabricator.wikimedia.org/T98193#1265808 (10Jd... [17:31:10] 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Switching from VE to read mode re-embiggens the text again and makes it impossible to edit twice in one session - https://phabricator.wikimedia.org/T98193#1262057 (10Jd... [17:31:20] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Dark highlight appears over the first match inside the link inspector after hovering over on some other article entry from the search result list - https://phabricator.wikimedia.org/T98085#1265815 (10Ryasmeen) [17:31:48] Krinkle, also the signature of wfLoadExtension/Skin seems stupid [17:31:53] ($name, $path=null) [17:32:08] 1. why would you every want to set a full fixed path [17:32:22] (03CR) 10Mvolz: [C: 04-1] "In the re-use tab, it shows the empty ref tag that the inspector automatically inserts when the inspector is created. I'm not sure if ther" [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [17:32:23] 2. if you set a path, the $name parameter becomes unused [17:33:59] (03Merged) 10jenkins-bot: Revert "Be more selective in application of 0.8em font size" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209273 (https://phabricator.wikimedia.org/T98193) (owner: 10Bartosz Dziewoński) [17:36:50] (03CR) 10Mvolz: "Also, the drop-down menu for the full list of citation templates still shows up to the right in the change." [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [17:40:48] (03PS2) 10Jforrester: jshint: Kill file-level globals and add "moment" to the list [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209044 [17:47:15] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1265866 (10Krinkle) >>! In T98080#1260022, @Esanders wrote: > https://gerrit.wikimedia.org/r/#/c/208694/ That relates to T98202, which is ab... [17:47:23] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Irregular highlighted borders appear around the text-box inside link inspector - https://phabricator.wikimedia.org/T98080#1265868 (10Krinkle) [17:47:26] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Link inspector opens with a red highlight around the empty text field except for the first time of ... - https://phabricator.wikimedia.org/T98202#1262287 [17:47:50] 10Citoid: Show "we were unable to make citation for you" instead of offering completely empty template. - https://phabricator.wikimedia.org/T92428#1265872 (10Mvolz) [17:47:55] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Link inspector should not have red highlight when still empty - https://phabricator.wikimedia.org/T98202#1265873 (10Krinkle) [17:49:36] 10VisualEditor, 10VisualEditor-Performance, 10Parsoid, 10RESTBase, 7RESTBase-API: RFC: API for retrieval and saving of top-level HTML elements / sections by element ID - https://phabricator.wikimedia.org/T94890#1265874 (10GWicke) [17:53:54] (03CR) 10Mvolz: [C: 031] "Tested without citoid extension enabled (obviously couldn't see widget there since there was no way to get to it but it didn't break anyth" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [18:00:46] (03CR) 10Jforrester: [C: 031] Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [18:00:57] edsanders: ^^^ Did you want that merged? [18:01:30] yup [18:04:54] (03CR) 10Esanders: "@Mvolz - that's an existing bug with the referenceListNode too. Will fix separately (in ve-mw)." [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [18:10:28] 10Citoid, 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: Book ISBN links are broken in content view because of linking to /w/ instead of /wiki/ - https://phabricator.wikimedia.org/T95700#1265974 (10Jdforrester-WMF) [18:11:07] (03PS5) 10Esanders: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) [18:11:09] (03PS1) 10Esanders: Don't show placeholder references in lists [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209285 [18:12:18] (03CR) 10Esanders: "@Mvolz fixed with https://gerrit.wikimedia.org/r/#/c/209285/ (and a one liner in this patch)." [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [18:16:21] (03PS2) 10Esanders: Don't show placeholder references in lists [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209285 [18:16:23] (03PS4) 10Esanders: Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 [18:19:46] 10VisualEditor, 10VisualEditor-MediaWiki, 10OOjs-UI, 10Vector, 3Editing Department 2014/15 Q4 blockers: Change OOUI MediaWiki theme to be optimized for font-size of 14px (0.875em×16px), instead of 12.8px (0.8em×16px), like MediaWiki Vector skin - https://phabricator.wikimedia.org/T97631#1248353 (10Jdforre... [18:25:15] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Link inspector tabs (Search/External) not consistent with VE/OOUI design pattern - https://phabricator.wikimedia.org/T98370#1266068 (10Krinkle) [18:25:19] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Switch link inspector over to using the new IndexLayout in OOjs UI - https://phabricator.wikimedia.org/T97878#1266070 (10Krinkle) [18:25:44] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Link inspector tabs (Search/External) not consistent with VE/OOUI design pattern - https://phabricator.wikimedia.org/T98370#1265682 (10Krinkle) I believe this would be resolved by T97878. Marking as duplicate since there isn't much value in keeping this around. [18:29:32] MatmaRex: Unless you're on it already, I'll fix pl.wiki Editnotice. [18:30:23] thanks [18:30:29] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Stage insertion and application of new links with a "preview" interstitial step before you actually apply - https://phabricator.wikimedia.org/T98273#1266112 (10Jdforrester-WMF) [18:35:22] (03CR) 10Jforrester: [C: 032] Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [18:38:10] (03Merged) 10jenkins-bot: Refactor reference source selection out into a widget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/208971 (owner: 10Esanders) [18:43:36] Visual editor loads but the toolbars doesn't come up in vagrant. By loading I mean the progress bar comes half way and disappears but slightly fades the page it usually does. Did pretty much everything said in http://www.mediawiki.org/wiki/Extension:VisualEditor#Basic_configuration_for_MediaWiki-VisualEditor , except the last one (not a private wiki). Anyone [18:43:37] knows why ? [18:50:25] tinajohnson, well what shows in the console? [18:50:34] that's the first thing you should check with these types of issues [18:52:30] ah ! "VisualEditor failed to load: Error: Module ext.visualEditor.mediawiki has failed dependencies" [18:53:43] did you run git submodule update --init --recursive [18:54:15] did run this, git submodule update --init [18:55:50] "Exception in module-execute in module ext.visualEditor.core:" [18:56:00] ^this one shows up too [18:57:30] if the error log helps, https://dpaste.de/yBoL [18:59:39] 10VisualEditor: [Regression wmf5]Cannot not open VE after switching to Read mode from Edit mode - https://phabricator.wikimedia.org/T98388#1266206 (10Ryasmeen) 3NEW [18:59:51] (03CR) 10Jforrester: [C: 031] Don't show placeholder references in lists [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209285 (owner: 10Esanders) [19:00:32] 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers, 5WMF-deploy-2015-05-06_(1.26wmf5): [Regression pre-wmf5] Switching from VE to read mode re-embiggens the text again and makes it impossible to edit twice in one session - https://phabricator.wikimedia.org/T98193#1266214 (10Ry... [19:01:09] 10VisualEditor: [Regression wmf5]Cannot not open VE after switching to Read mode from Edit mode - https://phabricator.wikimedia.org/T98388#1266217 (10Ryasmeen) p:5Triage>3Unbreak! [19:01:28] (03PS3) 10Jforrester: jshint: Kill file-level globals and add "moment" to the list [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209044 [19:01:41] (03CR) 10Jforrester: [C: 032] "Per Ed." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209044 (owner: 10Jforrester) [19:03:30] got the latest versions of VE-MW, VE core, and MW core tinajohnson? [19:05:39] the ordering on https://phabricator.wikimedia.org/T95700#1265974 is funny [19:05:49] Backlog: Urgent Fixes is first, i.e. older [19:05:52] then there's Backlog [19:06:01] Krenair: hm, this is a mediawiki vagrant created a few days ago. I believe it is up to date. [19:06:08] the actual column shown at the top is Backlog: Urgent Fixes [19:06:23] Krenair: http://newsletter-test.wmflabs.org/ this is the one [19:06:28] "A few days ago" generally isn't good enough [19:07:09] I'll make sure in few mins [19:07:23] huh [19:07:31] something is broken there [19:07:37] you have two "Edit source" tabs [19:08:08] 10VisualEditor, 10VisualEditor-EditingTools, 7Design: VisualEditor: Inspectors apply if you exit from them without a positive action, which can be a bit confusing - https://phabricator.wikimedia.org/T55546#1266228 (10Whatamidoing-WMF) We now have a "Remove" button, which is nice, but since the link is automa... [19:08:17] yeah, fishy I know.. happened when I added lines to Localsettings.php [19:08:32] I was following this http://www.mediawiki.org/wiki/Extension:VisualEditor#Setting_VisualEditor_up [19:09:21] 10VisualEditor: Add a "Cancel" button to all dialogs, inspectors, widgets, tools, etc. - https://phabricator.wikimedia.org/T97363#1266229 (10Whatamidoing-WMF) [19:09:43] (03Merged) 10jenkins-bot: jshint: Kill file-level globals and add "moment" to the list [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209044 (owner: 10Jforrester) [19:14:10] (03PS6) 10Mvolz: Redesign CiteFromIdInspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [19:16:22] Krenair: I commented out the require [19:17:10] sorry, I meant to say, I commented out the require_once statement and I get this now http://newsletter-test.wmflabs.org/ [19:18:21] I had role enabled visual editor in vagrant and then later I added the configuration statements to Localsettings.php manually. I suppose there is no need for that [19:21:48] (03CR) 10Mvolz: [C: 031] "Tested with I92d38e00eaa, works. (have not tested bonus :))" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209285 (owner: 10Esanders) [19:22:04] Krenair: VisualEditor came up [19:22:42] I did a git pull and all works fine now, thanks ! [19:24:58] I didn't do much [19:25:03] but you're very welcome :p [19:26:12] 10VisualEditor, 10VisualEditor-MediaWiki-Links: Opening the link inspector with text selected should auto-populate search results for that text - https://phabricator.wikimedia.org/T97723#1266300 (10Whatamidoing-WMF) Isn't that how it was working just a little while ago? This might be an accidental regression. [19:30:18] (03CR) 10Mvolz: [C: 031] "Tested with and without I8dc5146, works." [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/209198 (https://phabricator.wikimedia.org/T97390) (owner: 10Esanders) [19:30:47] Krenair: Vagrant did make it a lot easier, automatically configuring parsoid and all. I do have the same issue in my local wiki, I'll try a git pull there too :) [19:31:28] I've only ever seen these issues caused by mis-matching versions [19:32:10] these requirements change on a very frequent basis [19:35:15] yes, I require to use VE in a page for my extension. do I have to import VE ? [19:35:37] you want to integrate your extension with VE? [19:36:35] I mean to use it in my extension [19:37:32] Supposing I've a page where newsletters can be created. I need to have a textbox where the body can be created [19:37:44] VE can be really useful there [19:38:07] will you ever need to deal with editing existing wikitext? [19:38:47] hm, maybe yes. later in the project [19:38:53] as of now, no I guess [19:40:16] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: [Usability Suggestion] In link inspector, after switching back from External link to Search pages while a text is selected, retaining the original text in the text field w... - https://phabricator.wikimedia.org/T98210#1266370 [19:40:26] template more or less looks like the one used in Special:MassMessage. But instead of just a text box, I would love to have VE along with it [19:40:41] 10VisualEditor, 10VisualEditor-MediaWiki-Links: The URL in the context menu of an external link is clickable after adding label to it but not otherwise - https://phabricator.wikimedia.org/T98211#1266375 (10Ryasmeen) p:5Triage>3Normal [19:41:20] 10VisualEditor, 10VisualEditor-MediaWiki-Links: A mouse icon appears upon hovering over the search and external link icon in link inspector but none of them are clickable object - https://phabricator.wikimedia.org/T98213#1266378 (10Ryasmeen) p:5Triage>3Normal [19:43:15] James_F|Away: MatmaRex: In debugging the pl.wiki edit notice, I also noticed that VE's "Newarticletext" is broken. It uses a relative /wiki/... url instead of http://.../wiki/ and as such renders as plain text e.g. https://pl.wikipedia.org/wiki/ASpis_tre%C5%9Bci?veaction=edit says " the box below (see the [/wiki/Pomoc:Spis_tre%C5%9Bci help page] for more" [19:43:32] I dont know if that's a known issue, but I basically fixed it while debugging, will submit a patch soon FYI :) [19:44:19] ha. nice [19:44:45] sorry, i am busy today. i learned that i had an assignment due yesterday. [19:46:05] 10VisualEditor, 10VisualEditor-EditingTools: Please change the "highlight" color in the search results for the link inspector - https://phabricator.wikimedia.org/T98393#1266394 (10Whatamidoing-WMF) 3NEW [19:55:16] MatmaRex: Whoops. [19:55:45] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki-Links: Please change the "highlight" color in the search results for the link inspector - https://phabricator.wikimedia.org/T98393#1266423 (10Ryasmeen) [19:56:00] quite [19:57:19] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki-Links: Please change the "highlight" color in the search results for the link inspector - https://phabricator.wikimedia.org/T98393#1266440 (10Esanders) I agree, but this is an upstream issue with the OOJS-UI MediaWiki theme [19:59:34] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki-Links, 7Tracking: VisualEditor: Improvements and behaviour changes to the link input widget (tracking) - https://phabricator.wikimedia.org/T50789#1266456 (10Whatamidoing-WMF) [20:02:03] (03PS1) 10Krinkle: api: Expand helppage url before passing to 'newarticletext' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209350 [20:02:07] MatmaRex: ^ [20:02:18] MatmaRex: Ah, no worries then :) [20:03:38] (03CR) 10Jforrester: [C: 032] api: Expand helppage url before passing to 'newarticletext' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209350 (owner: 10Krinkle) [20:06:11] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki-Links, 10OOjs-UI, 7Design: Please change the "highlight" color in the search results for the link inspector - https://phabricator.wikimedia.org/T98393#1266474 (10Esanders) [20:13:04] 10Citoid, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Rendered references in context view when working from /w/index.php?title=…&veaction=edit link to /w/ not /wiki/ - https://phabricator.wikimedia.org/T95700#1266503 (10Jdforrester-WMF) [20:13:14] 10Citoid, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Rendered references in context view when working from /w/index.php?title=…&veaction=edit link to /w/ not /wiki/ - https://phabricator.wikimedia.org/T95700#1198015 (10Jdforrester-WMF) On re-testing, this is not fixe... [20:14:27] 10Citoid, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Rendered references in context view when working from /w/index.php?title=…&veaction=edit link to /w/ not /wiki/ - https://phabricator.wikimedia.org/T95700#1266523 (10Jdforrester-WMF) [20:14:35] 10Citoid, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Rendered references in context view when working from /w/index.php?title=…&veaction=edit link to /w/ not /wiki/ - https://phabricator.wikimedia.org/T95700#1198015 (10Jdforrester-WMF) [20:15:05] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki-Links, 10OOjs-UI, 7Design: Please change the "highlight" color in the search results for the link inspector - https://phabricator.wikimedia.org/T98393#1266529 (10Jdforrester-WMF) a:3Nirzar [20:16:12] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: [Usability Suggestion] In link inspector, after switching back from External link to Search pages while a text is selected, retaining the original text in the text field w... - https://phabricator.wikimedia.org/T98210#1266543 [20:16:25] 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: [Regression?] Removing edited articles from watchlist at it.wp - https://phabricator.wikimedia.org/T97838#1266545 (10Jdforrester-WMF) [20:16:42] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: Move the delete action outside of edit for link inspector and language inspector - https://phabricator.wikimedia.org/T98272#1266546 (10Jdforrester-WMF) [20:19:22] hola [20:22:58] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-Tables, 7Design: Tables: "Merge cells" and "Make header" should be in a context menu, not the toolbar - https://phabricator.wikimedia.org/T75228#1266600 (10Jdforrester-WMF) [20:23:00] 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: Table header/content cell control should be in context menu, not in toolbar - https://phabricator.wikimedia.org/T96947#1266599 (10Jdforrester-WMF) [20:31:30] (03Merged) 10jenkins-bot: api: Expand helppage url before passing to 'newarticletext' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/209350 (owner: 10Krinkle) [20:41:28] aharoni: Hey. [20:41:47] o/ RoanKattouw, I've got sad news. [20:42:02] When you have a chance, could you take a look at my last post here: https://phabricator.wikimedia.org/T95919 ? [20:47:15] 10VisualEditor: Table header/content cell control should be in context menu, not in toolbar - https://phabricator.wikimedia.org/T96947#1266763 (10Jdforrester-WMF) [20:47:34] halfak: *sigh* [20:47:48] I know that back sessions don't have an init event [20:47:50] 10VisualEditor: Table header/content cell control should be in context menu, not in toolbar - https://phabricator.wikimedia.org/T96947#1229950 (10Jdforrester-WMF) In the weekly triage meeting on 2015-05-06, we discussed this task. We felt that this was not as high priority as the other user-facing work that we n... [20:47:57] But they should have a ready event [20:49:46] 10VisualEditor, 10VisualEditor-DataModel, 10VisualEditor-MediaWiki-References, 3Editing Department 2014/15 Q4 blockers: VisualEditor: References created by templates numbered alone, not with the rest of the page, and don't show up as references to insert - https://phabricator.wikimedia.org/T52474#1266772 (1... [20:50:06] 10VisualEditor, 10VisualEditor-Performance, 3Editing Department 2014/15 Q4 blockers, 5Patch-For-Review, and 2 others: Internal nodes should eventually be in a separate document ("sub-documents") - https://phabricator.wikimedia.org/T49344#1266774 (10Jdforrester-WMF) [20:50:08] 10VisualEditor, 10VisualEditor-DataModel, 10VisualEditor-MediaWiki-References, 3Editing Department 2014/15 Q4 blockers: VisualEditor: References created by templates numbered alone, not with the rest of the page, and don't show up as references to insert - https://phabricator.wikimedia.org/T52474#1266773 (1... [20:51:35] 10VisualEditor, 10VisualEditor-ContentEditable, 3Editing Department 2014/15 Q4 blockers: VisualEditor: Pre-annotation is from up-page, so deleting and replacing the first character in a link de-links it; deleting and replacing the first bold character in a wor... - https://phabricator.wikimedia.org/T52678#1266775 [20:51:42] 10VisualEditor, 10VisualEditor-ContentEditable: VisualEditor: Pre-annotation is from up-page, so deleting and replacing the first character in a link de-links it; deleting and replacing the first bold character in a word de-bolds it; etc. - https://phabricator.wikimedia.org/T52678#1266776 (10Jdforrester-WMF) [20:51:57] 10VisualEditor, 10VisualEditor-DataModel, 10VisualEditor-MediaWiki-References: VisualEditor: References created by templates numbered alone, not with the rest of the page, and don't show up as references to insert - https://phabricator.wikimedia.org/T52474#558083 (10Jdforrester-WMF) [20:52:10] RoanKattouw, umm [20:52:22] don't we generate a new session ID on back? [20:52:27] and therefore it'll go into a separate session? [20:52:34] and this was deliberate? [20:52:59] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Stage insertion and application of new links with a "preview" interstitial step before you actually apply - https://phabricator.wikimedia.org/T98273#1266786 (10Jdforrester-WMF) In the weekly triage meeting on 2015-05-06,... [20:53:08] Yes adn yes [20:53:21] Well, technically we generate it on unload [20:53:32] after the last event has been sent [20:53:36] RoanKattouw, I can run another round or try something different to get that ready event. [20:54:01] Let me look at your log [20:54:25] kk. The worklog should be pretty clear (except for my typos -- sorry about those) [20:54:46] "Note that some sessions only have an attempt. " [20:54:49] I don't see that in the SQL result [20:55:50] Yeah. I think that might be a typo. It should have said "only have a success" [20:55:53] Hmm but the second log does have that [20:56:01] No, in the first log every session that has an attempt has a success [20:56:06] The second log does have a discrepancy [20:56:08] * halfak looks again [20:56:20] A session with only a success and nothing else [20:57:06] Also, I observe that you rely on page_title being populated [20:57:51] I don't know how it's populated, but for each of the session IDs that appear in your log, could you run a separate query to find all rows with that session ID regardless of other fields (except a restriction by timestamp to make the query not super slow) [20:59:13] halfak: ---^^ basically just to make sure that your event_page.title restrictions don't drop any events [21:00:06] +1 will give that a try. [21:00:32] * halfak runs to a meeting. [21:00:39] Will report on new queries when I'm done. [21:33:09] RoanKattouw, https://meta.wikimedia.org/wiki/Research_talk:VisualEditor%27s_effect_on_newly_registered_editors/Work_log/2015-05-06#Updates [21:33:16] Looks like filtering on page_title wasn't the issue. [21:33:25] When should I expect the "ready" event to fire? [21:34:24] ready should fire every time an edit form loads [21:34:42] So 1) right after init, 2) on preview/showchanges, 3) on back [21:38:18] RoanKattouw, I'll go see if waiting around helps and check the console for errors. [21:38:36] RoanKattouw, Krenair - This may be a somewhat surprising use of a VE JS hook: https://gerrit.wikimedia.org/r/#/c/209380/ [21:38:39] OK [21:38:58] Sorry for crossing the streams and putting part of this in #wikimedia-dev [21:39:11] Over there I suggested disabling sampling on a small wiki like mw.org to make testing easier [21:39:22] I tested it and it seems to do what I want, but it may be not entirely expected. Your review is welcome, if you have time. [21:39:30] And exclude the possibility that the session ID changes mid-session to something that gets dropped [21:40:00] halfak: Speaking of, could you look at events in the same minute to see if events from your session could possibly have been registered with a different sessionID? [21:40:43] RoanKattouw, so we'd need to do a different sessionId *and* a different page Id. [21:40:48] *title [21:41:30] Hmm yeah that does seem implausible [21:41:52] Is "ready" client-side? [21:41:56] Yes [21:42:12] ready and abort are client-side, init and save* are server-side [21:44:25] I can't seem to produce a request for event.gif in the edit pane whether I hit back or not [21:44:38] Wait... found it [21:44:48] OK. I got it without doing the back-dance [21:45:32] 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Move the delete action outside of edit for link inspector and language inspector - https://phabricator.wikimedia.org/T98272#1266970 (10Jdforrester-WMF) p:5Triage>3Normal [21:45:40] For some reason, i don't get a ready even when i click "edit" for my sandbox, but I get it on a random article. [21:45:50] 10VisualEditor, 10Parsoid, 3Editing Department 2014/15 Q4 blockers: and