mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-15 10:35:48 +00:00
76f563c929
This was done in 3 rather "random" places: 1. Whenever a template is manually added. But rather late, after the template was added, in an event handler that is about focus behavior. It should not continue to manipulate the template that was just added. 2. When the dialog opens with a template preloaded by name, as it is done from the citation menu. 3. When the dialog is about to finish loading. This patch fixes 2 issues: * Get rid of a duplicate call (number 2 and 3) when using the citation menu. * Move number 1 to a place where it's executed much earlier, and only when the user clicks "add template" in a template placeholder. There is no other way to add a template to an existing transclusion, but it's still a more appropriate place I feel. Bug: T311069 Change-Id: I8a65ad703b95ba2092e9ef73493e9903e96b0dd6 |
||
---|---|---|
.. | ||
ve.ui.MWAddParameterPage.js | ||
ve.ui.MWAdvancedSettingsPage.js | ||
ve.ui.MWCategoriesPage.js | ||
ve.ui.MWLanguagesPage.js | ||
ve.ui.MWParameterPage.js | ||
ve.ui.MWSettingsPage.js | ||
ve.ui.MWTemplatePage.js | ||
ve.ui.MWTemplatePlaceholderPage.js | ||
ve.ui.MWTemplatesUsedPage.js | ||
ve.ui.MWTransclusionContentPage.js |