From 67dba61ef6261be2951bd000ed86febd9af37259 Mon Sep 17 00:00:00 2001
From: nathan <nathangray.bsc+github@gmail.com>
Date: Thu, 24 Mar 2022 11:21:32 -0600
Subject: [PATCH] Et2Dialog changes - dialog can now adopt buttons from the
 template.  Preferrably set slot="buttons", but it'll find them anywhere if
 you don't - Reduce size & padding for non-template content

---
 api/js/etemplate/Et2Dialog/Et2Dialog.ts       | 28 +++++++++++++++++--
 .../etemplate/Et2Dialog/Et2DialogContent.ts   |  3 +-
 2 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/api/js/etemplate/Et2Dialog/Et2Dialog.ts b/api/js/etemplate/Et2Dialog/Et2Dialog.ts
index 90317ae4c9..d387bdc000 100644
--- a/api/js/etemplate/Et2Dialog/Et2Dialog.ts
+++ b/api/js/etemplate/Et2Dialog/Et2Dialog.ts
@@ -307,6 +307,7 @@ export class Et2Dialog extends Et2Widget(ScopedElementsMixin(SlotMixin(LionDialo
 		this._onOpen = this._onOpen.bind(this);
 		this._onClose = this._onClose.bind(this);
 		this._onClick = this._onClick.bind(this);
+		this._adoptTemplateButtons = this._adoptTemplateButtons.bind(this);
 
 		// Create this here so we have something, otherwise the creator might continue with undefined while we
 		// wait for the dialog to complete & open
@@ -530,6 +531,8 @@ export class Et2Dialog extends Et2Widget(ScopedElementsMixin(SlotMixin(LionDialo
 		// set template-name as id, to allow to style dialogs
 		this._template_widget.DOMContainer.setAttribute('id', this.__template.replace(/^(.*\/)?([^/]+?)(\.xet)?(\?.*)$/, '$2').replace(/\./g, '-'));
 
+		// Look for buttons after load
+		this._templateWidget.addEventListener("load", this._adoptTemplateButtons.bind(this));
 	}
 
 	render()
@@ -621,10 +624,31 @@ export class Et2Dialog extends Et2Widget(ScopedElementsMixin(SlotMixin(LionDialo
 		{
 			return this.buttons;
 		}
-		else
+	}
+
+	/**
+	 * Search for buttons in the template, and try to slot them
+	 *
+	 * We don't want to just grab them all, as there may be other buttons.
+	 */
+	_adoptTemplateButtons()
+	{
+		// Check for something with buttons slot set
+		let search_in = this._template_widget?.DOMContainer || this._overlayContentNode._contentNode;
+		let template_buttons = search_in.querySelectorAll('[slot="buttons"]') ||
+			// Look for a dialog footer, which will contain several buttons and possible other widgets
+			search_in.querySelectorAll(".dialogFooterToolbar") ||
+			// Look for buttons anywhere
+			search_in.querySelectorAll("et2-button");
+		if(template_buttons)
 		{
-			// TODO: Find buttons in template
+			template_buttons.forEach((button) =>
+			{
+				button.setAttribute("slot", "buttons");
+				this._overlayContentNode.appendChild(button);
+			})
 		}
+		return template_buttons;
 	}
 
 	/**
diff --git a/api/js/etemplate/Et2Dialog/Et2DialogContent.ts b/api/js/etemplate/Et2Dialog/Et2DialogContent.ts
index 5b3f58afe9..2bbe38dd5e 100644
--- a/api/js/etemplate/Et2Dialog/Et2DialogContent.ts
+++ b/api/js/etemplate/Et2Dialog/Et2DialogContent.ts
@@ -12,10 +12,9 @@ export class Et2DialogContent extends Et2Widget(LitElement)
 			:host {
 			display: block;
 			min-width: 200px;
-			min-height: 60px;
+			min-height: 32px;
 			}
 			.dialog {
-				padding: 5px;
 			}
 			.dialog_icon {
 				margin-right: 2ex;