Logos in "About" screen are hardly viisble in dark theme

[sivaraam]

Summary:
The logos in About screen were recently changed in #3444. But the new logos are hardly visible in dark theme. We should show appropriate logos which are visible well in dark theme.

Screen-shots:

[bharathbiju]

guess this issue has been cleared @sivaraam
if so could you please close the issue @nicolas-raoul

[madhurgupta10]

guess this issue has been cleared @sivaraam
if so could you please close the issue @nicolas-raoul

@bharathbiju I don't think this issue is solved yet!

[madhurgupta10]

Pinging @misaochan and @nicolas-raoul for suggestions!

[macgills]

Change them to white? Are these vectors? You can change 1 line to get them to be white, add that as a new icon and then use attributes set in light/dark theme to switch between the images, there are some examples for colors already there

[sivaraam]

Change them to white?

Changing color is nice. But changing it to white might make the icons stand out in the black background and they might possibly attract attention away from the text (which seems to be greyish in color). It would be nice if we could use a different color other than white.

But it seems, both Facebook and GitHub are against changing colors. So, I think it's better if we just use the light colored icons provided by them as it is.

• GitHub has a light colored icons in it's mark logo zip
• Facebook has both grey and white icons but I think we can just go with white to be consistent.
• For the globe icon we use Material public icon and we can download a white colored icon from their website.

[madhurgupta10]

How about keeping the color of logos black and adding a small white circle in the background similar to the Github Android App Icon.

[macgills]

Use white, I think we are all in agreement.

Now who wants the ticket assigned?

[madhurgupta10]

@macgills The issue seems to be perfect for first-time contributors.

[isnirp]

hello @macgills @nicolas-raoul I would like to handle this issue.

[sivaraam]

@isnirp I've assigned this to you. Let us know how you get on. Feel free to ask for any help, if need be. :)

[isnirp]

@sivaraam it's resolved. created an attr for images present in about. do I do a PR?

[sivaraam]

@isnirp That's great!

do I do a PR?

Sure, feel free to open a PR :)