[IMP] hr_employee : make UX improvements

task - 3203838

closes odoo#114280

Related: odoo/enterprise#37749
Signed-off-by: Kevin Baptiste <kba@odoo.com>

Author: jugj-odoo

addons/hr/views/hr_employee_views.xml

@@ -112,7 +112,7 @@
                                                 options='{"always_reload": True, "highlight_first_line": True}'/>
                                             <field name="work_location_id" context="{'default_address_id': address_id}" />
                                         </group>
-                                        <group name="managers" string="Approvers" class="hide-group-if-empty">
+                                        <group name="managers" string="Approvers" class="hide-group-if-empty" invisible="1">
                                             <!-- is overridden in other hr related modules -->
                                         </group>
                                         <group name="departure" string="Departure" attrs="{'invisible': [('active', '=', True)]}">
@@ -196,7 +196,7 @@
                                     <group string='Payroll' name="payroll_group">
                                         <field name="job_id"/>
                                     </group>
-                                    <group name="application_group"/>
+                                    <group name="application_group" string="Application Settings" invisible="1"/>
                                 </group>
                             </page>
                         </notebook>

addons/hr_fleet/views/employee_views.xml

@@ -15,7 +15,7 @@
                 </button>
             </div>
             <group name="application_group" position="attributes">
-                <attribute name="string">Application Settings</attribute>
+                <attribute name="invisible">0</attribute>
             </group>
             <group name="application_group" position="inside">
                 <field name="mobility_card" string="Fleet Mobility Card"/>

addons/hr_holidays/models/hr_employee.py

@@ -86,8 +86,7 @@ def _compute_allocation_count(self):
             ('holiday_status_id.active', '=', True),
             ('holiday_status_id.requires_allocation', '=', 'yes'),
             ('state', '=', 'validate'),
-            '|',
-            ('date_to', '=', False),
+            ('date_from', '<=', current_date),
             ('date_to', '>=', current_date),
         ], ['number_of_days:sum', 'employee_id'], ['employee_id'])
         rg_results = dict((d['employee_id'][0], {"employee_id_count": d['employee_id_count'], "number_of_days": d['number_of_days']}) for d in data)

addons/hr_hourly_cost/views/hr_employee_views.xml

@@ -7,7 +7,7 @@
         <field name="priority" eval="40"/>
         <field name="arch" type="xml">
             <group name="application_group" position="attributes">
-                <attribute name="string">Application Settings</attribute>
+                <attribute name="invisible">0</attribute>
             </group>
             <group name="application_group" position="inside">
                 <label for="hourly_cost"/>

addons/hr_org_chart/static/src/fields/hr_org_chart.xml

@@ -35,7 +35,7 @@
             t-att-style="'background-image:url(\'/web/image/hr.employee.public/' + employee.id + '/avatar_1024/\')'"/>
     </div>
 
-    <div class="d-flex flex-grow-1 align-items-start justify-content-between position-relative px-3">
+    <div class="d-flex flex-grow-1 align-items-center justify-content-between position-relative px-3">
         <a t-if="!is_self" t-att-href="employee.link" class="o_employee_redirect d-flex flex-column" t-att-data-employee-id="employee.id" t-on-click.prevent="() => this._onEmployeeRedirect(employee.id)">
             <b class="o_media_heading m-0 fs-6" t-esc="employee.name"/>
             <small class="text-muted fw-bold" t-esc="employee.job_title"/>