-
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
Lightbox filtering no longer working/isn't disallowing editing #59837
Comments
Thanks for opening the ticket, @annezazu. I had some more testing to do before deciding this was a definite regression, but you beat me to it. :) I've done some more testing with Twenty Twenty-Four. These are my {
"$schema": "https://schemas.wp.org/trunk/theme.json",
"version": 2,
"settings": {
"blocks": {
"core/image": {
"lightbox": {
"allowEditing": false,
"enabled": true
}
}
}
}
} As far as I can tell, this combination of settings is the only thing that differs between 6.4 an 6.5. With WordPress 6.4With the above With WordPress 6.5Using the same settings, the "Expand on click" button appears in the link dropdown in the toolbar (new location): |
Hey all, I created a PR to address the issue here. |
The fix was released in WordPress 6.5.2. |
Pulling this out of a separate issue and copying/pasting from @justintadlock (let's make sure to create separate issues here for 6.5 problems).
It looks like the filter itself works. However, the implementation in the editor might be broken. I'm testing against WP 6.5 RC2.
This combination isn't disallowing editing:
allowEditing
:false
enabled
:true
With those two settings, I'm still able to change the lightbox status of an existing Image block and a newly-added Image block in the editor.
Tested with filter:
I also tested this without the filter but setting it in
theme.json
:CC: @artemiomorales
This is a change in previous behavior from WordPress 6.4 and seems to be a regression and bug to me. I don't know if it's related to a change in the Lightbox toggle in the sidebar to the new location in the Link dropdown. #57608 Or something else altogether.
The text was updated successfully, but these errors were encountered: