[UI] Update placeholders and plugins logos

[andersonjeccel]

Q	A
Bug fix? (use the a.b branch)	[ ]
New feature/enhancement? (use the a.x branch)	[ ]
Deprecations?	[ ]
BC breaks? (use the c.x branch)	[ ]
Automated tests included?	[ ]
Related user documentation PR URL	mautic/mautic-documentation#...
Related developer documentation PR URL	mautic/developer-documentation#...
Issue(s) addressed	Fixes #13147

Description:

Based on feedback:

Imagery: replace the placeholder images by more modern ones

The user is suggesting that the current placeholder images used in the platform appear outdated and should be replaced with more contemporary visuals. This feedback indicates that the imagery does not align with modern design trends or the user’s aesthetic expectations.

The user’s broader need is for a visually appealing platform that reflects current design standards and resonates with a modern audience.

To address this feedback, this PR updates Mautic placeholder images with modern, high-quality images and logos that better represent current brands.

• fix .img-responsive with to fill 100%

Steps to test this PR:

1. Open this PR on Gitpod or pull down for testing locally (see docs on testing PRs here)
2. Open Contacts > create new > See the new placeholder for contacts without profile pic
3. Open right sidebar > Plugins > Look at the new social and brands logos

[shinde-rahul]

LGTM!

Thanks @andersonjeccel

[LordRembo]

I have the same remark as with PR #13629: if we're going to propose a UI overhaul / design system change anyway, is it worth pushing design updates for this design? @Mike-Dropsolid

[Mike-Dropsolid]

I have the same remark as with PR #13629: if we're going to propose a UI overhaul / design system change anyway, is it worth pushing design updates for this design? @Mike-Dropsolid

This wouldn't have to be redone at least. Especially the logos update feels useful to me already. I don't mind.

[LordRembo]

Alrighty, 👍

[LordRembo]

Good to go then!

[escopecz]

There is a failing test that must be fixed

[andersonjeccel]

I'll fix next Tuesday

[andersonjeccel]

The local file is .svg and Gravatar's .png

Actual code:

Gravatar
$this->assertSame('https://www.gravatar.com/avatar/96f1b78c73c1ee806cf6a4168fe9bf77?s=250&d=http%3A%2F%2Flocalhost%2Fimages%2Favatar.png', $avatar, 'Gravatar image should be returned');

Localhost
$this->assertSame('http://localhost/images/avatar.svg', $avatar, 'Default image image should be returned');

Even if the second is .png, it throws an error bc the existing file is .svg (less size, improves platform's performance)

[andersonjeccel]

@LordRembo help needed to fix the test 🙏🏻

[LordRembo]

@andersonjeccel You'll need to change the reference in the test. See file app/bundles/LeadBundle/Tests/Helper/AvatarHelperTest.php
Somewhere at the bottom, you'll see multiple references to avatar.png, you need to change the extension to .svg if that is the new format for the default avatar.
Maybe best to do another search for 'avatar.png' in the project, to check if there are any other incorrect references remaining.

[andersonjeccel]

@escopecz Seems fixed

[escopecz]

Nice! Is this ready for tests and review? If so, can it be moved out of draft?

[andersonjeccel]

@escopecz Now it's ready!

[escopecz]

the plugin logo changes look great. I'm not sure about the contact avatar. Is that a head with a steering wheel? If so, why?

[andersonjeccel]

It's an user icon

How could I make it better?

[escopecz]

Now I get it. I looked at all the Remix user icons and when I saw this:

https://remixicon.com/icon/user-3-line

I understood that the icon you selected is a head with shoulders. Not a steering wheel. But it's more abstract. I like the user, user-1 and user-2 better than user-6 as those I understand is a silhouette of a person. I couldn't get it from user-6 even though I tried.

[andersonjeccel]

Yeah, makes sense
I’ll change it soon!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.50%. Comparing base (3051381) to head (dfbf021).
Report is 34 commits behind head on 5.x.

Additional details and impacted files

@@             Coverage Diff              @@
##                5.x   #13628      +/-   ##
============================================
+ Coverage     61.29%   61.50%   +0.21%     
- Complexity    33998    34068      +70     
============================================
  Files          2238     2241       +3     
  Lines        101631   101852     +221     
============================================
+ Hits          62290    62647     +357     
+ Misses        39341    39205     -136     

Files	Coverage Δ
...les/LeadBundle/Twig/Helper/DefaultAvatarHelper.php	100.00% <100.00%> (ø)

... and 56 files with indirect coverage changes

[andersonjeccel]

@escopecz Say hi to Mrs Random