From a1b22ae0b585ad3167c6acc44046dcefdff0dc1b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20K=C3=BChn?= <aku@odoo.com>
Date: Fri, 1 Apr 2022 14:12:26 +0000
Subject: [PATCH] [IMP] mail: introduce AttachmentViewer/onClickPrevious

Task-2579306

Part-of: odoo/odoo#88152
---
 .../components/attachment_viewer/attachment_viewer.js | 11 -----------
 .../attachment_viewer/attachment_viewer.xml           |  2 +-
 addons/mail/static/src/models/attachment_viewer.js    |  9 +++++++++
 3 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
index 1d8ec556e22..8a3b7a73d01 100644
--- a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
+++ b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
@@ -223,17 +223,6 @@ export class AttachmentViewer extends Component {
         this._stopDragging();
     }
 
-    /**
-     * Called when clicking on previous icon.
-     *
-     * @private
-     * @param {MouseEvent} ev
-     */
-    _onClickPrevious(ev) {
-        ev.stopPropagation();
-        this.attachmentViewer.previous();
-    }
-
     /**
      * Called when clicking on print icon.
      *
diff --git a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
index 89164a16c6f..b41ed6834ef 100644
--- a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
+++ b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
@@ -84,7 +84,7 @@
                     </div>
                 </t>
                 <t t-if="attachmentViewer.attachments.length > 1">
-                    <div class="o_AttachmentViewer_buttonNavigation o_AttachmentViewer_buttonNavigationPrevious o_AttachmentViewer_buttonNavigationPreviousIcon btn btn-dark position-absolute top-0 bottom-0 start-0 align-items-center justify-content-center d-flex my-auto ms-3 rounded-circle" t-on-click="_onClickPrevious" title="Previous (Left-Arrow)" role="button">
+                    <div class="o_AttachmentViewer_buttonNavigation o_AttachmentViewer_buttonNavigationPrevious o_AttachmentViewer_buttonNavigationPreviousIcon btn btn-dark position-absolute top-0 bottom-0 start-0 align-items-center justify-content-center d-flex my-auto ms-3 rounded-circle" t-on-click="attachmentViewer.onClickPrevious" title="Previous (Left-Arrow)" role="button">
                         <span class="fa fa-chevron-left" role="img"/>
                     </div>
                     <div class="o_AttachmentViewer_buttonNavigation o_AttachmentViewer_buttonNavigationNext o_AttachmentViewer_buttonNavigationNextIcon btn btn-dark position-absolute top-0 bottom-0 end-0 align-items-center justify-content-center d-flex my-auto me-3 rounded-circle" t-on-click="attachmentViewer.onClickNext" title="Next (Right-Arrow)" role="button">
diff --git a/addons/mail/static/src/models/attachment_viewer.js b/addons/mail/static/src/models/attachment_viewer.js
index 3208b83b5c8..e3b84d5a16d 100644
--- a/addons/mail/static/src/models/attachment_viewer.js
+++ b/addons/mail/static/src/models/attachment_viewer.js
@@ -91,6 +91,15 @@ registerModel({
             ev.stopPropagation();
             this.next();
         },
+        /**
+         * Called when clicking on previous icon.
+         *
+         * @param {MouseEvent} ev
+         */
+        onClickPrevious(ev) {
+            ev.stopPropagation();
+            this.previous();
+        },
         /**
          * Called when clicking on embed video player. Stop propagation to prevent
          * closing the dialog.