Problem/Motivation

The footer type selector in the theme settings form lists components by name only (e.g. "Footer Two Columns"), with no indication of which theme they belong to. Since artisan_starterkit ships inside the contrib module and subthemes are generated from it, both the original starterkit components and the subtheme copies share identical names. This means duplicate, indistinguishable options appear in the selector even on a standard single-subtheme setup, before considering sites with multiple subthemes active.

The discovery logic in ArtisanCustomizations::themeSettingsAlter() (src/ArtisanCustomizations.php) scans all active themes via ComponentPluginManager::getAllComponents(), filters by group: Artisan Footer, and uses $component->metadata->name as the select option label with no theme context included.

User interface changes

Source of the component mus be displayed in the footer type selector.

CommentFileSizeAuthor
#3 footer-selector.png101.54 KBfrouco

Issue fork artisan-3590217

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:

Comments

frouco created an issue. See original summary.

frouco’s picture

Assigned: frouco » alzz
Status: Active » Needs review
StatusFileSize
new101.54 KB

Implementation has been completed.

Artisan Theme Footer selector

Ready for review.

omarlopesino’s picture

Status: Needs review » Needs work

The MR looks good to me. I added a suggestion (skippable if you disagree). It needs fixing the conflicts before reviewing.

omarlopesino’s picture

Status: Needs work » Fixed

Skipped the skippable suggestion and marked to merge.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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