Problem/Motivation

layout_builder_boolean_theme() is not needed thanks to layout_discovery_theme(), but only if the definition provides a template.
See \Drupal\Core\Layout\LayoutPluginManager::getThemeImplementations()

Proposed resolution

Add the template key to the deriver.

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Issue fork layout_builder_boolean-3231114

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

tim.plunkett created an issue. See original summary.

tim.plunkett opened merge request !1

danflanagan8 made their first commit to this issue’s fork.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented
Status: Active » Reviewed & tested by the community

I like it. The rare PR that results in less code.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented

Awesome, thanks for fixing!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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