jenkins-bot has submitted this change and it was merged.
Change subject: Template placeholder: Don't show delete button if we're the
only transclusion part
......................................................................
Template placeholder: Don't show delete button if we're the only transclusion
part
Bug: T95252
Change-Id: I62ebe56a2c5d5a0a39a5d806b2fcb348e25f6a17
---
M modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js
1 file changed, 4 insertions(+), 0 deletions(-)
Approvals:
Jforrester: Looks good to me, but someone else must approve
Esanders: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js
b/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js
index cfc93f8..1fb745e 100644
--- a/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js
+++ b/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js
@@ -56,6 +56,10 @@
} )
.connect( this, { click: 'onRemoveButtonClick' } );
+ if ( this.placeholder.getTransclusion().parts.length === 1 ) {
+ this.removeButton.toggle( false );
+ }
+
addTemplateActionFieldLayout = new OO.ui.ActionFieldLayout(
this.addTemplateInput,
this.addTemplateButton
--
To view, visit https://gerrit.wikimedia.org/r/230814
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I62ebe56a2c5d5a0a39a5d806b2fcb348e25f6a17
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Alex Monk <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
