From 05b10edc024bc39c4ce50ea7e632a57a099f3e03 Mon Sep 17 00:00:00 2001
From: Damien Bouvy <dbo@odoo.com>
Date: Tue, 2 May 2023 15:21:21 +0000
Subject: [PATCH] [IMP] web: controlpanel selection-actions slot

Selecting records in a list view will trigger the display of UI elements
that will allow users to manage the selection and run actions on the
selected records.

Prior to this commit, this UI section was displayed within the
'additional actions' slot of the CP, but for UI consistency when
switching to/from mobile view (where the searchbar is usually hidden),
a new slot for 'selection-actions' is added to the control panel, in the
same place as the searchbar (indeed the searchbar is hidden when a
selection occurs, so sharing the same space seems like a good idea as it
avoids clipping/overflow issues)

Part-of: odoo/odoo#116641
---
 .../search/control_panel/control_panel.xml    |  5 ++--
 .../static/src/views/list/list_controller.xml | 26 ++++++-------------
 2 files changed, 11 insertions(+), 20 deletions(-)

diff --git a/addons/web/static/src/search/control_panel/control_panel.xml b/addons/web/static/src/search/control_panel/control_panel.xml
index 312e54780f9..85343d02862 100644
--- a/addons/web/static/src/search/control_panel/control_panel.xml
+++ b/addons/web/static/src/search/control_panel/control_panel.xml
@@ -34,8 +34,9 @@
                     <span class="d-none d-xl-block me-auto"/> <!-- Spacer -->
                 </div>
 
-                <div t-if="display.layoutActions" class="o_control_panel_actions d-empty-none d-flex align-items-center justify-content-around order-2 order-lg-1 w-100 w-lg-auto">
-                    <t t-slot="layout-actions"/>
+                <div class="o_control_panel_actions d-empty-none d-flex align-items-center justify-content-start order-2 order-lg-1 w-100 w-lg-auto">
+                    <t t-if="display.layoutActions" t-slot="layout-actions"/>
+                    <t t-slot="control-panel-selection-actions"/>
                 </div>
 
                 <div class="o_control_panel_navigation d-flex flex-wrap flex-md-nowrap align-items-center justify-content-end gap-3 gap-lg-1 gap-xl-3 order-1 order-lg-2 flex-grow-1">
diff --git a/addons/web/static/src/views/list/list_controller.xml b/addons/web/static/src/views/list/list_controller.xml
index 645c7312e79..3971353caef 100644
--- a/addons/web/static/src/views/list/list_controller.xml
+++ b/addons/web/static/src/views/list/list_controller.xml
@@ -36,7 +36,14 @@
                 </t>
 
                 <t t-set-slot="control-panel-additional-actions">
-                    <div t-if="nbSelected" class="d-flex ms-3 gap-1">
+                    <CogMenu t-if="!nbSelected">
+                        <DropdownItem t-if="nbTotal and !nbSelected and activeActions.exportXlsx and isExportEnable" onSelected.bind="onDirectExportData">
+                            Export All
+                        </DropdownItem>
+                    </CogMenu>
+                </t>
+                <t t-set-slot="control-panel-selection-actions">
+                    <div t-if="nbSelected" class="d-flex gap-1">
                         <t t-call="web.ListView.Selection"/>
                         <t t-foreach="archInfo.headerButtons" t-as="button" t-key="button.id">
                             <MultiRecordViewButton
@@ -62,23 +69,6 @@
                                 onActionExecuted="() => model.load()"/>
                         </t>
                     </div>
-                    <t t-else="">
-                        <CogMenu>
-                            <DropdownItem t-if="nbTotal and !nbSelected and activeActions.exportXlsx and isExportEnable" onSelected.bind="onDirectExportData">
-                                Export All
-                            </DropdownItem>
-                            <t t-if="props.info.actionMenus">
-                                <ActionMenusItems
-                                    getActiveIds="() => model.root.selection.map((r) => r.resId)"
-                                    context="props.context"
-                                    domain="props.domain"
-                                    items="actionMenuItems"
-                                    isDomainSelected="model.root.isDomainSelected"
-                                    resModel="model.root.resModel"
-                                    onActionExecuted="() => model.load()"/>
-                            </t>
-                        </CogMenu>
-                    </t>
                 </t>
                 <t t-component="props.Renderer" list="model.root" activeActions="activeActions" archInfo="archInfo" allowSelectors="props.allowSelectors" editable="editable" openRecord.bind="openRecord" noContentHelp="props.info.noContentHelp" onAdd.bind="createRecord"/>
             </Layout>