odoo icon indicating copy to clipboard operation
odoo copied to clipboard

[IMP][15.0] mail: Add avatar when tagging name

Open Theanhngo2905 opened this issue 2 years ago • 7 comments

Current behavior before PR:

  • Currently, when the username tag ('@') only shows the name, but in the case of multiple names that are the same, it is very easy to be mistakenly tagged.

    => Annoying unrelated people. People who need to know do not receive information

Desired behavior after PR is merged:

  • Add avatar when tagging the person's name on the suggestion section

image


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Theanhngo2905 avatar Nov 18 '22 04:11 Theanhngo2905

Pull request status dashboard

robodoo avatar Nov 18 '22 04:11 robodoo

@tuyenphung

Theanhngo2905 avatar Nov 18 '22 08:11 Theanhngo2905

@alexkuhn Can you review this PR. Thank bro cc: @tde-banana-odoo

tranngocson1996 avatar Nov 18 '22 09:11 tranngocson1996

Hello, we don't do IMP in stable versions like 15.0, only FIX. So this is master work.

As for the IMP itself, would be best to group IM status and avatar in a similar way as other visual of avatar + im status in the bottom right: Screenshot 2022-11-18 at 15 21 20 So that avatar + name are more apparent (and im_status is less visible, as it should be)

alexkuhn avatar Nov 18 '22 14:11 alexkuhn

@alexkuhn thank for reply. So we can create this pr for master branch ?

tranngocson1996 avatar Nov 18 '22 14:11 tranngocson1996

@tranngocson1996 Yes, can make a PR for master for such kind of IMP. Thanks!

alexkuhn avatar Nov 19 '22 00:11 alexkuhn

@alexkuhn thank you. @Theanhngo2905 continue handle this pr

tranngocson1996 avatar Nov 19 '22 01:11 tranngocson1996