Hello. Thank you for this module. It saved our life earlier on our project.

Would you mind if I tweak LayoutBuilderCopyWidget a little so that I can jump in in a subclass?

There are two things I need to do:

1) I have an extension of the InlineBlock plugin, therefore checking for the start of the block id being 'inline_block:' is not working for me.
2) As the clone is happening I need to adjust the configuration that is stored in the component.

I will submit a patch for your consideration.

Thanks again.

CommentFileSizeAuthor
#3 make-layoutbuildercopywidget-more-overloadable-3086321-1.patch2.7 KBAndyThornton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AndyThornton created an issue. See original summary.

swentel’s picture

swentel CreditAttribution: swentel commented

Sure, always interested in ways to make this more flexible!

AndyThornton’s picture

AndyThornton CreditAttribution: AndyThornton commented
FileSize
make-layoutbuildercopywidget-more-overloadable-3086321-1.patch2.7 KB
swentel’s picture

swentel CreditAttribution: swentel commented
Status: Active » Needs review
swentel’s picture

swentel CreditAttribution: swentel commented
Status: Needs review » Reviewed & tested by the community

Works for me if it works for you :)

I'll commit next week - on holiday this week.

  • swentel committed 14e6a44 on 8.x-2.x authored by AndyThornton 
    Issue #3086321 by AndyThornton: Make LayoutBuilderCopyWidget more...
swentel’s picture

swentel CreditAttribution: swentel at eps & kaas for Dropsolid commented
Status: Reviewed & tested by the community » Fixed

committed and pushed, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.