Just ran into this issue and appreciate the previous work done here.
Would it perhaps be better to simply add the `type="button"` and `role="button"` attributes to the span instead of changing the element type? I can imagine scenarios where switching from span/button could have someone run afoul in previously implemented css/js selectors?
Or maybe I have this wrong and this plugin should be part of the Single Content Sync module?
frogdog_tech → created an issue.
Is it possible this also resolve the issue here with duplicate layout builder content? https://www.drupal.org/project/single_content_sync/issues/3436581 🐛 Layout builder content blocks are duplicated on import Needs work
frogdog_tech → created an issue.
frogdog_tech → created an issue.
Looks good -- but I don't think there are any maintainers to actually merge the change.
frogdog_tech → created an issue.
Updated and added merge request. Should work with 8.x-1.x-dev now.
Tested and confirmed #2 works great. Was also able to pass in the google_place_id via twig tweak params very nicely.
I had to remove the patch from: https://www.drupal.org/project/layout_builder_st/issues/3366070 🐛 PHP Fatal error: Trait method Active
This introduces the same Trait that already exists in layout_builder_st
thanks for this. tested and working in 8.x-1.15. huge time saver.
frogdog_tech → created an issue.