πŸ‡ͺπŸ‡ΈSpain @alexahumada

Account created on 11 October 2009, over 15 years ago
#

Recent comments

πŸ‡ͺπŸ‡ΈSpain alexahumada

I'm closing is as a duplicate, as I shouldn't get credit from a MR that was not merged and was solved in another issue.
Thanks for your review @vbouchet!

πŸ‡ͺπŸ‡ΈSpain alexahumada

I checked again on a clean install. Yes, it seems that MR15 solved my problem but introduces that another issue. Patch from 3342348 also solves this issue, sorry about the confusion. We should mark this as solved.

πŸ‡ͺπŸ‡ΈSpain alexahumada

Also, I should clarify code from the patch in #3 and in MR15 is not the same.
The patch just reverted changes from commit 10620a91 and was supposed to be a temporary fix and the MR is my proposed solution.

πŸ‡ͺπŸ‡ΈSpain alexahumada

That was a fast merge! Thanks @Daniel.Moberly!

πŸ‡ͺπŸ‡ΈSpain alexahumada

Adding a path in case somebody needs it before the MR gets merged.

πŸ‡ͺπŸ‡ΈSpain alexahumada

Hi @smustgrave,

sorry i forgot about the new interdiff policy.

The new patch can't be applied to Drupal 10 becouse some changes in code in 10.0.x and 10.1.x branches, notably some css classes present in #71 patch have been removed. Also, notable difference in size is caused by Drupal 10 using PostCSS to transpile pcss files to css using yarn β†’ , which require changes to new .pcss.css files.

I'll try to work on the remaining task we have in #77 and #85, but sadly this new patch only addresses Drupal 10 compatibility.

πŸ‡ͺπŸ‡ΈSpain alexahumada

grimreaper, take a look at https://www.drupal.org/project/layout_builder_browser/issues/3351162 πŸ› Undefined array key "" in Drupal\layout_builder_browser\Form\BlockForm Closed: duplicate , and try the patch I posted there, maybe it can solve your case temporarily until we find a better solution.

πŸ‡ͺπŸ‡ΈSpain alexahumada

MR 5 has been merged already, we only need a new release.
Thanks benhamelin and dineshkumarbollu!

πŸ‡ͺπŸ‡ΈSpain alexahumada

Maybe issue 3351162 is related to this.

That issue can't be reproduced on a fresh install, you need at least one extra language set to default (in my case ES set as default language and EN as secondary) and it's not caused by other installed modules.

πŸ‡ͺπŸ‡ΈSpain alexahumada

Can't be reproduced on a fresh install, you need at least one extra language set to default (in my case ES set as default language and EN as secondary).

πŸ‡ͺπŸ‡ΈSpain alexahumada

Quick patch reverting changes in 10620a91 in case somebody needs it before we find a better solution.

πŸ‡ͺπŸ‡ΈSpain alexahumada

I'm so sorry, I misunderstood your message.

There's another issue when adding a new block to a block category using the Place Block button in Layout Builder Browser settings, but is not related to this one.

πŸ‡ͺπŸ‡ΈSpain alexahumada

The issue is not related with jquery.once, it is caused by an undefined array key.

Logs info:
Warning: Undefined array key "" in Drupal\layout_builder_browser\Form\BlockForm->form() (line 101 /var/www/html/web/modules/contrib/layout_builder_browser/src/Form/BlockForm.php)

It's caused by commit https://git.drupalcode.org/project/layout_builder_browser/-/commit/10620... in src/Form/BlockForm.php, if we revert the changes in that file modal works as expected.

πŸ‡ͺπŸ‡ΈSpain alexahumada

alexahumada β†’ made their first commit to this issue’s fork.

Production build 0.71.5 2024