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

Fix block gap added to the block templates skip link #34986

Merged
merged 1 commit into from
Sep 21, 2021

Conversation

youknowriad
Copy link
Contributor

In trunk the skip link is added inside the wp-site-blocks div dynamically but that container block has "block gap" styles meaning the hidden link will have a visible "margin" added to separate from the other elements using the block gap config.

To avoid this conflict, this PR moves to the skip link outside the wp-site-blocks

Testing instructions

  • Use a block template with a main area.
  • Notice that there's no extra "gap" added at the top of the page in frontend.

@youknowriad youknowriad added [Type] Bug An existing feature does not function as intended Customization Issues related to Phase 2: Customization efforts [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. labels Sep 21, 2021
@youknowriad youknowriad self-assigned this Sep 21, 2021
Copy link
Member

@aristath aristath left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Note: This will need to be backported to WP so I'm adding the corresponding tag.

@aristath aristath added the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Sep 21, 2021
@youknowriad youknowriad merged commit 846e869 into trunk Sep 21, 2021
@youknowriad youknowriad deleted the fix/skip-link-block-gap branch September 21, 2021 10:10
@Mamaduka Mamaduka added this to the Gutenberg 11.6 milestone Sep 21, 2021
@noisysocks noisysocks removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 7, 2021
nylen pushed a commit to nylen/wordpress-develop-svn that referenced this pull request Nov 25, 2021
Fix Twenty Twenty-one having an erroneous margin above the Header template part
by moving the skip link outside of the wp-site-blocks canvas.

This is a backport of WordPress/gutenberg#34986.

Props youknowriad.
Fixes #54491.





git-svn-id: https://develop.svn.wordpress.org/trunk@52243 602fd350-edb4-49c9-b593-d223f7449a82
pento pushed a commit to WordPress/wordpress-develop that referenced this pull request Nov 25, 2021
Fix Twenty Twenty-one having an erroneous margin above the Header template part
by moving the skip link outside of the wp-site-blocks canvas.

This is a backport of WordPress/gutenberg#34986.

Props youknowriad.
Fixes #54491.





git-svn-id: https://develop.svn.wordpress.org/trunk@52243 602fd350-edb4-49c9-b593-d223f7449a82
markjaquith pushed a commit to markjaquith/WordPress that referenced this pull request Nov 25, 2021
Fix Twenty Twenty-one having an erroneous margin above the Header template part
by moving the skip link outside of the wp-site-blocks canvas.

This is a backport of WordPress/gutenberg#34986.

Props youknowriad.
Fixes #54491.




Built from https://develop.svn.wordpress.org/trunk@52243


git-svn-id: http://core.svn.wordpress.org/trunk@51835 1a063a9b-81f0-0310-95a4-ce76da25c4cd
gMagicScott pushed a commit to gMagicScott/core.wordpress-mirror that referenced this pull request Nov 25, 2021
Fix Twenty Twenty-one having an erroneous margin above the Header template part
by moving the skip link outside of the wp-site-blocks canvas.

This is a backport of WordPress/gutenberg#34986.

Props youknowriad.
Fixes #54491.




Built from https://develop.svn.wordpress.org/trunk@52243


git-svn-id: https://core.svn.wordpress.org/trunk@51835 1a063a9b-81f0-0310-95a4-ce76da25c4cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customization Issues related to Phase 2: Customization efforts [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants