Layout Builder checkmark status should be on

Created on 15 December 2023, 11 months ago
Updated 18 December 2023, 11 months ago

When the Umami demo profile website has been installed and you view this URL:

/admin/structure/types/manage/recipe/display

Use Layout Builder is unchecked. Also Allow each content item to have its layout customized fails to display when Use Layout Builder is checked on, as is normal behavior. Both checkboxes should be checked on to remove the bug. Regardless of this behavior, Layout Builder has been enabled for the Recipe Content Type, and individual pages can be uniquely configured as well.

Skip to number #14 📌 Layout Builder checkmark status should be on Active

This report has been edited as the bug has been better defined.

Problem/Motivation

Steps to reproduce

Proposed resolution

📌 Task
Status

Closed: works as designed

Version

11.0 🔥

Component
Umami 

Last updated 16 days ago

Created by

🇺🇸United States amstercad

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @amstercad
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    The recipe page is using layout builder

    I think this is won't fix

  • 🇺🇸United States amstercad

    The recipe Content Type is not using Layout Builder as a Display, so I don't see how Layout Builder is being used. Thank you @larowlan for your reply, I hope you can clarify further.

    I think if I used Configuration Management and had some time and patience, I could strip out the requirement from the repo.

  • Status changed to Postponed: needs info 11 months ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    The recipe (singular) page uses layout builder

    https://git.drupalcode.org/project/drupal/-/blob/11.x/core/profiles/demo...

    I still assert this is won't fix

  • 🇬🇧United Kingdom longwave UK

    Agree this should be won't fix, Umami is designed to showcase various features of core and Layout Builder is one of those features, SDC can be another and they can coexist side by side?

  • Status changed to Active 11 months ago
  • 🇺🇸United States amstercad

    Of course both SDC and Layout Builder can be used alongside each other, however Layout Builder is not actually being used in the Umami demo website. The CM code Larowlan points to are just legacy settings that are wholly unused now. Those are format settings someone can select if they want, but they aren't actually being used. Maybe I am wrong. If so, someone please show me what Layout Builder is actually being used for.

    Calling Layout Builder a requirement when it actually isn't being used isn't fair for people trying to learn Drupal Best Practices by reverse-engineering this working code. The new implementation of Umami demo makes not only much use of SDC but also View Modes. Previously Layout Builder was used. The new Umami demo website code functions very much like Mario Hernandez has presented and documented using the legacy Components module.

    I've spent much time recently reverse-engineering the Umami demo website recently to better implement SDC myself. This code is the best 'documentation' for getting SDC to work myself. Seeing legacy Layout Builder cruft is confusing for people in my situation.

    Don't get me wrong, I really like Layout Builder. I just don't think it is fair to keep this legacy, unused cruft as a pre-installed requirement for no good reason.

  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Layout builder is neither legacy or cruft

    If you visit node/6/layout after installing umami you can drag and drop things around

    It is most definitely in use

  • 🇺🇸United States amstercad

    Yes, I see you are correct larowlan. Thank you for the example. It works as designed, independently of SDC.

    Still, I see a bug worth noting here. If you look at this URL:

    /admin/structure/types/manage/recipe/display

    Use Layout Builder is unchecked. Also, Allow each content item to have its layout customized, fails to display when Use Layout Builder is checked on, as is normal behavior. Both checkboxes should be checked on to remove the bug.

    The reason it works this way is because Configuration Management works with the current code during the installation of the website profile. Had the buttons worked as expected, I would not have been fooled as I was.

  • Status changed to Closed: works as designed 11 months ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Ah, that's confused me before

    Check the full tab, that's used for the node page.

    I'm so used to default being used instead I got lost during some user testing Lauriii was conducting

  • Status changed to Active 11 months ago
  • 🇺🇸United States amstercad

    I disagree, this issue is not working as designed. I've clarified the bug. Those checkmarks should be displayed as being on and they're not by default. True, Layout Builder is working as designed, however those checkmarks displaying incorrectly are the bug. If you agree, please reopen.

    Yes, I agree the Layout tab displaying on the node page is working as designed.

  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Go to admin > structure > types > recipe > manage display > full

    The checkbox is on thete

  • 🇺🇸United States amstercad

    Thank you larowlan. The situation is clear to me now. There are no bugs and everything works well as designed. This issue is closed.

    Ah, that's confused me before

    You and me both. No doubt as SDC becomes more mainstream a search engine will propose this thread to someone, so I'll clearly document what's happening as it is not obvious.

    Whether or not Layout Builder or SDC renders the Content Type depends on a checkbox setting, for each View Mode.

    There are 3 Content Types in the Umami website example. In each Content Type only one of the View Modes use Layout Builder, the rest use SDC if they're used at all. Only the Full content View Mode uses Layout Builder.

    For example, these are the View Modes used by the Recipe Content Type:

    Default
    Card
    Card Common
    Card common alt
    Full content*
    Teaser

    * The Layout Builder checkbox is on.

  • Status changed to Closed: works as designed 11 months ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    No worries, thanks for following up

Production build 0.71.5 2024