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 alignment issue with the comment edit link #40584

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

carolinan
Copy link
Contributor

What?

This is the equivalent of #40582 but for the comment edit link.

Assures that the comment edit link text alignment setting works and that the block displays correctly on the front.
Also removes duplicate escaping.

Partial for #40269

Why?

The block had duplicate font-size classes on the front. This caused the alignment class and the font size class to be output as one word without spacing in between them, and the CSS was not applied. Because of this, the alignment option did not work.

How?

By removing the extra output of the font size class, the class name for the text alignment is correct and the CSS applied.

Testing Instructions

  1. Add a comment query loop to a post or page that has comments.
  2. Select the inner comment edit link block.
  3. Test the toolbar option for text alignment.
  4. Test the font size option in the block settings sidebar.
  5. Save and view the front. Confirm that both the text alignment and the font size works.

Screenshots or screencast

After:
the comment edit link is aligned to the right

@carolinan carolinan added [Type] Bug An existing feature does not function as intended [Block] Comments Affects the Comments Block - formerly known as Comments Query Loop labels Apr 25, 2022
Copy link
Contributor

@cbravobernal cbravobernal left a comment

Choose a reason for hiding this comment

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

LGTM!

@cbravobernal cbravobernal merged commit d8fd70c into trunk Apr 26, 2022
@cbravobernal cbravobernal deleted the update/comment-edit-link-classes branch April 26, 2022 09:45
@github-actions github-actions bot added this to the Gutenberg 13.2 milestone Apr 26, 2022
@adamziel
Copy link
Contributor

This was backported to wp/6.0 branch to be included in WordPress 6.0 Beta 3 today: 1c6faeb

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 [Type] Bug An existing feature does not function as intended
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants