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.

Sign up for GitHub

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

Jump to bottom

Update the Element logo to the correct file #27886

Closed
daniellekirkwood opened this issue Aug 1, 2024 · 6 comments · Fixed by #27900
Closed

Update the Element logo to the correct file #27886

daniellekirkwood opened this issue Aug 1, 2024 · 6 comments · Fixed by #27900
Assignees
@t3chguy
Labels
A-Appearance O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect T-Task Tasks for the team like planning

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Aug 1, 2024 •
edited
Loading

As discussed here we're using the wrong Element logo.

We should update the file to the correct instance...
@daniellekirkwood
Copy link
Contributor Author

The design team will shortly be adding a file with the correct logo

@dosubot dosubot bot added A-Appearance T-Task Tasks for the team like planning labels Aug 1, 2024
@t3chguy
Copy link
Member

t3chguy commented Aug 1, 2024

@daniellekirkwood please let the designer know these variants are needed for the logo: https://github.com/element-hq/element-web/tree/develop/res/vector-icons

@daniellekirkwood
Copy link
Contributor Author

@element-hq/design @amshakal & @JordanConnor please review the link from @t3chguy

@JordanConnor
Copy link

JordanConnor commented Aug 2, 2024 •
edited
Loading

Here's a file updating the icons without a background or "roundel" (i've left the full background ones):
element-web-icon-update.zip

@t3chguy
Copy link
Member

t3chguy commented Aug 2, 2024

Thanks @JordanConnor though that zip seems to not include a bunch of the necessary icons, apple-touch-icon-*, favicon.ico, mstile-* - should those be left untouched?

@JordanConnor
Copy link

We could update them all (but the favicon) for consistency: element-web-icon-update-all.zip

@t3chguy t3chguy self-assigned this Aug 5, 2024
@t3chguy t3chguy mentioned this issue Aug 5, 2024
Merged
@dbkr dbkr added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users labels Aug 5, 2024
@namazso namazso mentioned this issue Aug 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@t3chguy t3chguy

Labels
A-Appearance O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect T-Task Tasks for the team like planning
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update Element icons element-hq/element-web
4 participants
@dbkr @t3chguy @JordanConnor @daniellekirkwood