Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New icons lack transparent background #27930

Closed
namazso opened this issue Aug 18, 2024 · 4 comments · Fixed by #28040
Closed

New icons lack transparent background #27930

namazso opened this issue Aug 18, 2024 · 4 comments · Fixed by #28040
Assignees
Labels

Comments

@namazso
Copy link

namazso commented Aug 18, 2024

Your use case

What would you like to do?

Change the icons

Why would you like to do it?

To have an icon that isn't ugly

How would you like to achieve it?

Replacing the icon

Have you considered any alternatives?

Putting up with it

Additional context

image

@namazso
Copy link
Author

namazso commented Aug 18, 2024

This was introduced in #27900 that's fixing #27886.

@dosubot dosubot bot added A-Appearance S-Tolerable Low/no impact on users labels Aug 18, 2024
@dbkr
Copy link
Member

dbkr commented Aug 19, 2024

This isn't particularly helpful or constructive. If what you're saying is what the icon has a white background in some circumstances, which I'm guessing from your screenshot, you could provide more info on how you're running element. Otherwise, this is a candidate for closing.

@namazso
Copy link
Author

namazso commented Aug 19, 2024

The files in the PR have white background on their own. That particular instance comes from PWA installation, and can be reproduced across any combination of Windows, Linux (GNOME), Edge, Brave.

I don’t know if it’s intentional or a bug, but it looks awful in either case.

@dbkr dbkr changed the title New icons look terrible New icons lack transparent background Aug 19, 2024
@JordanConnor
Copy link

Updated with transparent background:
element-logos-alpha.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants