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

Site Editor: Add a screen for adding or uploading a site logo / favicon #59298

Closed
andrewserong opened this issue Feb 23, 2024 · 6 comments
Closed
Labels
[Block] Site Logo Affects the Site Logo Block [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.

Comments

@andrewserong
Copy link
Contributor

andrewserong commented Feb 23, 2024

What problem does this address?

Currently, in the site editor, if a user wishes to use a site icon / favicon that is different to what's used by the Site Logo block, there is a link to direct users over to the customizer in order to update it. It would be nicer if there were a dedicated part of the site editor for adding / updating site icons explicitly, rather than using the customizer. Here is how it currently looks:

Site Logo block controls They redirect to the Customizer page
image image

What is your proposed solution?

Add a separate page / screen to the left-hand navigation of the site editor's browse mode for adding or updating site icons.

Note: as of WordPress 6.5, there will be the ability for folks to update the site icon from the general settings page of wp-admin (see trac ticket: https://core.trac.wordpress.org/ticket/54370) — this may or may not influence whether or not there should be a corresponding section in the site editor. (I.e. maybe it's fine if we're linking folks to go to the general settings page)

A few questions

  • What would the menu item be called?
  • Which controls would be available within the screen? I.e. would it just be for the site logo, or would it also include other controls for site title, tagline, etc?

An extremely quick mockup of an extra item on the left hand panel to indicate which area I mean:

image

I couldn't find an existing issue for this exactly, so if there is one, let me know and I'll close this out as a duplicate!

@andrewserong andrewserong added [Type] Enhancement A suggestion for improvement. Needs Design Needs design efforts. [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Block] Site Logo Affects the Site Logo Block labels Feb 23, 2024
@bgardner
Copy link

Would this complement the new UI on the General Settings page?

https://twitter.com/bgardner/status/1761077710426710038

@tomxygen
Copy link

tomxygen commented Feb 24, 2024

Would this complement the new UI on the General Settings page?

I think so.
As more things are placed in the site editor, I don't see why not adding the favicon as well.
The new UI in General Settings works fine, especially for those who are not using a block theme yet.

Also, what do you think about using the favicon/logo to replace the WP logo in wp-login?

@andrewserong
Copy link
Contributor Author

andrewserong commented Feb 25, 2024

Thanks folks, I hadn't seen that!

Would this complement the new UI on the General Settings page?

I'd think so — the main idea behind this issue is that it'd be good for folks not to have to leave the site editor in order to make changes.

@andrewserong
Copy link
Contributor Author

Just linking the trac ticket for the site icon in the general settings page: https://core.trac.wordpress.org/ticket/54370

@hanneslsm
Copy link

I couldn't find an existing issue for this exactly, so if there is one, let me know and I'll close this out as a duplicate!

I think the previous discussion would be this #50860 but it didn't go into details and solutions.

@jameskoster made a note for "global settings", which I like better than "Site identity". ("Site identity" and "Styles" would be too close)

@annezazu
Copy link
Contributor

This is a duplicate IMO of #50860. Going to close this out as a result and let's continue the discussion there. @andrewserong please add your thoughts as a comment!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Site Logo Affects the Site Logo Block [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

5 participants