New JS icon is not legible/recognizable in small size

[MunifTanjim]

• fix: update js icon #366

This is the new JS Icon:

In small size, it's not legible/recognizable at all:

The previous one was much better:

Same for CJS and MJS:

• fix: update cjs & mjs icons #368

[MunifTanjim]

I think we should revert those 2 PRs.

cc: @alex-courtis @ronitkrshah

[ronitkrshah]

@MunifTanjim Well it's looking fine for me in Termux also in my PC

[MunifTanjim]

I guess it's terminal dependant.

MacOS

Alacritty:

Kitty:

WezTerm:

[ronitkrshah]

It depends on the font size of your terminal. Naturally, if you are using a smaller font size, it will show a small icon, and vice versa for a bigger font size.

[MunifTanjim]

Also depends on how the terminal renders the font.

In either way, the new icon is not very recognizable in small size. The previous one was immediately recognizable.

[ronitkrshah]

Hmm ....Then we should revert this 2 PRs

[gegoune]

I am leaning toward using 'official' icons as much as possible, mainly in order to avoid discussions which icon fits user's preferences best. @MunifTanjim obviously I am not having a go or disrespecting you and your concern here, it's just that this icon is what it is. Just my opinion.

[MunifTanjim]

Makes sense. No worries.

[gegoune]

Let's hear @alex-courtis' and @Akmadan23 opinions on this as well.

[alex-courtis]

Let's stick to the official icons.

If the official icon is not usable, it's something that should and very likely will be fixed in the font itself.

[alex-courtis]

Closing off; we can revisit and raise upstream if it continues to be a problem.