From ffdf6df09c3c78e510df771c390529aaa6198291 Mon Sep 17 00:00:00 2001
From: hadi <hnategh@gmail.com>
Date: Mon, 22 May 2023 11:45:56 +0200
Subject: [PATCH] Make et2link file type draggable

---
 api/js/etemplate/Et2Link/Et2Link.ts     | 45 ++++++++++++++++++++++-
 api/js/etemplate/Et2Link/Et2LinkList.ts | 47 ++-----------------------
 2 files changed, 46 insertions(+), 46 deletions(-)

diff --git a/api/js/etemplate/Et2Link/Et2Link.ts b/api/js/etemplate/Et2Link/Et2Link.ts
index 44a4c5c14e..4ff883d90b 100644
--- a/api/js/etemplate/Et2Link/Et2Link.ts
+++ b/api/js/etemplate/Et2Link/Et2Link.ts
@@ -189,7 +189,7 @@ export class Et2Link extends ExposeMixin<Et2Widget>(Et2Widget(LitElement)) imple
 				.replace(/ /g, '\u00a0');
 		}
 		return html`
-            <div part="base" class="link et2_link">
+            <div part="base" class="link et2_link" draggable="${this.app == 'file'}" @dragstart=${this._handleDragStart.bind(this, this.dataset)}>
                 ${this._thumbnailTemplate({id: this.entryId, app: this.app, ...this.dataset})}
                 <span part="title" class="link__title">${title}</span>
                 <span part="remark" class="link__remark">${this.dataset.remark}</span>
@@ -330,6 +330,49 @@ export class Et2Link extends ExposeMixin<Et2Widget>(Et2Widget(LitElement)) imple
 		}
 	}
 
+	/**
+	 * Handle dragstart event for dragging out a file
+	 *
+	 * @param _data
+	 * @param _ev
+	 * @protected
+	 */
+	protected _handleDragStart (_data, _ev)
+	{
+		// // Unfortunately, dragging files is currently only supported by Chrome
+		if(navigator && navigator.userAgent.indexOf('Chrome') >= 0) {
+
+			if (_ev.dataTransfer == null) {
+				return;
+			}
+			if (_data && _data.type && _data.download_url) {
+				_ev.dataTransfer.dropEffect = "copy";
+				_ev.dataTransfer.effectAllowed = "copy";
+
+				let url = _data.download_url;
+
+				// NEED an absolute URL
+				if (url[0] == '/') url = egw.link(url);
+				// egw.link adds the webserver, but that might not be an absolute URL - try again
+				if (url[0] == '/') url = window.location.origin + url;
+
+				// Unfortunately, dragging files is currently only supported by Chrome
+				if (navigator && navigator.userAgent.indexOf('Chrome')) {
+					_ev.dataTransfer.setData("DownloadURL", _data.type + ':' + _data.title + ':' + url);
+				}
+
+				// Include URL as a fallback
+				_ev.dataTransfer.setData("text/uri-list", url);
+			}
+
+			if (_ev.dataTransfer.types.length == 0) {
+				// No file data? Abort: drag does nothing
+				_ev.preventDefault();
+				return;
+			}
+		}
+	}
+
 	_handleClick(_ev : MouseEvent) : boolean
 	{
 		// If we don't have app & entryId, nothing we can do
diff --git a/api/js/etemplate/Et2Link/Et2LinkList.ts b/api/js/etemplate/Et2Link/Et2LinkList.ts
index 0d16d1a76a..a77635499d 100644
--- a/api/js/etemplate/Et2Link/Et2LinkList.ts
+++ b/api/js/etemplate/Et2Link/Et2LinkList.ts
@@ -207,57 +207,14 @@ export class Et2LinkList extends Et2LinkString
 	protected _rowTemplate(link) : TemplateResult
 	{
 		return html`
-            <div id="${this._get_row_id(link)}" draggable="${link.app == 'file'}" 
-                 @dragstart=${this._handleDragStart.bind(this, link)}
-				 @mouseover=${this._handleRowHover}
+            <div id="${this._get_row_id(link)}"  
+                 @mouseover=${this._handleRowHover}
                  @mouseout=${this._handleRowHover}
                  @contextmenu=${this._handleRowContext}>
                 <slot name="${this._get_row_id(link)}"></slot>
             </div>`;
 	}
 
-	/**
-	 * Handle dragstart event for dragging out a file
-	 *
-	 * @param _data
-	 * @param _ev
-	 * @protected
-	 */
-	protected _handleDragStart (_data, _ev)
-	{
-		// // Unfortunately, dragging files is currently only supported by Chrome
-		if(navigator && navigator.userAgent.indexOf('Chrome') >= 0) {
-
-			if (_ev.dataTransfer == null) {
-				return;
-			}
-			if (_data && _data.type && _data.download_url) {
-				_ev.dataTransfer.dropEffect = "copy";
-				_ev.dataTransfer.effectAllowed = "copy";
-
-				let url = _data.download_url;
-
-				// NEED an absolute URL
-				if (url[0] == '/') url = egw.link(url);
-				// egw.link adds the webserver, but that might not be an absolute URL - try again
-				if (url[0] == '/') url = window.location.origin + url;
-
-				// Unfortunately, dragging files is currently only supported by Chrome
-				if (navigator && navigator.userAgent.indexOf('Chrome')) {
-					_ev.dataTransfer.setData("DownloadURL", _data.type + ':' + _data.title + ':' + url);
-				}
-
-				// Include URL as a fallback
-				_ev.dataTransfer.setData("text/uri-list", url);
-			}
-
-			if (_ev.dataTransfer.types.length == 0) {
-				// No file data? Abort: drag does nothing
-				_ev.preventDefault();
-				return;
-			}
-		}
-	}
 
 	/**
 	 * Handle show/hide delete button