#76 works on 9.5.11 as well! Thanks for the updated patch
#23 and #25 both wouldn't apply for me. Here is a new patch that works against 8x-3.1
@dsnopek: Thanks alot for trying to reproduce those issues! Perhaps something else in my environment is the culprit. I'll try and isolate the problem and get back to you on those first few issues. Thanks so much for your work on this.
Edit: I found this core issue that widens the off-canvas tray: ✨ Modify Layout Builder's use of the off-canvas tray to improve UX of larger block forms Needs work . Perhaps it would be a good for your Panopoly changes.
I tested #9 on an existing site and noticed a few incompatibilities with popular layout builder modules. I'm not entirely sure if it would need to be adjusted in the patch or if the modules would need to be adjusted to support the changes done here.
- When using Bootstrap Layout Builder → with the optional Layout Builder Blocks → dependency, configuring sections is no longer possible. The "configure section" button doesn't do anything.
- When adding sections, normally the configure section dialog should appear, but it no longer does with this patch enabled. Configuring a section is only possible when changing the layout.
- The manage layout part of this patch only shows text, without any of the visual elements (borders, etc). When changing layout (and if configuring sections would be possible most likely), the manage layout settings are shown on every tab. It should most likely only show these settings on the first tab.
- I tried adding a class to a section's row via Layout Builder Component Attributes → while changing a layout (since configuring isn't possible on my end as explained above), but the class is either not saved or the form is cleared every type you attempt to change the layout.
Otherwise the functionality of changing the layouts works great and am absolutely excited to see the progress on this. I hope this early feedback doesn't hinder your progress.
Also is it possible to somehow use that Panopoly patch without the Panopoly distribution?