-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Lightbox UI appearing with interactivity experiment disabled #51025
Lightbox UI appearing with interactivity experiment disabled #51025
Conversation
429e289
to
b33d064
Compare
b33d064
to
775543c
Compare
Size Change: +23 B (0%) Total Size: 1.39 MB
ℹ️ View Unchanged
|
Flaky tests detected in db6c878. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5095560839
|
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.
Solves the issue as the PR describes 👍
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.
Looks great to me 🙂 Thanks a lot for the fix!
Cherry picked for Gutenberg 15.9 RC2 |
* Fix lightbox ui appearing with interactivity experiment disabled * Update e2e tests to enable the experiment
…ss#51025) * Fix lightbox ui appearing with interactivity experiment disabled * Update e2e tests to enable the experiment
What?
A Lightbox feature for images has been recently merged into Gutenberg for the 15.9. This feature can be enabled with a dropdown on advanced settings of the Image block. As it requires the Interactivity API, it only works if you have the experiment enabled.
Lightbox option:
Experiments required to be enabled:
Right now, the Behavior UI interface appears even when the experiment is disabled. As we have only one behavior, and requires the Interactivity API to be enabled, it shouldn't be like that. This PR fixes.
How?
Hiding the behaviors UI if the Interactivity API is disabled.
Testing Instructions