Skip to content

64877 : Default site icon fallback still uses blue logo#11283

Open
nirav7707 wants to merge 1 commit intoWordPress:trunkfrom
nirav7707:64877-update-wp-site-icon-fallback-image
Open

64877 : Default site icon fallback still uses blue logo#11283
nirav7707 wants to merge 1 commit intoWordPress:trunkfrom
nirav7707:64877-update-wp-site-icon-fallback-image

Conversation

@nirav7707
Copy link

@github-actions
Copy link

github-actions bot commented Mar 18, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props niravsherasiya7707, huzaifaalmesbah, nikunj8866, sabernhardt.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link
Member

@huzaifaalmesbah huzaifaalmesbah left a comment

Choose a reason for hiding this comment

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

I think it would be better to follow the implementation already used in WordPress core. See the changeset: r61989 Maybe we can update it here as well: L1750

@nirav7707
Copy link
Author

@huzaifaalmesbah Umm, but it should match the new primary color right?

@nikunj8866
Copy link

@nirav7707 I agree with @huzaifaalmesbah. We should follow the existing WordPress core implementation and use the grey color, as seen on the login screen, to maintain consistency.

@nirav7707
Copy link
Author

@huzaifaalmesbah @nikunj8866 The grey logo isn’t very visible. While it doesn’t directly impact accessibility, it requires extra effort to notice and could be improved for better visibility. What's your thought on this?
Screenshot 2026-03-19 at 10 54 18 AM

@sabernhardt
Copy link

The logo image needs a white background. See #11293.

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.

4 participants