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

Comment Edit Link: Add spacing support #43657

Conversation

ndiego
Copy link
Member

@ndiego ndiego commented Aug 26, 2022

Related:

What?

Add padding and margin support to the Comment Edit Link block.

Why?

To create consistency across blocks.

How?

Added the relevant block supports in block.json

Note that box-sizing: border-box was not applied to this block like in other implementations. Since the Comment Edit Link block is always contained within the Comments Template, box-sizing: border-box did not seem to be needed in my testing.

Testing Instructions

  1. Insert a new Comment Edit Link block.
  2. Confirm the Dimension control panel allows you to add both padding and margin.
  3. Adding padding and margin.

Screenshots or screencast

comment-edit-link-spacing

The visualizers in the Site Editor are a little wonky, but not related to this PR.

@ndiego ndiego added [Type] Enhancement A suggestion for improvement. [Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi labels Aug 26, 2022
@ndiego ndiego self-assigned this Aug 26, 2022
Copy link
Contributor

@carolinan carolinan left a comment

Choose a reason for hiding this comment

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

theme.json, global styles and editing individual blocks work well.

Copy link
Contributor

@andrewserong andrewserong left a comment

Choose a reason for hiding this comment

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

This LGTM, too!

Since the Comment Edit Link block is always contained within the Comments Template, box-sizing: border-box did not seem to be needed in my testing.

Yes, the sizing appears to be working nicely for me in the editor and on the site frontend without needing to add the box-sizing rule 👍

image

I'll merge this in and update the tracking issue.

@andrewserong andrewserong merged commit 250cf8f into WordPress:trunk Sep 1, 2022
@github-actions github-actions bot added this to the Gutenberg 14.1 milestone Sep 1, 2022
@cbravobernal cbravobernal added the [Block] Comments Affects the Comments Block - formerly known as Comments Query Loop label Sep 13, 2022
@femkreations femkreations added the Needs User Documentation Needs new user documentation label Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Comments Affects the Comments Block - formerly known as Comments Query Loop [Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi Needs User Documentation Needs new user documentation [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants