From 4736b2afd548f6815b17f4eb1012b497c1490bc6 Mon Sep 17 00:00:00 2001
From: Brieuc-brd <brd@odoo.com>
Date: Fri, 4 Aug 2023 08:47:46 +0000
Subject: [PATCH] [FIX] web: ListView selection and action icons

=== ISSUE 1 ===
Prior to this commit, there was a missing padding on  the
`o_list_selection_box`.

=== ISSUE 2 ===
On mobile, Cog and Print icons had no margin and were stick next to
their label.

This commit fixes these issues.

task-3454841
Part of task-3326263

closes odoo/odoo#134219

X-original-commit: 80f82816b17d32efbf4cbe4a1956b2457eac8ad9
Signed-off-by: Pierre Paridans (app) <app@odoo.com>
---
 addons/web/static/src/search/action_menus/action_menus.xml | 4 ++--
 addons/web/static/src/views/list/list_controller.xml       | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/addons/web/static/src/search/action_menus/action_menus.xml b/addons/web/static/src/search/action_menus/action_menus.xml
index 58a55ecafa1..e025310f7e9 100644
--- a/addons/web/static/src/search/action_menus/action_menus.xml
+++ b/addons/web/static/src/search/action_menus/action_menus.xml
@@ -5,7 +5,7 @@
         <div class="o_cp_action_menus d-flex pe-2 gap-1">
             <Dropdown t-if="printItems.length" class="'d-inline-block'" togglerClass="'btn btn-secondary'" hotkey="'shift+u'">
                 <t t-set-slot="toggler">
-                    <i class="me-md-1 fa fa-print"/>
+                    <i class="fa fa-print me-1"/>
                     <span class="o_dropdown_title">Print</span>
                 </t>
                 <t t-foreach="printItems" t-as="item" t-key="item.key">
@@ -17,7 +17,7 @@
 
             <Dropdown t-if="actionItems.length" class="'d-inline-block'" togglerClass="'btn btn-secondary'" hotkey="'u'">
                 <t t-set-slot="toggler">
-                    <i class="me-md-1 fa fa-cog"/>
+                    <i class="fa fa-cog me-1"/>
                     <span class="o_dropdown_title">Actions</span>
                 </t>
                 <t t-foreach="actionItems" t-as="item" t-key="item.key">
diff --git a/addons/web/static/src/views/list/list_controller.xml b/addons/web/static/src/views/list/list_controller.xml
index 25f55725a45..515adb74427 100644
--- a/addons/web/static/src/views/list/list_controller.xml
+++ b/addons/web/static/src/views/list/list_controller.xml
@@ -100,8 +100,8 @@
     </t>
 
     <t t-name="web.ListView.Selection">
-        <div class="o_list_selection_box list-group flex-row me-2" role="alert">
-            <span class="list-group-item active d-flex align-items-center alisgn-self-stretch pe-0 py-0 rounded-1 lh-1">
+        <div class="o_list_selection_box list-group flex-row" role="alert">
+            <span class="list-group-item active d-flex align-items-center pe-0 pe-md-3 py-0 rounded-1 lh-1">
                 <span t-if="isDomainSelected">All <b t-esc="nbTotal"/> selected</span>
                 <t t-else="">
                     <b class="me-1" t-esc="nbSelected"/> selected
@@ -109,7 +109,7 @@
                         <i class="oi oi-fw oi-arrow-right"/> Select all <span t-esc="nbTotal"/>
                     </a>
                 </t>
-                <a href="#" title="Unselect All" class="o_list_unselect_all btn btn-link py-0 ms-2" t-on-click="onUnselectAll">
+                <a href="#" title="Unselect All" class="o_list_unselect_all btn btn-link ms-0 ms-md-2 py-0" t-on-click="onUnselectAll">
                     <i class="oi oi-close oi-large align-text-bottom"/>
                 </a>
             </span>