Skip to content

Add Logo blocks in the headers#95

Merged
richtabor merged 2 commits into
mainfrom
Add-logo-blocks-in-header
Oct 3, 2022
Merged

Add Logo blocks in the headers#95
richtabor merged 2 commits into
mainfrom
Add-logo-blocks-in-header

Conversation

@amjadr360

Copy link
Copy Markdown
Contributor

Discussion: https://github.com/extendify/user-research/issues/6

This PR adds the logo to the header-title-nav-button.html and header-title-social-nav.html headers.

@richtabor

Copy link
Copy Markdown

Could you also adjust this here (menu/social spacing):

CleanShot 2022-09-29 at 16 17 10@2x

@amjadr360

amjadr360 commented Sep 29, 2022

Copy link
Copy Markdown
Contributor Author

Just for the information:

The spacing issue appears when the Gutenberg plugin is active. I think due to step spacing.
image

For that, we need to update the theme.json too.

@amjadr360

amjadr360 commented Sep 30, 2022

Copy link
Copy Markdown
Contributor Author

Could you also adjust this here (menu/social spacing):

This spacing issue is related to Gutenberg V14.2.0 because CSS for the gap (Block Spacing) is not rendering properly. This issue only appears in the Site Editor. If we set custom spacing in numeric, CSS renders fine.

I think it's due to Gutenberg V14.2.0 switched to using numbers: WordPress/gutenberg#44247

image

@richtabor richtabor merged commit d755df7 into main Oct 3, 2022
@amjadr360 amjadr360 deleted the Add-logo-blocks-in-header branch September 11, 2024 14:46
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.

2 participants