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

CommentsPagination: Set font-size to inherit for pagination items #67296

Merged

Conversation

yogeshbhutkar
Copy link
Contributor

@yogeshbhutkar yogeshbhutkar commented Nov 26, 2024

Fixes: #67295

What?

Inside the Comments Pagination block, updating the font sizes from the Typography Controls do not get reflected in the Editor as well as the Front End. This PR aims to resolve that issue.

Why?

This PR addresses a bug that does not let the font sizes of the items contained in the Comments Pagination block change upon changing the font size from the Typography Controls.

More info about the bug is reported in the issue.

How?

Using font-size: inherit appears to resolve the issue.

Testing Instructions

  1. Create a Comments Block.
  2. Inside Comments Block, choose the Comment Pagination block and try changing the font sizes to Large or Extra Large. Observe the size change getting reflected both on the Editor as well as the front-end.
  3. The font size will be reflected on the next comment text, previous comment text and comment pagination numbers unless overwritten by changing font-sizes of each of those components separately, in that case, the separate styles will be prioritized.

Screenshots or screencast

Screen.Recording.2024-11-26.at.12.55.52.PM.mov

@yogeshbhutkar yogeshbhutkar marked this pull request as ready for review November 26, 2024 07:26
Copy link

github-actions bot commented Nov 26, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: yogeshbhutkar <[email protected]>
Co-authored-by: cbravobernal <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@youknowriad youknowriad added [Block] Comments Pagination Previous Affects the Comments Pagination Previous Block [Block] Comment Pagination Next Affects the Comments Pagination Next Block labels Nov 26, 2024
@cbravobernal cbravobernal added the [Type] Bug An existing feature does not function as intended label Nov 27, 2024
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. Tested and works as expected.

@cbravobernal cbravobernal enabled auto-merge (squash) November 28, 2024 09:37
@cbravobernal cbravobernal merged commit a48a414 into WordPress:trunk Nov 28, 2024
62 checks passed
@github-actions github-actions bot added this to the Gutenberg 19.9 milestone Nov 28, 2024
im3dabasia pushed a commit to im3dabasia/gutenberg that referenced this pull request Dec 4, 2024
…rdPress#67296)

* CommentsPagination: Set font-size to inherit for pagination items

* Remove extra spaces

---------
Co-authored-by: yogeshbhutkar <[email protected]>
Co-authored-by: cbravobernal <[email protected]>
michalczaplinski pushed a commit that referenced this pull request Dec 5, 2024
…7296)

* CommentsPagination: Set font-size to inherit for pagination items

* Remove extra spaces

---------
Co-authored-by: yogeshbhutkar <[email protected]>
Co-authored-by: cbravobernal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Comment Pagination Next Affects the Comments Pagination Next Block [Block] Comments Pagination Previous Affects the Comments Pagination Previous Block [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Comments Pagination]: The Typography setting SIZE for comments pagination block is not working.
3 participants