- Issue created by @Hydra
- Merge request !5#3411037 Remove event listener 'drupalContextualLinkAdded' from window, so... β (Open) created by Hydra
- last update
12 months ago 40 pass - π©πͺGermany Hydra
I decided to add this as a MR for this module, so people can get a patch for the workaround and don't need to add the code by them self.
- π©πͺGermany tomsaw Essen
Same here. Thanks for the workaround @Hydra! Saved my evening.
- πΊπΈUnited States millerrs
Saved me as well. Content editors reported this issue after updating to Drupal 10.2. A big thank you to @Hydra. Also, let's escalate the severity of this issue. Without this patch, users cannot translate layouts, which is a key functionality of this module.
- πͺπΈSpain joe_carvajal Seville (Spain)
This one did the trick for me too, thank you!
- Status changed to RTBC
11 months ago 5:56pm 12 February 2024 - Status changed to Needs work
11 months ago 11:34am 14 February 2024 - πΊπΈUnited States kmonty San Francisco, CA
I'd say this is `Needs Work`, as this is an `ugly, temporary solution`.
At the very least, it should remove the dependency on jQuery.
- π¨π¦Canada OwilliwO Montreal
Patch is working well on my english/french site !
Thanks a lot.Here is my "Layout Builder Stack":
Layout Builder (layout_builder) Enabled 10.2.2 Layout Builder Restrictions (layout_builder_restrictions) Enabled 8.x-2.19 Layout Builder Restrictions By Region (layout_builder_restrictions_by_region) Disabled 8.x-2.19 Layout Builder Symmetric Translations (layout_builder_st) Enabled 8.x-1.0-alpha3 Layout Builder Styles (layout_builder_styles) Disabled 2.0.0
- Status changed to RTBC
10 months ago 12:48pm 1 March 2024 - π¦πΉAustria fago Vienna
I agree, the dependency on jquery should not block moving on here. Setting back to RTBC then.
- π¨π¦Canada slideaway
@hydra Could you clarify what I need to do to install this fix? I've never installed a fix like this and haven't had any luck trying to implement it.
- π©πͺGermany Hydra
@slideaway Oh you need to download the patch from the merge request. Here is some documentation on how to do this: https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... β And of course you need to apply the patch to your version of layout_builder_st.
- last update
7 months ago 40 pass - π§π·Brazil carolpettirossi Campinas - SP
Downloading the diff and uploading the patch to your local repo is the recommended approach.
However, I'm attaching the patch from the MR here in case someone is used to this old approach of adding patches to the issue.The code works successfully in my case. Can't wait to see this merged and released.
- π¨π¦Canada slideaway
I was able to successfully add the patch and got things working again, thanks for aiming me in the right direction @hydra.
And thank you @carolpettirossi, I didn't actually know that was the old approach versus the new preferred way. I'd just been adding them to my composer.json using composer-patches from support threads like this, so I was unfamiliar with how to do it the way Hydra had mentioned.
In my case it doesn't work at all, the tranleta option still doesn't appear or maybe I need to configure something else
- last update
6 months ago 37 pass, 2 fail - π§π·Brazil igorgoncalves
@carolpettirossi patch fixed the contextual link issue and they're back!
Thanks carol.But in my project it ending up showing another issue after trying to editing blocks using those links, and this issue/patch helps me:
https://www.drupal.org/project/drupal/issues/3208687 β
sharing just in case another one faces the same.
- π¨πSwitzerland juagarc4
Hi all,
The patch #14 works in my project properly and no other issues were found by testing it.
Context:
Drupal 10.3.5
PHP 8.3+1 on RTBC.