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: Correctly reset the site icon #40046

Merged
merged 5 commits into from
Apr 6, 2022

Conversation

akasunil
Copy link
Member

@akasunil akasunil commented Apr 5, 2022

What?

If the logo image is reset or the site logo option "Use as site icon" is toggled off, the site icon is still used in the navigation sidebar in the site editor as per issue 39922. this pull request consist fix for site icon when it logo reset or "use as site icon" option is unchecked.

Fixes #39922.

Testing Instructions

Please follow step by step instructions to test this PR.

  1. Upload a site logo.
  2. Enable it as the site icon, save.
  3. Refresh and see that the site icon replaces the WordPress logo in the navigation sidebar.
  4. Disable the "Use as site icon" option, save.
  5. See that the site icon still replaces the WP logo.
  6. Reset the site logo image from the block toolbar, save.
  7. See that the site icon still replaces the WP logo.

@akasunil akasunil requested a review from ajitbohra as a code owner April 5, 2022 12:41
@github-actions github-actions bot added the First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository label Apr 5, 2022
@github-actions
Copy link

github-actions bot commented Apr 5, 2022

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @sunil25393! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

@Mamaduka Mamaduka added [Type] Bug An existing feature does not function as intended [Block] Site Logo Affects the Site Logo Block labels Apr 5, 2022
@Mamaduka Mamaduka changed the title Fix/39922 site icon Site Logo: Correctly reset the site icon Apr 5, 2022
@Mamaduka
Copy link
Member

Mamaduka commented Apr 5, 2022

Thanks for contributing, @sunil25393. I left a few suggestions that would be nice to include in this PR.

@Mamaduka Mamaduka requested a review from stacimc April 5, 2022 16:14
@paaljoachim
Copy link
Contributor

paaljoachim commented Apr 5, 2022

Thank you, thank you and again another thank you!
Some time ago I added a custom site icon (Twenty Twenty Two). Which replaced the W logo seen in the top left corner.
Even though I turned off the "Use as site icon" toggle for the Site Logo the custom icon was still seen. I was not able to get the W logo back.

Testing your PR and Toggling off the "Use as site icon" for the Site Logo the W returned.

This works well for me!

I thought I would inform both of you about this PR.
@annezazu @kjellr

@paaljoachim paaljoachim self-requested a review April 5, 2022 20:12
Copy link
Contributor

@paaljoachim paaljoachim left a comment

Choose a reason for hiding this comment

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

Feature fix works well!

@Mamaduka Mamaduka merged commit 113a809 into WordPress:trunk Apr 6, 2022
@github-actions
Copy link

github-actions bot commented Apr 6, 2022

Congratulations on your first merged pull request, @sunil25393! We'd like to credit you for your contribution in the post announcing the next WordPress release, but we can't find a WordPress.org profile associated with your GitHub account. When you have a moment, visit the following URL and click "link your GitHub account" under "GitHub Username" to link your accounts:

https://profiles.wordpress.org/me/profile/edit/

And if you don't have a WordPress.org account, you can create one on this page:

https://login.wordpress.org/register

Kudos!

@github-actions github-actions bot added this to the Gutenberg 13.0 milestone Apr 6, 2022
@akasunil
Copy link
Member Author

akasunil commented Apr 6, 2022

I have linked my github account to my wordpress profile.
https://profiles.wordpress.org/sunil25393/

Thank you.

@Mamaduka
Copy link
Member

Mamaduka commented Apr 6, 2022

Thanks, @sunil25393, and congrats on the first contribution 🎉

@akasunil akasunil deleted the fix/39922-site-icon branch April 7, 2022 04:42
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 First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Site icon is not removed from the Navigation Sidebar
3 participants