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 Logo: Make it possible to set / update / remove via the Inspector #49273

Closed
jameskoster opened this issue Mar 22, 2023 · 10 comments · Fixed by #49992
Closed

Site Logo: Make it possible to set / update / remove via the Inspector #49273

jameskoster opened this issue Mar 22, 2023 · 10 comments · Fixed by #49992
Assignees
Labels
[Block] Site Logo Affects the Site Logo Block

Comments

@jameskoster
Copy link
Contributor

jameskoster commented Mar 22, 2023

What problem does this address?

The Site Logo block makes it possible to set a logo or update an existing one, but only via the block toolbar. If you're mid-workflow flicking between the canvas and Inspector this can feel a little unexpected.

What is your proposed solution?

Make it possible to set / update / remove the site logo (and site icon) via the block Inspector.

When setting a site logo initially, we have a new "Media" panel in the inspector:

Site logo inspector, Setting initially

The Add media button would mainly do the same as the "Upload" button in the canvas.

Once an image is set, the file and a small thumbnail is shown in the inspector:

Site logo inspector, editing existing


This issue was updated Apr 13.

@jameskoster jameskoster added Needs Design Needs design efforts. [Block] Site Logo Affects the Site Logo Block labels Mar 22, 2023
@jameskoster jameskoster moved this to Has issue in 6.3 Design Mar 22, 2023
@carolinan
Copy link
Contributor

Is the new position replacing the option in the toolbar, or should the option be in both places?

@jameskoster
Copy link
Contributor Author

I think both. I suspect that you'd mostly interact with the toolbar to do this, but it's an option that would also feel very natural in the Inspector.

@jasmussen

This comment was marked as resolved.

@jasmussen jasmussen moved this from Needs design to Needs feedback in 6.3 Design Apr 12, 2023
@jasmussen jasmussen added Needs Design Feedback Needs general design feedback. and removed Needs Design Needs design efforts. labels Apr 12, 2023
@carolinan
Copy link
Contributor

🤔 should it have a 3 dot menu for opening the additional options? Or is it obvious enough that clicking the miniature image opens these extra options?

@jasmussen
Copy link
Contributor

jasmussen commented Apr 12, 2023

My instinct is that the filename suggests that it's interactive, but we can always augment it with iconography if we find that to be insufficient.

@carolinan
Copy link
Contributor

carolinan commented Apr 12, 2023

I assume in the long run, we don't want inconsistencies; for example, the other media blocks and the file block would need similar settings.

@jameskoster
Copy link
Contributor Author

Seems like a good place to start. Would the 'Add media' button also be a drop zone?

@jasmussen

This comment was marked as resolved.

@jasmussen jasmussen added Needs Dev Ready for, and needs developer efforts and removed Needs Design Feedback Needs general design feedback. labels Apr 13, 2023
@jasmussen jasmussen moved this from Needs feedback to Needs dev in 6.3 Design Apr 13, 2023
@jasmussen
Copy link
Contributor

I've updated this issue with mockup and marked it ready for dev.

@paaljoachim
Copy link
Contributor

paaljoachim commented Apr 30, 2023

I am adding the comment here to tie in the comment with this issue.

Currently adding background image or video (file or embed) issues have many different solutions. With the common thread that these issues are getting old and have gone no where.

Expanding the media option added through this issue to for instance the Cover block and the Group block could be one solution to solving this issue:
Add Youtube or similar video service using an embed URL in the background of the Cover block
#28860
as well as this
Group Block - add support for Background Images
#14744

I mentioned the Add Youtube video service issue in the Design Slack channel here: https://wordpress.slack.com/archives/C02S78ZAL/p1682152917077289 @jasmussen happen to mention the issue for the Site Logo adjustment. As well as this:

Agreed, we need a bit of a generalized interface for backgrounds that can work across cover and groups. I think this seemingly unrelated interface might actually work there, showing the filename in the inspector to provide an “input” for the background.

@ntsekouras Perhaps you could create two new PR's to where we can explore adding a URL for a video service and the other PR for adding a background image to the Group block?

Perhaps for the Cover and Group block when clicking Add media it would not open the Media Library but the drop down instead. Something along the lines of this:
Add Media drop down

Clicking the Add Media button would show the drop down options of going to the Media Library, Uploading or embed URL. It seems like a simple solution for adding a video to the Cover block. The Add Media option in the toolbar would also need to be adjusted.

For the Group block one would then add the new panel Media with the options it contains.

@jasmussen jasmussen moved this from Needs dev to Done in 6.3 Design May 4, 2023
@priethor priethor removed the [Status] In Progress Tracking issues with work in progress label May 11, 2023
@priethor priethor removed the Needs Dev Ready for, and needs developer efforts label Jul 24, 2023
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
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants