Skip to content

[Branding] Use new logo on menu bar#3788

Merged
huchenlei merged 1 commit intomainfrom
logo
May 6, 2025
Merged

[Branding] Use new logo on menu bar#3788
huchenlei merged 1 commit intomainfrom
logo

Conversation

@huchenlei
Copy link
Contributor

@huchenlei huchenlei commented May 6, 2025

image
image

┆Issue is synchronized with this Notion page by Unito

@huchenlei huchenlei requested a review from a team as a code owner May 6, 2025 17:47
@huchenlei huchenlei requested a review from webfiltered May 6, 2025 18:03
Copy link
Contributor

@webfiltered webfiltered left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wanted to approve this twice.

@huchenlei huchenlei merged commit 131fae7 into main May 6, 2025
10 checks passed
@huchenlei huchenlei deleted the logo branch May 6, 2025 19:18
@novaexe
Copy link

novaexe commented May 7, 2025

@huchenlei can we please get the ability to disable that, my thoughts about the logo aside that is a substantial amount of white/black in a single place and will cause burn-in over time on most monitors

@Comfy-Org Comfy-Org locked as resolved and limited conversation to collaborators May 7, 2025
@catboxanon
Copy link
Contributor

This breaks the branding guidelines laid out in https://blog.comfy.org/p/comfyui-native-api-nodes, does it not?

image

@webfiltered
Copy link
Contributor

webfiltered commented May 7, 2025

Merged PRs are unfortunately not a great place for discussion - happy for the PR to be linked in an issue / discussion, but talking about it here just ensures comments will not be seen by the vast majority of people.

Edit: To avoid wasting anyone's time, the logo colours are likely to be updated at some point soon.

@christian-byrne
Copy link
Contributor

christian-byrne commented May 7, 2025

happy for the PR to be linked in an issue / discussion, but talking about it here just ensures comments will not be seen by the vast majority of people.

Since this has been made clear, it seems like it's okay to unlock the conversation and unhide comments. I don't want to discourage feedback as we are community-driven always-open-source project -- ComfyUI is everyone's!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants