#57708 closed defect (bug) (fixed)
Site Editor: Global Style Button block Shadow presets is empty
Reported by: | franz00 | Owned by: | hellofromTonya |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | 6.2 |
Component: | Editor | Keywords: | has-screenshots has-patch gutenberg-merge has-testing-info commit |
Focuses: | Cc: |
Description
Shadow presets window is empty when trying to apply the presets for Button block in the Global Style.
Testing environment:
WP 6.2 beta 1, Firefox Dev Ed. 110.0b9, Chrome 110.0.5481.78, Windows 11, TT3 & Powder themes.
GIF image https://imgur.com/a/gDh828C
Steps to reproduce:
- Activate a block theme (TT3 or other)
- Click Global Styles
- Select Button block to customize
- Choose Shadow property
- Observe the Shadow presets window is empty
Attachments (2)
Change History (17)
#1
@
22 months ago
- Summary changed from Global Style Shadow presets window is empty to Site Editor: Global Style Button block Shadow presets is empty
#2
@
22 months ago
- Milestone changed from Awaiting Review to 6.2
Thanks @franz00! Moving to the 6.2 milestone for visibility and decision about whether to use this ticket as a tracking ticket for a possible backport from GB.
#3
@
22 months ago
@costdev I have checked gutendev and found no issues there. But the issue still remains on wordpress-develop side. Thanks
#4
@
22 months ago
Thanks @dhrupo! Yes it appears something was not merged to wordpress-develop
as noted in this comment.
@ndiego Do you want to use this ticket to for tracking the merge, or handle this separately?
#5
@
22 months ago
It looks like I missed some recent changes when syncing the theme.json
file to the core.
I will create a new patch to include everything for shadow presets.
This ticket was mentioned in PR #4114 on WordPress/wordpress-develop by @Mamaduka.
22 months ago
#6
- Keywords has-patch added
PR sync core shadow presets from the Gutenberg plugin. This should also fix shadow presets being empty in 6.2 beta3.
Trac ticket: https://core.trac.wordpress.org/ticket/57708
@Mamaduka commented on PR #4114:
22 months ago
#7
The failing E2E tests seem unrelated to the changes in this PR.
@hellofromTonya commented on PR #4114:
22 months ago
#8
These changes were made in Gutenberg PR https://github.com/WordPress/gutenberg/pull/46502 and this commit.
#9
@
22 months ago
- Keywords gutenberg-merge added
- Owner set to hellofromTonya
- Status changed from new to reviewing
#10
@
22 months ago
- Keywords has-testing-info added
- Version set to trunk
Testing Instructions
These steps define how to reproduce the issue, and indicate the expected behavior.
Steps to Reproduce
- Activate the Twenty Twenty-Three (TT3) theme.
- Open the Site Editor.
- Click anywhere in the content area to edit.
- Open the Global Styles UI.
- Select
Blocks
menu. - Select
Button
. - Select
Shadow
. - 🐞 Select
Shadow
> which will open the Shadow modal popup.
Expected Results
When testing a patch to validate it works as expected:
- ✅ The Shadow modal popup should include tools.
When reproducing a bug:
- ❌ The Shadow modal popup is empty, i.e. no tools.
Test Report Icons:
🐞 <= Indicates where issue ("bug") occurs.
✅ <= Behavior is expected.
❌ <= Behavior is NOT expected.
#11
@
22 months ago
- Keywords needs-testing removed
Test Report
Patch: https://github.com/WordPress/wordpress-develop/pull/4114
Environment
- OS: macOS
- WordPress:
trunk
- Localhost: wp-env (with Docker)
- Browser: Firefox, Edge, and Chrome
- Theme: TT3
- Plugins: none
Actual Results
Before Applying the Patch
trac57708-before.gif shows it in action.
Yes, can reproduce the reported issue ✅
After Applying the Patch
trac57708-after.gif shows it in action.
Yes, the patch resolves the issue ✅
#12
@
22 months ago
- Keywords commit added
Patch: https://github.com/WordPress/wordpress-develop/pull/4114
- Confirmed the
theme.json
changes from https://github.com/WordPress/gutenberg/pull/46502 are all included in this PR ✅ - Test Report confirms the patch resolves the issue ✅
Ready for commit 👍 Prepping now.
@hellofromTonya commented on PR #4114:
22 months ago
#14
Committed via https://core.trac.wordpress.org/changeset/55405.
@Mamaduka commented on PR #4114:
22 months ago
#15
Thank you, @hellofromtonya!
Reported upstream https://github.com/WordPress/gutenberg/issues/48061