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

Latest comments: Add color block support #63419

Merged
merged 2 commits into from
Jul 23, 2024
Merged

Conversation

carolinan
Copy link
Contributor

@carolinan carolinan commented Jul 11, 2024

What?

Adds text, background, gradient, and link color support to the Latest comments block.
Background, text and link are set to show by default.

Partial for #43245

Why?

The block is missing these options. Adding them improves the consistency and enables more design options.

How?

The PR updates the block supports in block.json, and the documentation.

Testing Instructions

Create a new post and add a latest comments block. Adjust the color settings and compare the result in the editor and front.
Next test the global styles options from the Styles sidebar in the Site Editor.

Screenshots or screencast

@carolinan

This comment was marked as resolved.

@carolinan carolinan added [Type] Enhancement A suggestion for improvement. [Block] Latest Comments Affects the Latest Comments Block labels Jul 11, 2024
@carolinan carolinan changed the title WIP Latest comments: Add color block support Latest comments: Add color block support Jul 17, 2024
@carolinan carolinan marked this pull request as ready for review July 17, 2024 03:53
Copy link

github-actions bot commented Jul 17, 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: carolinan <poena@git.wordpress.org>
Co-authored-by: ndiego <ndiego@git.wordpress.org>

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

@carolinan carolinan added the Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code label Jul 17, 2024
Copy link
Member

@ndiego ndiego left a comment

Choose a reason for hiding this comment

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

Works as expected. Nice enhancement.

image

@carolinan carolinan merged commit 3f7b260 into trunk Jul 23, 2024
71 checks passed
@carolinan carolinan deleted the add/latest-comments-color branch July 23, 2024 10:39
@github-actions github-actions bot added this to the Gutenberg 18.9 milestone Jul 23, 2024
westonruter added a commit that referenced this pull request Jul 23, 2024
* trunk: (2604 commits)
  Update "Versions in WordPress" page (#63869)
  SelectControl: Fix hover/focus color in wp-admin (#63855)
  Add margin-bottom lint rules for RangeControl (#63821)
  JSON Schema Docgen Rework (#63868)
  JSON Schema Reorganization and Fixes (#63591)
  DataForm: Add a simple story for the DataForm component (#63840)
  Quick Edit: Support bulk selection (#63841)
  Update dataviews docs (#63860)
  Bump the github-actions group across 1 directory with 4 updates (#63808)
  Add unit tests for the gutenberg_render_block_core_post_title() function.
  Make hover block outlines not present in Distraction Free (#63819)
  DataViews: Rename the header property of fields to label (#63843)
  Fix: Error while Calling edit-site getCurrentTemplateTemplateParts selector (#63818)
  Revert "Update HeightControl component to label inputs" (#63839)
  Zoom out: hide vertical toolbar when block is not full width (#63650)
  Latest comments: Add color block support (#63419)
  Core Data: Remove leftover 'todo' comment (#63842)
  Tabs: keep full opacity of focus ring on disabled tabs (#63754)
  Fix selected row styles in table layout (#63811)
  Align checkbox, radio, and toggle input design (#63490)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Latest Comments Affects the Latest Comments Block Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants