Een hart voor de Drupal Community

Thanks for the MR. Manually updated the fix for the new OOP hook implementations

Sven Decabooter added a comment on drupal.org: Use OOP hook implementations

Problem/Motivation

Use OOP hooks, as per [#3442349]

svendecabooter created an issue. See original summary.

Thanks - committed now.

This looks like a duplicate of #3347330: Modal Does not Close Automatically (possibly only on multilingual sites), caused by the absolute URL. Can you report back in that ticket if the problem still persists on the latest version of this module?

Sven Decabooter added a comment on drupal.org: Block translation don't open in modal with layout_builder_st

Did the addition of a dialog class in #3308949: Add a class to the dialog container help here in any way? Or would the patch by kpaxman still need to be applied to help resolve this issue?

This patch & MR no longer apply to the current 1.3.x state. I have also tested on a multilingual site (with paths instead of domains configured though), and cannot reproduce the issue. Can people who encountered this, retest with the latest version of this module, and report back if the problem still exists?

Can you check if this got resolved by the change in #3348594: Custom route to open in modal is restricted? That fix should also fix the issue with layout_builder_st

Thanks for the patch and rebase. Committed now.

Thanks for the patch. Committed now.

Thanks for the patch. Committed now.

The patch in attachment fixed the issue for me.

Problem/Motivation

When using this module I noticed there are console.log notices:

I think the code in highlight-custom.js can be altered to fix this when once is introduced in this logic.

f0ns created an issue. See original summary.

Brecht Ceyssens added a comment on drupal.org: Attach library to block

The library should not get loaded on every page, only when the block is used.

bceyssens created an issue. See original summary.

This patch fixed this behavior in the settings form for me.

Problem/Motivation

The default value of the Copy Button Background Color field is not set properly in the configuration form (it looks at copy bg transparent) which gives weird behavior / makes it impossible to set your own value.

f0ns created an issue. See original summary.

Hi julianriemersma,

Thanks for reporting the issue! I've finally gotten around to provide a fix, I had to fix it a bit differently because removing 'authenticated' from disallowed roles was creating some other issues.

I added a new release.

Sven Decabooter added a comment on drupal.org: Support language switcher and multilingual

Part of this is tackled in #3509600: Layout Builder translation issues - to fix the layout_builder_at issues currently in the project.

Sven Decabooter added a comment on drupal.org: Update to config split 2.x for patching capability

Sven Decabooter added a comment on drupal.org: Layout Builder translation issues

Problem/Motivation

Creating translations for a Component page do not seem to work: - Blocks from the original translations are not copied over - There are some issues with stale data

Proposed resolution

- Configure Layout field of Component page form display correctly - Update to latest layout_builder_at release, which fixes the other issues

svendecabooter created an issue. See original summary.

Thanks for the MR & reviews. Committed now.