-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add shadow support for column, columns and image #57982
Conversation
This pull request changed or added PHP files in previous commits, but none have been detected in the latest commit. Thank you! ❤️ |
Size Change: +104 B (0%) Total Size: 1.69 MB
ℹ️ View Unchanged
|
50da155
to
6fa2a0f
Compare
Would it be possible to add box shadow support to the Column block as well? I believe this would be a valuable addition to enhance individual column styling. |
6fa2a0f
to
936f2f9
Compare
This PR now enables support for those blocks, plus the Column block, and except the group block, which is in progress. We can use this WordPress Playground link to the Gutenberg PR previewer in order to play around with the changes introduced here.
Noted, and added! |
628fa23
to
6c802e5
Compare
67704f9
to
01af895
Compare
Flaky tests detected in 10e89bf. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7782529319
|
Investigated further on this. This is happening because of next group (when it has background), shadow goes under the next block and not visible because of the background. This is a theme specific issue and can be ignored. |
01af895
to
52156bd
Compare
52156bd
to
028ee37
Compare
028ee37
to
c0560de
Compare
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 Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @bgardner. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. Core SVNIf you're a Core Committer, use this list when committing to
GitHub Merge commitsIf you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
c0560de
to
e333d9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went ahead to revert some of the additional changes of column and column blocks because the serialization change #58306 is now merged.
Also, removing group and cover block related changes as more testing is required.
LGTM for column
, columns
and image
blocks.
2ce0b4f
to
4cfa087
Compare
Depends on: #57654
This PR adds box shadow support to the following blocks:
Cover— Requires additional work.Group- Requires additional testingIssue: #57103