Hello,

when for a content type new core layout builder is activated, all the field groups aren't displayed.
Only the fields inside the field group are rendered.

Comments

dercheffe created an issue. See original summary.

sime’s picture

Status: Active » Postponed (maintainer needs more info)

Please confirm this against the latest development version, as there are layout builder bugs that have been fixed.

p4trizio’s picture

I can confirm this issue even with latest development version

matt_paz’s picture

Also confirming that latest dev doesn't seem to work with 8.7 LB

matt_paz’s picture

Version: 8.x-3.0-beta1 » 8.x-3.x-dev
Category: Bug report » Feature request
Status: Postponed (maintainer needs more info) » Active

> Please confirm this against the latest development version, as there are layout builder bugs that have been fixed.

In addition to testing with 8.7.1 LB, I went back and took a look at some of the prior commits.
I could find some that reference layouts, but not layout builder.
I couldn't see where any layouts were defined (ala layouts.yml) nor anything anything that uses Drupal\Core\Layout\LayoutDefault

In field_group_field_ui_form_params I did find this ...

  // Get possible regions.
  $layout_info = $display->getThirdPartySettings('field_layout');
  if (!empty($layout_info) && isset($layout_info['id'])) {
    $layout = \Drupal::service('plugin.manager.core.layout')->getDefinition($layout_info['id']);

... but that was all I noticed.

This led me to reframe this as a prospective feature request for compatibility with Layout Builder, but if I'm mistaken, please update accordingly.

matt_paz’s picture

seanb’s picture

I can confirm updating to the latest dev version fixed the issue for me. Thanks!

matt_paz’s picture

@seanB - Any chance you could provide a screenshot of field_group working with layout_builder?

I'm unable to use to configure any field groups from layout_builder using Drupal 8.7.3 and latest field_group 3.x-dev (b245b85f99c48221edf5fb60344dfe53737ae0f9).

matt_paz’s picture

matt_paz’s picture

Please confirm this against the latest development version, as there are layout builder bugs that have been fixed.

I went back and reviewed the work in #2878359 too.
Not sure if other layout related commits made to dev, but it seems like work was only pursued to co-exist with field layout, but not layout builder.

I hope that helps clarify the original request, but let me know if I'm mistaken.

matt_paz’s picture

Just ran across a potentially interesting work around in #layouts in slack.

There’s a *Chaos Tools > Entity view (Content)* block that you can place in a Layout that will allow you to render your node in a specific view mode. So I’ve been actually placing that *Entity view (Content)* block in my layout and using different view modes for them, which saves you creating a view just to have some fields and eliminates that overhead of having more views as well.

- via nord102

It sounds like some folks are using LB on the default/full display, but then adding a different display to load the same node in a block with a conventional display (at which point I think field_group should load LB just fine). Folks are using it to control layout for a select group of fields, while leveraging the power of LB for additional content. I wonder about performance/overhead, but in theory, this might yield a prospective solution that gets the best of both worlds.

AndyThornton’s picture

Hi. I am trying to use Field Group on the backend, to group fields on my Layout Builder blocks forms to make life easier for editors. I realize it might be hard to say, but ... does this bug feel like an existential issue for Field Group? It doesnt seem like Field Group is even getting a look in, as the HTML of the front just looks like it would if FG wasn't enabled.

epicflux’s picture

StatusFileSize
new1.28 KB

Here's a patch to get field groups to show up on the Layout Builder add/update block forms.

epicflux’s picture

StatusFileSize
new1.38 KB

Adjusting the patch to avoid errors with non-custom block types.

philosurfer’s picture

#14 is working for me.
I am able to add all field group types.

Thank you for this patch!

rfsbsb’s picture

I can confirm #14 works for me as well.

andypost’s picture

Status: Active » Needs work
Issue tags: +Needs tests
swentel’s picture

Status: Needs work » Needs review

Checking the bot

providence_matt’s picture

I tested the patch in #14 on and it has mostly worked for me. The tabs widget, especially vertical tabs do not display correctly. The rest of the field group widgets display and function as intended.

adriancid’s picture

StatusFileSize
new1.36 KB

Rerroling the patch

nils.destoop’s picture

I tested the patch, and everything looks ok except for the horizontal tabs. They are rendering correctly, but the hidden css gets overridden by css of builder, resulting in all tabs being displayed. I'm going to check this issue and after that commit the patch.

corn696’s picture

Fresh installation D8.8.1 new core composer template, standard profile, language german, layoutbuilder and latest dev of fieldgroup with patch 14 or 20 installed.

Activated layout builder for article.
Added groups of all types.

I am not able to find the groups under manage layout.

justcaldwell’s picture

Title: Field groups are not displayed in layout builder » Field groups are not displayed in layout builder block config forms

@Corn696 I was confused by this, too. Only after reading the whole thread a couple times did I understand that this patch only enables field groups on custom block configuration forms displayed within the Layout Builder sidebar.

Unless I am mistaken, you can't use Field Groups to group node/content fields in Layout Builder displays, unless you do something like the approach described in #11.

I tweaked to the issue title a bit to try to clarify. The description could probably use a little work, too.

  • nils.destoop committed a82665c on 8.x-3.x authored by epicflux
    Issue #3044595 by epicflux, adriancid, matt_paz, dercheffe, philosurfer...
nils.destoop’s picture

Status: Needs review » Fixed

I modified a bit the javascript of horizontal tabs to match the modifications that core did for vertical tabs. Tabs are now working for the builder block forms. I committed the patch

nils.destoop’s picture

Status: Fixed » Closed (fixed)

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

murz’s picture

Could anyone please share a screenshot of how this should look? Tried all the things, but couldn't mange to get the field groups visible.