From 50fc5ff3471e08de64745cee8c8fa3dbcd294e0a Mon Sep 17 00:00:00 2001
From: "stefanorigano (SRI)" <sri@odoo.com>
Date: Thu, 20 May 2021 13:47:56 +0000
Subject: [PATCH] [REF] web: switch company menu design

---
 .../src/legacy/scss/switch_company_menu.scss  | 18 +------------
 .../switch_company_menu.xml                   | 25 +++++++++++--------
 2 files changed, 15 insertions(+), 28 deletions(-)

diff --git a/addons/web/static/src/legacy/scss/switch_company_menu.scss b/addons/web/static/src/legacy/scss/switch_company_menu.scss
index 6bc4fa4d5f6..5d17864303e 100644
--- a/addons/web/static/src/legacy/scss/switch_company_menu.scss
+++ b/addons/web/static/src/legacy/scss/switch_company_menu.scss
@@ -4,21 +4,5 @@
         > a .oe_topbar_name {
             @include o-text-overflow($max-width: 250px);
         }
-        .dropdown-item {
-            cursor: pointer;
-
-            &:hover {
-                background-color: white;
-            }
-
-            .o_py {
-                padding-top: 0.2rem !important;            
-                padding-bottom: 0.2rem !important;            
-                
-                &:hover {
-                    background-color: #e9ecef !important;
-                }
-            }
-        }
-    }    
+    }
 }
diff --git a/addons/web/static/src/webclient/switch_company_menu/switch_company_menu.xml b/addons/web/static/src/webclient/switch_company_menu/switch_company_menu.xml
index 66c366941d7..1572b9c3ffc 100644
--- a/addons/web/static/src/webclient/switch_company_menu/switch_company_menu.xml
+++ b/addons/web/static/src/webclient/switch_company_menu/switch_company_menu.xml
@@ -21,35 +21,38 @@
     <div>
         <t t-set="isCompanySelected" t-value="selectedCompanies.includes(company.id)"/>
         <t t-set="isCurrent" t-value="company.id === companyService.currentCompany.id"/>
-        <DropdownItem>
-            <div class="dropdown-item d-flex py-0 px-0" data-menu="company" t-att-data-company-id="company.id">
+        <DropdownItem class="p-0 bg-white">
+            <div class="d-flex" data-menu="company" t-att-data-company-id="company.id">
                 <div
                     role="menuitemcheckbox"
                     t-att-aria-checked="isCompanySelected"
                     t-att-aria-label="company.name"
+                    t-att-title="(isCompanySelected ? 'Hide ' : 'Show ') + company.name + ' content.'"
                     tabindex="0"
-                    class="ml-auto pl-3 pr-3 border border-top-0 border-left-0 border-bottom-0 toggle_company o_py"
+                    class="border-right toggle_company o_py"
+                    t-attf-class="{{isCurrent ? 'border-primary' : ''}}"
                     t-on-click.stop="toggleCompany(company.id)">
 
-                    <span style="height: 2rem;">
-                        <i class="fa fa-fw pt-2" t-attf-class="{{isCompanySelected ? 'fa-check-square' : 'fa-square-o'}}"/>
+                    <span class="btn btn-light border-0 p-2">
+                        <i class="fa fa-fw py-2" t-att-class="isCompanySelected ? 'fa-check-square text-primary' : 'fa-square-o'"/>
                     </span>
-
                 </div>
 
                 <div
                     role="button"
                     t-att-aria-pressed="isCurrent"
-                    aria-label="Switch to this company"
+                    t-att-aria-label="'Switch to ' + company.name "
+                    t-att-title="'Switch to ' + company.name "
                     tabindex="0"
-                    class="d-flex flex-grow-1 align-items-center py-0 log_into pl-3 o_py"
-                    t-attf-style="{{isCurrent ? 'background-color: lightgrey;' : ''}}"
+                    class="d-flex flex-grow-1 align-items-center py-0 log_into pl-2 o_py"
+                    t-att-class="isCurrent ? 'alert-primary ml-1 mr-2' : 'btn btn-light font-weight-normal border-0'"
                     t-on-click="logIntoCompany(company.id)">
 
-                    <span class='mr-3 company_label' t-attf-class="{{selectedCompanies.includes(company.id) ? '' : 'text-muted'}}">
+                    <span
+                        class='company_label pr-3'
+                        t-att-class="isCurrent ? 'text-900 font-weight-bold' : 'ml-1'">
                         <t t-esc="company.name"/>
                     </span>
-
                 </div>
             </div>
         </DropdownItem>