From 0cc569004dbacd9dcadb255ec9eccb76b09dede8 Mon Sep 17 00:00:00 2001
From: mano-odoo <mano@odoo.com>
Date: Wed, 16 Aug 2023 13:29:07 +0200
Subject: [PATCH] [FIX] web: o_quick_assign same color as the clock

As the Activity button (clock icon) is the same kind of action
it make sense to apply the text-muted on the quick assign.

To make the hover clearer we apply a text-dark on hover using the
btn-link + contextual classes function. We apply the same to the
`subtask_list_button` and milestones for consistency.

task-3382159

closes odoo/odoo#134196

X-original-commit: 4b8b6e2155eca66652b4a57dfff27b0e81d62573
Signed-off-by: Pierre Paridans (app) <app@odoo.com>
---
 addons/mail/static/src/core/web/activity_button.js             | 2 +-
 addons/project/views/project_project_views.xml                 | 3 ++-
 addons/project/views/project_task_views.xml                    | 3 ++-
 .../many2many_tags_avatar/many2many_tags_avatar_field.xml      | 2 +-
 .../src/views/fields/many2one_avatar/many2one_avatar_field.xml | 2 +-
 5 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/addons/mail/static/src/core/web/activity_button.js b/addons/mail/static/src/core/web/activity_button.js
index a72a7d2347a..88330b003f2 100644
--- a/addons/mail/static/src/core/web/activity_button.js
+++ b/addons/mail/static/src/core/web/activity_button.js
@@ -46,7 +46,7 @@ export class ActivityButton extends Component {
                     classes.push(this.props.record.data.activity_type_icon);
                     break;
                 }
-                classes.push("fa-clock-o");
+                classes.push("fa-clock-o btn-link text-dark");
                 break;
         }
         return classes.join(" ");
diff --git a/addons/project/views/project_project_views.xml b/addons/project/views/project_project_views.xml
index ec99a96582e..780cddc5c12 100644
--- a/addons/project/views/project_project_views.xml
+++ b/addons/project/views/project_project_views.xml
@@ -479,7 +479,8 @@
                                                 </div>
                                             </a>
                                             <a groups='project.group_project_milestone' t-if="record.allow_milestones and record.allow_milestones.raw_value and record.milestone_count.value &gt; 0"
-                                                class="o_kanban_inline_block text-muted small"
+                                                class="o_kanban_inline_block btn-link text-dark small"
+                                                role="button"
                                                 name="action_get_list_view"
                                                 type="object"
                                                 t-attf-title="#{record.milestone_count_reached.value} Milestones reached out of #{record.milestone_count.value}"
diff --git a/addons/project/views/project_task_views.xml b/addons/project/views/project_task_views.xml
index 76407ec5e05..0e5b51683ee 100644
--- a/addons/project/views/project_task_views.xml
+++ b/addons/project/views/project_task_views.xml
@@ -599,7 +599,8 @@
                                         <t t-if="record.project_id.raw_value and record.subtask_count.raw_value">
                                             <a t-if="record.project_id.raw_value and record.subtask_count.raw_value and record.subtask_count.value &gt; record.closed_subtask_count.value"
                                                t-attf-title="{{ record.closed_subtask_count.value }} sub-tasks closed out of {{ record.subtask_count.value }}"
-                                               class="subtask_list_button text-muted">
+                                               class="subtask_list_button btn-link text-dark"
+                                               role="button">
                                                 <span class="fa fa-check-square-o me-1"/>
                                                 <t t-out="record.closed_subtask_count.value + '/' + record.subtask_count.value"/>
                                             </a>
diff --git a/addons/web/static/src/views/fields/many2many_tags_avatar/many2many_tags_avatar_field.xml b/addons/web/static/src/views/fields/many2many_tags_avatar/many2many_tags_avatar_field.xml
index d0c51ce77a9..a82a2d1b580 100644
--- a/addons/web/static/src/views/fields/many2many_tags_avatar/many2many_tags_avatar_field.xml
+++ b/addons/web/static/src/views/fields/many2many_tags_avatar/many2many_tags_avatar_field.xml
@@ -29,7 +29,7 @@
         <xpath expr="//t[@t-foreach='visibleTags']" position="before">
             <t t-if="canDisplayQuickAssignAvatar">
                 <a t-on-click.stop.prevent="openPopover" tabIndex="-1" href="#" title="Assign"
-                   aria-label="Assign" class="fa fa-user-plus o_quick_assign o_m2m_avatar d-flex align-items-center"/>
+                   aria-label="Assign" class="o_quick_assign fa fa-user-plus o_m2m_avatar btn-link d-flex align-items-center text-dark" role="button"/>
             </t>
         </xpath>
         <xpath expr="//span[hasclass('o_m2m_avatar_empty')]" position="attributes">
diff --git a/addons/web/static/src/views/fields/many2one_avatar/many2one_avatar_field.xml b/addons/web/static/src/views/fields/many2one_avatar/many2one_avatar_field.xml
index 3637199ac95..5f0f0aa0cb7 100644
--- a/addons/web/static/src/views/fields/many2one_avatar/many2one_avatar_field.xml
+++ b/addons/web/static/src/views/fields/many2one_avatar/many2one_avatar_field.xml
@@ -21,7 +21,7 @@
         <xpath expr="//span[hasclass('o_m2o_avatar_empty')]" position="replace">
             <t t-if="props.record.data[props.name] === false and props.isEditable">
                 <a t-on-click.stop.prevent="openPopover" tabIndex="-1" href="#" title="Assign"
-                   aria-label="Assign" class="fa fa-user-plus o_quick_assign d-flex align-items-center"/>
+                   aria-label="Assign" class="o_quick_assign fa fa-user-plus btn-link d-flex align-items-center text-dark" role="button"/>
             </t>
         </xpath>
     </t>