Problem/Motivation

This module requires Layout builder (and consequentially Layout discovery) to be enabled. Some websites may only want to have new navigation, but not necessarily the other modules it depends upon.

Proposed resolution

Refactor the navigation module in a way it only requires minimal set of modules by default so it can be used in minimal scenarios too.

Comments

KlemenDEV created an issue. See original summary.

klemendev’s picture

klemendev commented
Title: Mandatory requirement of layout builder and workflows » Mandatory requirement of layout builder and workspaces
Issue summary: View changes
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Active » Postponed (maintainer needs more info)

It uses layout builder to place blocks
You don't need to use it in any node types etc so there's no impact until you turn it on for a node type

It doesn't rely on workspaces

klemendev’s picture

klemendev commented
Title: Mandatory requirement of layout builder and workspaces » Mandatory requirement of layout builder
Issue summary: View changes
klemendev’s picture

klemendev commented

Sorry, I got confused and wrote the wrong module, I probably enabled multiple modules and thought it was navigation requesting that one.

Couldn't it (layout builder) be a soft dependency instead and if people don't want to place blocks using the new navigation, this feature is simply not shown in navigation entries?

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)

Using layout builder let us remove 4000 lines of code from navigation module. I can't see them being added back