- 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 9:28pm 15 December 2023 - 🇦🇺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 12:30am 16 December 2023 - 🇺🇸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 10:05am 16 December 2023 - 🇦🇺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 10:59am 16 December 2023 - 🇺🇸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 4:31am 18 December 2023